Bug 539476 – zshdb-0.9 - Debugger for zsh

• Status: UNCONFIRMED
• Product: Gentoo Linux

[Description Rocky Bernstein 2015-02-09 15:06:27 UTC]

Created attachment 395982 [details]
zshdb-0.09.ebuild

Please find attached an ebuild for the Zsh debugger zshdb.

Thanks

[Comment 1 Jeroen Roovers (RETIRED) 2015-02-10 07:16:04 UTC]

Comment on attachment 395982 [details]
zshdb-0.09.ebuild

>EAPI=4

What is wrong with 5?

>inherit autotools 
>
>DESCRIPTION="Debugger for zsh"
>HOMEPAGE="http://rocky.github.io/zshdb/"
>SRC_URI="mirror://sourceforge/bashdb/${P}.tar.bz2"

>LICENSE="GPL3"

Invalid license. Try GPL-3.

>SLOT="0"
>KEYWORDS=""
>IUSE=""

No need to set an empty IUSE.

>RDEPEND="app-shells/zsh"
>DEPEND="${RDEPEND}"
>
>src_prepare() {
>	eautoreconf
>}
>
>src_install() {
>	emake DESTDIR="${D}" install || die
>	dodoc AUTHORS ChangeLog NEWS README THANKS || die

These don't need to die(). In fact it looks suspiciously like the default src_install() _except_ for "|| die".

[Comment 2 Rocky Bernstein 2015-02-10 10:40:29 UTC]

> What is wrong with 5?

After submitting, I noticed and tried with 5. EAPI=5 works fine.

I am new to this. 

One other thing I realized just now (and don't know how to do).
T
his package should recommend installing pygmentize, since if that's around it use that to colorize source-code listings. 

Thanks for the comments and help.

[Comment 3 Rocky Bernstein 2015-02-10 10:41:55 UTC]

> What is wrong with 5?

After submitting, I noticed and tried with 5. EAPI=5 works fine.

I am new to this. 

One other thing I realized just now (and don't know how to do).
This package should recommend installing pygmentize, since if that's around it will use that to colorize source-code listings. 

Thanks for the comments and help.

[Comment 4 Rocky Bernstein 2015-02-10 11:00:07 UTC]

Created attachment 396070 [details]
zsh 0.9 ebuild revised

Simpler and with Jeroen Roover's suggestions.

[Comment 5 Rocky Bernstein 2015-02-10 11:05:03 UTC]

Created attachment 396072 [details]
zsh 0.9 ebuild revised

Fill keywords ~amd64 and ~x86