Openembedded Devel Discussions
 help / color / mirror / Atom feed
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."



  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