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

Re: r56755 - in trunk/packages: auto-install/debian kbd-chooser/debian localechooser/debian

On Saturday 22 November 2008, Philip Hands wrote:
> Author: philh
> Date: Sat Nov 22 10:34:05 2008
> New Revision: 56755
> Log:
> in auto-installs, only delay kbd/localechooser when there is a preseed
> available
> Modified:
>    trunk/packages/auto-install/debian/auto-install.postinst
>    trunk/packages/auto-install/debian/changelog
>    trunk/packages/kbd-chooser/debian/changelog
>    trunk/packages/kbd-chooser/debian/kbd-chooser.isinstallable
>    trunk/packages/localechooser/debian/changelog
>    trunk/packages/localechooser/debian/localechooser.isinstallable

[21:03:36] <fjp> fil: Aren't your tests inverted in your last commit in 
the isinstallable scripts?
[21:05:46] <fjp> fil: I also don't like the file name in /var/run/ very 
much because it gets set repeatedly (each time you get back to main menu) 
and when it does get set the "had-to-ask" is incorrect as auto-install 
has not yet run.
[21:06:02] <fjp> Very messy and IMO maybe not Lenny material.
[21:06:26] <fjp> Also:
[21:06:27] <fjp> -localechooser (2.09) UNRELEASED; urgency=low
[21:06:28] <fjp> +localechooser (2.09+nmu1) UNRELEASED; urgency=low
[21:06:42] <fjp> Please get rid of the +nmu1.
[21:08:08] <fjp> (Blame devscript devs. and see doc/devel/package-upload 
(IIRC) for suggested settings.)

Reply to: