From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] gpsd: Add config options for USB, Bluetooth, Qt.
Date: Thu, 23 May 2013 21:12:23 +0200 [thread overview]
Message-ID: <20130523211223.59edf1b8@skate> (raw)
In-Reply-To: <519E67FA.8060608@mind.be>
Dear Arnout Vandecappelle,
On Thu, 23 May 2013 21:03:22 +0200, Arnout Vandecappelle wrote:
> If enabling the option adds a significant amount of overhead (typically
> rootfs size overhead but it could also be performance overhead), then it
> is definitely worthwhile to make it an option. So for the qt bindings
> it's probably not worthwhile (qt itself is much larger). But for the
> Bluetooth or USB support, it could make a difference.
Regarding the Qt binding, if it installs additional files for example,
I wouldn't mind having a separate sub-option to disable it.
> > Maybe (2) is what makes sense here. The cost of new options in terms of
> > maintenance is not so high,
>
> You forgot about updating the 'depends on' statements when a suboption
> wants to select e.g. glib...
Hum, not sure to understand what you meant here. Did you mean
"propagate the 'depends on'" from the selected package into the
selecting package, for dependencies on toolchain features and things
like that?
> > so maybe that's what we should do. I don't know.
>
> My dream is that the whole Config.in could be generated from the .mk
> file... but I'm not sure if that is really realistic. OpenWRT does it but
> it doesn't look that great either.
>
> So I think we'll stick to
>
> 3) Decide on a case-by-case basis whether to go for suboptions or
> automatic enable/disable.
I agree, as you don't see much other reasonable solutions.
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2013-05-23 19:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-23 14:43 [Buildroot] [PATCH 1/1] gpsd: Add config options for USB, Bluetooth, Qt Arn R
2013-05-23 14:56 ` Thomas Petazzoni
2013-05-23 15:30 ` Arn R
2013-05-23 15:46 ` Thomas Petazzoni
2013-05-23 19:03 ` Arnout Vandecappelle
2013-05-23 19:12 ` Thomas Petazzoni [this message]
2013-05-24 5:16 ` Arnout Vandecappelle
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=20130523211223.59edf1b8@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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