[gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread Tomáš Chvátal
Hi, we prepared new eclass for x11 packages that should be used as replacement for x-modular.eclass. After long discussion with others on irc we choose to name it as xorg-2.eclass. Whats new/changed: 1 removed function call (pkg_preinst iirc) and 1 new added pkg_setup. Requires eapi3 or any

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread Jeremy Olexa
On Thu, 18 Feb 2010 23:33:42 +0100, Tomáš Chvátal scarab...@gentoo.org wrote: Requires eapi3 or any later when added, so wont work with eapi 012. Hey, can you explain why you choose to do this and what benefit there is to doing so? Thanks. -Jeremy

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread Tomáš Chvátal
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Dne 18.2.2010 23:39, Jeremy Olexa napsal(a): On Thu, 18 Feb 2010 23:33:42 +0100, Tomáš Chvátal scarab...@gentoo.org wrote: Requires eapi3 or any later when added, so wont work with eapi 012. Hey, can you explain why you choose to do this and

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread David Leverton
On Thursday 18 February 2010 22:33:42 Tomáš Chvátal wrote: [[ ${PN} == util-macros ]] || DEPEND+= =x11-misc/util-macros-1.3.0 [[ ${PN} == font-util ]] || DEPEND+= =media-fonts/font-util-1.1.1-r1 Do non-fonts really need font-util there? Looks like that sets up a nice circular

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread Tomáš Chvátal
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Dne 19.2.2010 00:11, David Leverton napsal(a): On Thursday 18 February 2010 22:33:42 Tomáš Chvátal wrote: [[ ${PN} == util-macros ]] || DEPEND+= =x11-misc/util-macros-1.3.0 [[ ${PN} == font-util ]] || DEPEND+=

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread Tomáš Chvátal
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 [[ -e ./configure.ac ]] eautoreconf || ewarn Unable to autoreconf the configure script. Things may fail. That'll ewarn if configure.ac doesn't exist at all. Doesn't eautoreconf die anyway if it fails, and if not, is it

Re: [gentoo-dev] [RFC] New eclass for x11 packages

2010-02-18 Thread David Leverton
On Thursday 18 February 2010 23:16:54 Tomáš Chvátal wrote: That || die is not for eautoreconf [[ -e something ]] somethingexists || somethingisnotexisting For your behaviour it would have to look like this [[ -e something ]] { somethingexists || die if the commands failed ; } Do you mean

[gentoo-dev] Re: [RFC] New eclass for x11 packages

2010-02-18 Thread Ryan Hill
On Thu, 18 Feb 2010 23:33:42 +0100 Tomáš Chvátal scarab...@gentoo.org wrote: Hi, we prepared new eclass for x11 packages that should be used as replacement for x-modular.eclass. After long discussion with others on irc we choose to name it as xorg-2.eclass. Whats new/changed: Fonts