[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#448034: live-helper: support LH_BOOTSTRAP_FLAVOUR=minimal



Justin Pryzby <jpryzby+d at quoininc.com> writes:

> On Mon, Oct 29, 2007 at 08:35:22PM -0200, Otavio Salvador wrote:
>> Justin Pryzby <jpryzby+d at quoininc.com> writes:
>> 
>> > Package: live-helper
>> > Version: 1.0~a31-1
>> > Severity: normal
>> > Tags: patch
>> > File: /usr/bin/lh_bootstrap_cdebootstrap
>> >
>> > Maarten mentioned this here:
>> > http://lists.alioth.debian.org/pipermail/debian-live-devel/2007-October/002492.html
>> >
>> > It seems like it was meant to be supported but only used a heureustic
>> > or partial implementation until now.
>> >
>> > I wrote this patch while trying to work around sid's unsatisfied
>> > depends from aptitude, but didn't manage to create an image yet.  This
>> > patch might require with --apt apt.
>> 
>> aptitude breakage is due new ABI that has been introduced on lastest
>> APT release (0.7.8) and ought be already solved on today's sid pulse.
> Yeah.  I don't understand the source of the problem though or what
> caused it to become fixed.

The problem was a application binary interface that has change due my
last APT upload. This broke all package that was using libapt-pkg.so
and those needed to be rebuild against the lastest source to get the
new symbols and different binary interface used.

>> Personaly, I don't understand the reason for your patch since it
>> doesn't do anything special besides beak the mini flavour ;-)
> There's an exposed option
> $ lh_config --help |grep mini
>             [-f|--bootstrap-flavour minimal|standard]
>
> but it has no functional effect, as Marteen pointed.  If the option
> isn't intended to be further supported (and is perhaps only retained
> to avoid breaking builds for people who already use it) then it should
> eventually be removed.  I guess -p mini would be the supported
> interface.

mini is different the minimal. If you take a look on the hooks dir,
there're two files (one for each) with different content. mini does a
lot of hacks to reduce the final image size while minimal tries to
stay a "debian" without much breackage (just removing packages and
like).

>> Can you elaborate why you needed the patch?
> I was hoping the only reason aptitude was being installed was since it
> is priority:important, so I hoped that this would avoid [attempting
> to] install it.  As it turns out it didn't work, apparently due to
> #448210.
>
> Plus it fixes a partially-unimplemented feature.
>
> BTW how did my patch break mini?

You've removed the support for it to use minimal cdebootstrap
flavour. :-)

-- 
        O T A V I O    S A L V A D O R
---------------------------------------------
 E-mail: otavio at 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 to: