From: "Santiago M. Mola" <coldwind@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] [RFC] Ability to pass arguments to src_configure/src_compile
Date: Sat, 6 Sep 2008 22:45:56 +0200 [thread overview]
Message-ID: <3c32af40809061345u5d8e2a44kcf01367525f2db28@mail.gmail.com> (raw)
In-Reply-To: <48C2E3C5.5040701@gentoo.org>
On Sat, Sep 6, 2008 at 10:10 PM, Ben de Groot <yngwin@gentoo.org> wrote:
> Alec Warner wrote:
>> On Sat, Sep 6, 2008 at 10:36 AM, Thomas Anderson <gentoofan23@gentoo.org> wrote:
>>> Hi,
>>> Currently we have a lot of:
>>> src_configure() {
>>> econf $(use_enable dvdr) \
>>> $(use_with ipv6 ssl) \
>>> --with-system-zlib
>>> }
>>>
>>> Introducing(Idea shamelessly taken from Exherbo):
>>> DEFAULT_SRC_CONFIGURE_USE_{WITHS,ENABLES}
>>> DEFAULT_SRc_CONFIGURE_EXTRA_PARAMS
>>>
>>> The code from above could be rewritten like so:
>>>
>>> DEFAULT_SRC_CONFIGURE_USE_ENABLES=( 'dvdr' )
>>> DEFAULT_SRC_CONFIGURE_USE_WITHS=( 'ipv6 ssl' )
>>> DEFAULT_SRC_CONFIGURE_EXTRA_PARAMS=( '--with-system-zlib' )
>>>
>>> That's much simpler.
>>
>> It saves you 1 line and reduces readability and intuitiveness by a
>> fair margin; how is it simpler?
>
> It may be 2 lines less, but it is 42 characters more.
> Plus, I dislike caps. :-p
In the example I posted it's 339 characters less. Almost half of the
original ;-)
Regards,
--
Santiago M. Mola
Jabber ID: cooldwind@gmail.com
next prev parent reply other threads:[~2008-09-06 20:45 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-06 17:36 [gentoo-dev] [RFC] Ability to pass arguments to src_configure/src_compile Thomas Anderson
2008-09-06 19:00 ` Alec Warner
2008-09-06 19:01 ` Alec Warner
2008-09-06 20:10 ` Santiago M. Mola
2008-09-06 20:10 ` Ben de Groot
2008-09-06 20:45 ` Santiago M. Mola [this message]
2008-09-08 21:57 ` [gentoo-dev] " Steve Long
[not found] <b99LY-5lh-39@gated-at.bofh.it>
[not found] ` <b9bkA-7sM-7@gated-at.bofh.it>
[not found] ` <b9cqk-nW-9@gated-at.bofh.it>
2008-09-07 15:31 ` [gentoo-dev] " Vaeth
2008-09-07 15:40 ` Ciaran McCreesh
2008-09-07 16:46 ` Marcus D. Hanwell
2008-09-07 16:50 ` Ciaran McCreesh
2008-09-07 17:11 ` Santiago M. Mola
2008-09-08 19:05 ` Luca Barbato
2008-09-08 21:33 ` Donnie Berkholz
2008-09-08 22:13 ` Ciaran McCreesh
2008-09-08 23:03 ` Donnie Berkholz
2008-09-08 23:31 ` Alec Warner
2008-09-12 17:36 ` Arun Raghavan
[not found] <b9uwZ-6YY-23@gated-at.bofh.it>
[not found] ` <b9uwZ-6YY-25@gated-at.bofh.it>
[not found] ` <b9ux0-6YY-27@gated-at.bofh.it>
[not found] ` <b9uwZ-6YY-21@gated-at.bofh.it>
[not found] ` <b9uGz-79d-7@gated-at.bofh.it>
2008-09-08 7:48 ` Vaeth
2008-09-08 8:02 ` David Leverton
2008-09-08 8:40 ` Santiago M. Mola
[not found] ` <b9vCE-8mw-21@gated-at.bofh.it>
[not found] ` <b9vMy-8ws-33@gated-at.bofh.it>
[not found] ` <b9w5F-FY-5@gated-at.bofh.it>
2008-09-08 8:07 ` Vaeth
2008-09-08 8:44 ` Alec Warner
2008-09-08 8:58 ` Santiago M. Mola
2008-09-08 9:31 ` Alec Warner
2008-09-08 9:40 ` Mike Auty
[not found] <b9JFB-1eE-13@gated-at.bofh.it>
[not found] ` <b9JFB-1eE-15@gated-at.bofh.it>
[not found] ` <b9JFB-1eE-17@gated-at.bofh.it>
[not found] ` <b9JFC-1eE-19@gated-at.bofh.it>
[not found] ` <b9JFC-1eE-21@gated-at.bofh.it>
[not found] ` <b9JFB-1eE-11@gated-at.bofh.it>
[not found] ` <b9KBD-2uE-1@gated-at.bofh.it>
2008-09-08 11:56 ` Vaeth
2008-09-08 12:10 ` Santiago M. Mola
[not found] ` <b9JYY-1P6-19@gated-at.bofh.it>
[not found] ` <b9JYY-1P6-21@gated-at.bofh.it>
[not found] ` <b9JYY-1P6-23@gated-at.bofh.it>
[not found] ` <b9JYX-1P6-17@gated-at.bofh.it>
[not found] ` <b9KBK-2uE-31@gated-at.bofh.it>
2008-09-08 12:46 ` Vaeth
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3c32af40809061345u5d8e2a44kcf01367525f2db28@mail.gmail.com \
--to=coldwind@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox