Bug 64919 – ebuild for the museek soulseek client

• Status: RESOLVED WONTFIX
• Product: Gentoo Linux

[Description daelstorm 2004-09-21 17:27:00 UTC]

A split C++ daemon client and QT GUI client for the SoulSeek network

[Comment 1 daelstorm 2004-09-21 17:28:52 UTC]

Created attachment 40135 [details]
museek ebuild

[Comment 2 daelstorm 2004-09-21 17:30:47 UTC]

Created attachment 40136 [details, diff]
patch removes ogg support to museek build code if USE flags dictate it so

[Comment 3 daelstorm 2004-09-21 17:32:45 UTC]

Created attachment 40137 [details, diff]
patch adds online notification to museeq

[Comment 4 Andrej Kacian (RETIRED) 2004-09-27 08:48:49 UTC]

The ebuild doesn't have correct header - please have a look at common ebuild mistakes in developer documentation on http://www.gentoo.org

[Comment 5 daelstorm 2004-10-14 13:36:36 UTC]

Comment on attachment 40135 [details]
museek ebuild

# Copyright 1999-2004 Gentoo Foundation
# Distributed under the terms of the GNU General Public License v2
# $Header: $
>
>Name="museek"
>DESCRIPTION="A split C++ daemon client and QT GUI client for the Soulseek network"
>HOMEPAGE="<A href="http://museek.thegraveyard.org/" class="linkification-ext">http://museek.thegraveyard.org/</A>"
>SRC_URI="<A href="http://museek.thegraveyard.org/" class="linkification-ext">http://museek.thegraveyard.org/</A>${Name}-${PV}.tar.bz2"
>LICENSE="GPL-2"
>SLOT="0"
>KEYWORDS="x86"
>IUSE="qsa qt oggvorbis" #fam
>DEPEND="dev-cpp/libxmlpp
>	qt? >=x11-libs/qt-3.2
>	dev-lang/swig
>	dev-lang/python
>	oggvorbis? ( media-libs/libogg media-libs/libvorbis )
>	qsa? ( >=dev-libs/qsa-1.1 )
>	dev-util/scons"
>#	fam? ( app-admin/fam )
>
>RDEPEND="dev-cpp/libxmlpp
>	qt? >=x11-libs/qt-3.2
>	dev-lang/python
>	oggvorbis? ( media-libs/libogg media-libs/libvorbis )
>	qsa? ( >=dev-libs/qsa-1.1 )"	
>
>src_compile() {
>	local myconf=""
>	if use qt; then 
>	    epatch ${FILESDIR}/onlinealert-${Name}-${PV}-${PR}.patch
>	else 
>	    myconf="${myconf} MUSEEQ=0"
>	fi
>	use oggvorbis || epatch ${FILESDIR}/noogg-${Name}-${PV}-${PR}.patch
>	use qsa	|| myconf="${myconf} QSA=0"
>
>	scons ${myconf} PREFIX=/usr DESTDIR=${D}
>}
>src_install() {
>	scons DESTDIR=${D} install
>}

[Comment 6 daelstorm 2004-10-18 03:23:41 UTC]

Comment on attachment 40135 [details]
museek ebuild

># Copyright 1999-2004 Gentoo Foundation
># Distributed under the terms of the GNU General Public License v2
># $Header:
>
>Name="museek"
>DESCRIPTION="A split C++ daemon client and QT GUI client for the Soulseek network"
>HOMEPAGE="<A href="http://museek.thegraveyard.org/" class="linkification-ext">http://museek.thegraveyard.org/</A>"
>SRC_URI="<A href="http://museek.thegraveyard.org/$" class="linkification-ext">http://museek.thegraveyard.org/$</A>{Name}-${PV}.tar.bz2"
>LICENSE="GPL-2"
>SLOT="0"
>KEYWORDS="x86"
>IUSE="qsa qt oggvorbis" #fam
>DEPEND="dev-cpp/libxmlpp
>	qt? >=x11-libs/qt-3.2
>	dev-lang/swig
>	dev-lang/python
>	oggvorbis? ( media-libs/libogg media-libs/libvorbis )
>	qsa? ( >=dev-libs/qsa-1.1 )
>	dev-util/scons"
>#	fam? ( app-admin/fam )
>
>RDEPEND="dev-cpp/libxmlpp
>	qt? >=x11-libs/qt-3.2
>	dev-lang/python
>	oggvorbis? ( media-libs/libogg media-libs/libvorbis )
>	qsa? ( >=dev-libs/qsa-1.1 )"	
>
>src_compile() {
>	local myconf=""
>	if use qt; then 
>	    epatch ${FILESDIR}/onlinealert-${Name}-${PV}-${PR}.patch
>	else 
>	    myconf="${myconf} MUSEEQ=0"
>	fi
>	use oggvorbis || epatch ${FILESDIR}/noogg-${Name}-${PV}-${PR}.patch
>	use qsa	|| myconf="${myconf} QSA=0"
>
>	scons ${myconf} PREFIX=/usr DESTDIR=${D}
>}
>src_install() {
>	scons DESTDIR=${D} install
>}

[Comment 7 Stefan Vunckx 2005-03-07 14:12:20 UTC]

Shouldn't the ebuild inherit something like distutils ??
I get a warning that "epatch" command cannot be found ... Added inherit distutils at the beginning of the ebuild ...

[Comment 8 Andrej Kacian (RETIRED) 2005-03-07 15:59:58 UTC]

Wasn't epatch part of eutils.eclass ?

[Comment 9 Stefan Vunckx 2005-03-08 02:50:57 UTC]

Yup, my mistake

[Comment 10 trejkaz 2005-03-12 20:57:23 UTC]

Another thing, there should be a "|| die" after scons in case it fails, like it did for me.

[Comment 11 Carsten Lohrke (RETIRED) 2005-10-24 16:39:05 UTC]

- the ebuild header is invalid

[Comment 12 Carsten Lohrke (RETIRED) 2005-10-24 16:39:05 UTC]

- the ebuild header is invalid¹
- remove Name="museek"
- replace ${Name}-${PV}  with ${P}
- ${Name}-${PV}-${PR} == ${PF}, but usually it suffices when the patch has $PV
in it's filename
- keyword should be ~x86


[1] http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml?part=2&chap=3

[Comment 13 Jon Hood (RETIRED) 2006-05-18 20:34:45 UTC]

daelstorm, could you please at least update the header information and as many things Carlo pointed out as you can? Post your updated ebuild and we'll run through it. I'd be happy to add it to portage as soon as we get it cleaned up a bit.

[Comment 14 Jon Hood (RETIRED) 2006-06-13 19:45:06 UTC]

Please reopen if you can at least update the header information.