Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 76736 - cdargs-1.31.ebuild (New Package)
Summary: cdargs-1.31.ebuild (New Package)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Gentoo Shell Tools project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-05 00:22 UTC by Stefan Kamphausen
Modified: 2005-01-07 02:33 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
cdargs-1.31.ebuild (cdargs-1.31.ebuild,1.50 KB, text/plain)
2005-01-05 00:24 UTC, Stefan Kamphausen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Kamphausen 2005-01-05 00:22:28 UTC
Dear Gentoo Developers and Users,

please find attached my first try at an ebuild, an ebuild for (my own) cdargs program.

CDargs provides bookmarks and a browser for the shells built-in cd-command.  Bash users will also find the possibility to copy and move files to locations stored in their bookmarks.

Since this is my first ebuild it surely needs the keen eye of an experienced gentoo developer :-)

I suggest app-shells/cdargs as the portage directory.

Find cdargs at http://www.skamphausen.de/software/cdargs

Kind Regards
Stefan
Comment 1 Stefan Kamphausen 2005-01-05 00:24:57 UTC
Created attachment 47685 [details]
cdargs-1.31.ebuild
Comment 2 Ciaran McCreesh 2005-01-06 16:56:43 UTC
Hm, when did this one come in? I've got an ebuild in my overlay for this. It's basically the same but doesn't violate all the whitespace rules :) No need for einstall either, make DESTDIR=${D} seems to be fine.
Comment 3 Stefan Kamphausen 2005-01-06 23:14:52 UTC
Hi, 

I submitted this ebuild on 2005-01-05 because I wanted cdargs to appear in gentoo but didn't find any ebuild for it.  Thus I wrote it myself, not knowing about whitespace rules (is it indention?) and that einstall should be avoided.  

Regards
Stefan
Comment 4 Aaron Walker (RETIRED) gentoo-dev 2005-01-07 02:29:20 UTC
Stefan, don't worry about it.  It was actually a damn good attempt at a first ebuild.

Just to further explain on what ciaranm said, there were a few whitespace errors (I think there was one line with a trailing char) and yes, the indentation (ebuilds should be indented 4 chars).  As far as einstall goes, it is ok to use it but only in cases where the Makefile does not support the DESTDIR variable.  I'm pretty sure that any Makefile produced by GNU autotools supports it, however (unless upstream adds local hooks that don't respect DESTDIR).

I've just committed it as app-misc/cdargs, as I feel that is more appropriate than app-shells.  In addition to the small fixes already mentioned, I also slightly changed the wording of the pkg_postinst message.

Thanks.
Comment 5 Stefan Kamphausen 2005-01-07 02:33:50 UTC
Cool :-)

Thanks and Regards
Stefan