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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.