From: Otavio Salvador <otavio@debian.org>
To: openembedded-devel@lists.openembedded.org
Cc: openembedded-devel@openembedded.org
Subject: Re: [PATCH 3/6] package.bbclass, package_ipk.bbclass: add dynamic depends on locale packages (ipk specific)
Date: Thu, 12 Feb 2009 22:04:23 -0200 [thread overview]
Message-ID: <871vu3mcc8.fsf@neumann.lab.ossystems.com.br> (raw)
In-Reply-To: <1234478544.4890.404.camel@lenovo.internal.reciva.com> (Phil Blundell's message of "Thu, 12 Feb 2009 22:42:24 +0000")
Phil Blundell <pb@reciva.com> writes:
> On Thu, 2009-02-12 at 12:38 -0600, Mike (mwester) wrote:
>> Phil Blundell wrote:
>> > This looks like it will add locale dependencies to all the subpackages
>> > (e.g. foo-doc, foo-dev and the like) as well as the main "foo" package,
>> > which is a bit different to how the old code used to work. But I guess
>> > that's harmless enough: the worst effect will be to bloat the Packages
>> > file a bit, and I doubt anybody will care too much. So, thumbs up from
>> > me.
>>
>> How much bloat?
>
> For a source package "foo", you'd be talking about adding one copy of
> the literal string "foo-locale-*" to the Depends line for each of its
> subpackages (foo-doc, foo-dev, foo-dbg and the like). The exact amount
> obviously depends on the length of the package name, but I guess you
> could call it about 16 bytes per binary package as a rough
> approximation.
>
> Exactly what the real-world impact of that would be will depend on your
> distro policy. If you already strip out the -dev and -doc packages from
> the main Packages feed then obviously the impact will be virtually nil
> because most of the affected packages won't be there in the first place.
> At the other extreme, if your Packages files currently contain
> everything from the control data (including Source:, Description: and so
> on) then they are probably so huge already that another sixteen bytes
> per package is negligible in percentage terms.
>
> If those 16*n bytes did turn out to cause a problem for someone then it
> probably wouldn't be that hard to reinstate the original behaviour of
> only adding this dependency on the primary binary.
Now I have two good points and I'm uncertain where to go :-)
Push it and sort the regressions out depending how hard is it or make it
optional?
--
O T A V I O S A L V A D O R
---------------------------------------------
E-mail: otavio@debian.org UIN: 5906116
GNU/Linux User: 239058 GPG ID: 49A5F855
Home Page: http://otavio.ossystems.com.br
---------------------------------------------
"Microsoft sells you Windows ... Linux gives
you the whole house."
next prev parent reply other threads:[~2009-02-13 0:05 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-12 16:38 [RFC] Patches being used by O.S. Systems and ready to be pushed Otavio Salvador
2009-02-12 16:38 ` [PATCH 1/6] rt61-firmware: add 1.2 Otavio Salvador
2009-02-12 16:38 ` [PATCH 2/6] cups: add 1.3.8 Otavio Salvador
2009-02-12 16:38 ` [PATCH 3/6] package.bbclass, package_ipk.bbclass: add dynamic depends on locale packages (ipk specific) Otavio Salvador
2009-02-12 16:38 ` [PATCH 4/6] parted: drop versions up to 1.8.8 Otavio Salvador
2009-02-12 16:38 ` [PATCH 5/6] sun-jre6: add 1.6.0.10 Otavio Salvador
2009-02-12 16:38 ` [PATCH 6/6] flash-plugin: add 10.0.12.36 Otavio Salvador
2009-02-12 17:29 ` Koen Kooi
2009-02-12 23:49 ` Otavio Salvador
2009-02-13 0:35 ` Koen Kooi
2009-02-12 20:27 ` Phil Blundell
2009-02-12 21:36 ` Koen Kooi
2009-02-12 22:47 ` Phil Blundell
2009-02-12 17:29 ` [PATCH 5/6] sun-jre6: add 1.6.0.10 Koen Kooi
2009-02-12 18:39 ` Ken Gilmer
2009-02-12 20:31 ` Koen Kooi
2009-02-12 23:48 ` Otavio Salvador
2009-02-13 0:36 ` Koen Kooi
2009-02-12 17:26 ` [PATCH 4/6] parted: drop versions up to 1.8.8 Koen Kooi
2009-02-12 23:51 ` Otavio Salvador
2009-02-13 0:31 ` Koen Kooi
2009-02-13 0:51 ` Philip Balister
2009-02-12 17:25 ` [PATCH 3/6] package.bbclass, package_ipk.bbclass: add dynamic depends on locale packages (ipk specific) Koen Kooi
2009-02-12 23:52 ` Otavio Salvador
2009-02-12 17:47 ` Phil Blundell
2009-02-12 18:38 ` Mike (mwester)
2009-02-12 22:36 ` Koen Kooi
2009-02-13 3:29 ` Tom Rini
2009-02-13 9:03 ` Koen Kooi
2009-02-12 22:42 ` Phil Blundell
2009-02-13 0:04 ` Otavio Salvador [this message]
2009-02-13 0:02 ` Otavio Salvador
2009-02-13 0:35 ` Koen Kooi
2009-02-12 17:24 ` [PATCH 2/6] cups: add 1.3.8 Koen Kooi
2009-02-12 17:24 ` [PATCH 1/6] rt61-firmware: add 1.2 Koen Kooi
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=871vu3mcc8.fsf@neumann.lab.ossystems.com.br \
--to=otavio@debian.org \
--cc=openembedded-devel@lists.openembedded.org \
--cc=openembedded-devel@openembedded.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