[csw-pkgsubmissions] newpkgs libhistory4, libhistory5, libhistory6(...)

[Dagobert Michelsen]

Hi Phil,

Am 11.03.2011 um 00:04 schrieb Philip Brown:
> On Thu, Mar 10, 2011 at 6:38 AM, Dagobert Michelsen <dam at opencsw.org> wrote:
>> Version bump and split
> 
> I have some comments and questions about this split.
> 
> 1. Is it really beneficial to split out separate "libhistory" and
> "libreadline" here?
> They will pretty much ALWAYS be upgraded together.

Right.

> Splitting it makes it almost more likely to have a botched upgrade sometime
> 
> 2. does it really make sense to have this package named this way, with
> your current split?

There are however lots of uses of libreadline.so but only very few
that use libhistory.so

>>  + libreadline_dev-6.2,REV=2011.03.10-SunOS5.9-sparc-CSW.pkg.gz
>>  + libreadline_dev-6.2,REV=2011.03.10-SunOS5.9-i386-CSW.pkg.gz
> 
> Given the naming of the other "separate" libhistory and libreadline,
> that almost implies it is only for "libreadline".. which may make
> people looking at "libhistory" wonder where its dev files are.
> If you're being strictly consistent, seems like you should Either make
> a more generic
> "readline_dev"
> 
> Or, make *both*
> libreadline_dev
> libhistory_dev

This is not good. Usually we have just one -dev package regardless of the
number of libraries involved where the name of the -dev package is
derived from the upstream name and usually is lent from one of the libraries.
This is also the case here. As CSWlibreadline-dev is an R-dep of libhistory
it should be fairly easy to find out :-)

> Contrariwise:
> It may be simpler to just silently bundle in "libhistory.so.6" in with
> the "libreadline6"  package ?
> This also matches up with the naming conventions, since the "dev"
> files for libhistory, are in
> /opt/csw/include/readline/history.h
> 
> This implies they are a logical set and should not be split from each other.
> 
> thoughts?

Splitting allows retiriing libraries one at a time. And indeed: libhistory.so.4
is not needed anywhere, but libreadline.so.4 is, so you could skip CSWlibhistory4
for release.


Best regards

  -- Dago