Hi. I have made just another ebuild to my favourite distro. gwenview is an image viewer for KDE. I have used the kde-base eclass (thx danamark!). It should be in media-gfx/gwenview. I should attach the Changelog, but for experience no one of my 40 previous Changelog has been respected, so this time I won't do it. It works for me, although ------------------------------------------------------------------------------- Status for gwenview-0.14.0.ebuild * Testing for illegal space characters, weird backslash formatting : passed * Testing for malformed headers : passed - (W) Has illegal or suspect headers: |Suspect copyright year: # Copyright 1999-2001 Gentoo Technologies, Inc. * Testing for occurence of deprecated try : passed * Testing for superfluous A=${P}.tar.gz : passed * Testing for empty DEPEND : passed * Testing for empty HOMEPAGE : passed * Testing for empty DESCRIPTION : passed * Testing for presence of env vars : passed - (W) Missing SLOT= - (W) Missing S= * Testing for sane USE flag usage : passed ----- Danamark said that missing SLOT and S was OK... "- S=${WORKDIR}/${P} not necessary (eclasses do that). - LICENSE setting removed because I'm going to do a master license sweep/update of kde apps this weekend (long story...). - SLOT=0 not necessary because eclasses automatiaclly set SLOT=3 for kde3 apps. " --- Well, slt, and Happy Gentooing ;)
Created attachment 1564 [details] gwenview-0.14.0.ebuild
I'm adding this to portage. A note about the ebuild: when you write DEPEND="something", you in effect remove everything that the kde eclasses put there (kdelibs, gcc and some other stuff). You should always extend these variables, i.e. write DEPEND="$DEPEND something". Also, if you want to add something to both DEPEND and RDEPEND in one go, you can call newdepend() (i.e. newdepend "something"). That's what I've done with the committed ebuild. Otherwise it's ok, thanks for submitting.
