| Summary: | Updated Ebuild: net-www/bookmarksync-0.3.3.ebuild | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | M3Dlor <gentoo> |
| Component: | New packages | Assignee: | Josh Glover (RETIRED) <jmglov> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | gentoo, miles |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
net-www/bookmarksync-0.3.3.ebuild
bookmarksync-0.3.3.ebuild net-www/bookmarksync-0.3.3.ebuild (Update) net-www/bookmarksync-0.3.3.ebuild (Update) net-www/bookmarksync-0.3.4.ebuild (Version bump) |
||
|
Description
M3Dlor
2004-04-15 01:02:14 UTC
Created attachment 29313 [details]
net-www/bookmarksync-0.3.3.ebuild
I have made an ebuild for bookmarksync-0.3.3, which copies documents and optionally installs the bookmarksync.pl. Attachment to follow. (I would have just suggested to do this, but I didnt see this bug report) Created attachment 33802 [details]
bookmarksync-0.3.3.ebuild
Thanks Lubin This new ebuild is given upstream and will be included in feature versions. I will take this one. It is not really a web app, and I use it. :) Created attachment 33815 [details]
net-www/bookmarksync-0.3.3.ebuild (Update)
Miles, your ebuild looks good except for a minor issue:
You must add a RESTRICT="nomirror" line when adding an ebuild for a package
from SourceForge or the like (if the URI starts with mirror://).
This is to avoid having the Gentoo mirrors duplicate existing mirrored content.
Who are we to steal sf.net's traffic? :)
Info: There will ne a new version of bookmarksync the next days. Created attachment 33817 [details] net-www/bookmarksync-0.3.3.ebuild (Update) Repoman (bless its boots), caught a few things that I missed: RepoMan scours the neighborhood... Traceback (most recent call last): File "/usr/bin/repoman", line 735, in ? mydep=portage.dep_check(myvalue,portage.db["/"]["porttree"].dbapi,repoman_settings,use="all",mode=matchmode) File "/usr/lib/portage/pym/portage.py", line 3290, in dep_check mysplit=dep_opconvert(mysplit,myusesplit,mysettings) File "/usr/lib/portage/pym/portage.py", line 2897, in dep_opconvert if k[0]=="!": IndexError: string index out of range OK, that is not so nice, but Bug #36857 (http://bugs.gentoo.org/show_bug.cgi?id=36857) tells us that it is because of the lines: RDEPEND="${DEPEND} perl ? ( dev-lang/perl )" There can be no space between the USE variable that is being checked and the "?" operator. Even though I prefer the space myself, Repoman is boss. So we change that to: RDEPEND="${DEPEND} perl? ( dev-lang/perl )" And now Repoman can give us some useful output: ebuild.minorsyn 1 net-www/bookmarksync/bookmarksync-0.3.3.ebuild: Trailing whitespace Syntax Error. Line 26 And sure enough, there is a single space on the end of line 26. Deleted that sucker. :) Miles, your ebuild looks really good, other than the minor issues that Repoman picked up. I would recommend that you run Repoman against ebuilds before submitting them to Bugzilla. From the net-www/bookmarksync directory in your Portage overlay directory, run: PORTDIR_OVERLAY="/usr/local/portage" repoman fix Cheers, Josh This new ebuild is in CVS. Thanks, Miles and M3Dlor. M3Dlor, are you the upstream author? Yes. I am.. added some patch to source today. Will include fixed ebuild and open new bug when new version is available. M3Dlor, cheers for the excellent utility. I am working on using bookmarksync and net-www/online-bookmarks to keep my various bookmarks constantly synched and accessible. I will give you the details when I have worked something out. When you have new revisions, please re-open this bug. No need to clog up Bugzilla with a bug for every revision bump. I will get new revisions into CVS as quickly as you can re-open this bug! :) Thx for all the pointers, I did run repoman, but I got that nasty error and didnt know what to do. Created attachment 33876 [details]
net-www/bookmarksync-0.3.4.ebuild (Version bump)
Version bump to 0.3.4.
Ebuild seems not to be working this minute, because sourceforge mirrors dont
have the file yet.
As requested i reopened the bug due to version bump. M3Dlor: I will handle the bump tonight when I get home (I am in GMT -4). That should give the source enough time to get propagated through the SourceForge mirrors. Miles: glad that you tried to run Repoman at all! Very few users do that before submitting ebuilds. The error is nasty, but at least they know about it in Bugzilla. :) OK, 0.3.4 is in CVS! M3Dlor, I must say that you have been one of the most hands-up upstream authors with whom I have had the pleasure of working. Cheers! One small note, when a new version does not need a new ebuild, such as in the case of 0.3.3 -> 0.3.4, you can just put in a comment: "New version uses net-www/bookmarksync-0.3.3.ebuild (Update), attachment ID 33817". Then, you do not have to create a new attachment. Closing this bug until the next version. :) |
