Re: [cdebconf] helper macros, i18n, backup, etc

[Randolph Chung <tausq@debian.org>]

sorry for the late reply, i was out of town for a few days.

> 1. The helper macros recently introduced do break several packages
> under debian-installer/tools/ which used to declare their own
> debconf_input function.  Maybe we could remove these macros, I
> wonder whether they are that useful.

i thought you asked for them? :-)

unless i hear otherwise i'll commit a patch tomorrow to wrap them inside
a #ifdef WITH_DEBCONF_HELPER_MACROS or something like that.

> 2. The attached backup.patch finishes support for backing up.  As it
> slightly changes cdebconf interface, I prefer sending it there.

looks ok in general, but perhaps you can add an enum for the -1, 0, 1
stuff instead of using a number? thanks! this is a long-needed bugfix.

> 3. May I set a _ macro in frontend.h in order to start i18n of
> frontends?  As libdiscover already calls dcgettext, I would like to
> set it to
>    #define _(x) dcgettext("cdebconf", (x), LC_MESSAGES)

sounds fine to me

> 4. As esplained in #172218, current TITLE command is not l10n-friendly.
> I will wait for Joey's solution before changing this in cdebconf, but
> the new progress bar support suffers from the same problem.

hrm, i see.... making it a template seems like a reasonable thing to
do...

btw, someone (waldi, I think) asked for debconf-communicate support in
cdebconf. i hacked up something while i was on the plane a few days ago
and it's now in cvs. seems to work, but yell if it breaks stuff. right
now it's built for both deb and udeb. If this becomes a size problem
we can always remove it from the udeb.

btw2, we should really not have this piece of code in debconf.c:
    q = questions->methods.get(questions, "debian-installer/language");
can this be just debconf/language instead?

putting cdebconf in debian-installer cvs was more of a convenience.
cdebconf is not meant to be a debian-installer-specific application.

btw3, istr some talk about implementing a http based db backend. is that
available some place? sorry if i missed this on the list, i haven't been
following d-boot too closely.

thanks,
randolph