Bugzilla Bug 46153

Hi there

I really like cadubi and installed it on my other (non-Gentoo) systems to create some fancy ASCII-Art stuff. Now I want this piece of software in my favourite distribution ;)
The development of cadubi has been stopped, so this is the last version of it.

For more information about cadubi go to: http://langworth.com/CadubiProject

You'll find my ebuild attached to this bug report. I've tested it on my local Gentoo box and it seems to work. I'd suggest to put it into app-misc (like figlet, a more or less similar program)

I'm looking forward to see my ebuild in the official Gentoo distribution.

Greetings from Switzerland

Tobias

Created an attachment (id=28305) [details]
cadubi-1.3.ebuild (new package)

Hum, looks nice ;) I'll take it.

Thanks for introducing such a cute package. You ebuild looks fine but
the cadubi perl script needs "help.txt" to be in the same place as it
is installed (/usr/bin) in order to read the help.  However,
we don't want simple text file to be installed into /usr/bin, I
suggest to install it to /usr/lib/cadubi/help.txt and change cadubi
script accordingly (you may want to read the script and make a patch
to do that).

P.S.
"cd ${WORKDIR}/${P}" in src_install() isn't necessary as the initial
working directory of src_install() is automatically set to S 
(= ${WORKDIR}/${P}). See man 5 ebuild for more detail.

Created an attachment (id=28308) [details]
Patch to change the path to the help.txt file

I hope it's OK like that.

Created an attachment (id=28309) [details]
cadubi-1.3.ebuild (new package) changed as proposed

I changed the ebuild as proposed. I hope it's OK like that.

You forgot to actually apply the patch ;) We usually do that
in src_unpack(). It may look like this:

src_unpack() {
	unpack ${A}
	cd ${S}
	epatch ${FILESDIR}/${P}-helpfile.patch
}

(where ${P} is expanded to "cadubi-1.3") Also, dolib installs help.txt
under /usr/lib (so it ends with /usr/lib/help.txt). You need to use
insinto/doins functions.

src_install() {
	...
	insinto /usr/lib/${PN}
	doins help.txt
	...
}

(again, ${PN} is expanded to "cadubi")

Hope this will help you writing the ebuild.

Created an attachment (id=28310) [details]
cadubi-1.3.ebuild (new package) changed as proposed #2

It's late here in Switzerland ;) I hope the ebuild works now.

Thanks for your help usata

Created an attachment (id=28313) [details]
cadubi-1.3.ebuild (new package) I'm just too stupid...

Two times src_install() ;)

Cool. I've just committed it to CVS. If you find any other package
useful/interesting/funny, feel free to create an ebuild and submit
it to bugzilla ;) Thanks for the contribution.

Do I need to mark this ebuild stable by myself and add it to bugzilla or does
someone of the nice Gentoo guys do this?

Marked as stable. Thanks for the report. 

We usually wait for a month after introducing the ebuild to CVS.
If you find an ebuild which isn't marked stable for more than
several months (and you think the ebuild stable), please open 
a new bug for each ebuild that says "app-xxx/yyy-z.z.z.ebuild 
stable on x86" or something like that. diff to the modified ebuild
is preferred (If you just change KEYWORD, no attachment is needed).