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

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

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.

> 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

> 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

Plus it fixes a partially-unimplemented feature.

BTW how did my patch break mini?

Reply to: