Bugzilla Bug 76736

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

Created an attachment (id=47685) [edit]
cdargs-1.31.ebuild

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.

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

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.

Cool :-)

Thanks and Regards
Stefan