Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-17 Thread Christian Perrier
Quoting Frans Pop ([EMAIL PROTECTED]): > Maybe even define a new class "debconf directives" that can be included > in strings. > This could be something like ${!TAB} or ${!COL} and ${!SPAN=2}. Advantage > is that ${...} is already used for variables and thus already special cased. > The added "!"

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-17 Thread Frans Pop
Otavio wrote: > I prefer to: > - use a easy to identify character as separator (& | or whatever) > - a way to escape this character (\& \| or whatever) I'd suggest to do it the other way around: define a special character sequence that is recognized by cdebconf as a column separator (and a second

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-17 Thread Otavio Salvador
Joey Hess <[EMAIL PROTECTED]> writes: > Otavio Salvador wrote: >> This isn't accurate enough, I'm sorry. I think that any char we choose >> as a separator needs to have a way to be escaped. > > A tab *means* "advance to the next column". Therefore, it would do > exactly what it's intended to do, t

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-16 Thread Joey Hess
Otavio Salvador wrote: > This isn't accurate enough, I'm sorry. I think that any char we choose > as a separator needs to have a way to be escaped. A tab *means* "advance to the next column". Therefore, it would do exactly what it's intended to do, therefore there is no point in escaping it. --

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-16 Thread Otavio Salvador
Christian Perrier <[EMAIL PROTECTED]> writes: > Quoting Otavio Salvador ([EMAIL PROTECTED]): >> Joey Hess <[EMAIL PROTECTED]> writes: >> >> > Using almost any other character than a tab does have the issue that >> > said separator character would need a way to be escaped and displayed. >> >> And

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-16 Thread Christian Perrier
Quoting Otavio Salvador ([EMAIL PROTECTED]): > Joey Hess <[EMAIL PROTECTED]> writes: > > > Using almost any other character than a tab does have the issue that > > said separator character would need a way to be escaped and displayed. > > And why does tab don't need a way to be escaped? I'd say

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-16 Thread Otavio Salvador
Joey Hess <[EMAIL PROTECTED]> writes: > Using almost any other character than a tab does have the issue that > said separator character would need a way to be escaped and displayed. And why does tab don't need a way to be escaped? -- O T A V I OS A L V A D O R --

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-16 Thread Joey Hess
Frans Pop wrote: > I agree with Otavio that using a tab as column separator is not a good > idea, but for a completely different reason. > I can imagine you'd have a case where you'd want a string including > columns to be translated. If you use a tab, this is almost guaranteed to > result in broke

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-15 Thread Frans Pop
Jérémy Bobbio wrote: > Attached is a patch adding a new capability to cdebconf named "align", > following Anton's proposal [1] and as discussed during the september IRC > meeting [2]. I've given this patch a try, and it looks really good! I agree with Otavio that using a tab as column separator

Re: [RFC] Support column alignment in (multi)select debconf questions

2007-10-13 Thread Otavio Salvador
Jérémy Bobbio <[EMAIL PROTECTED]> writes: > * The \t (TAB) character is the column separator. > > This last point raised some concerns for Otavio as tabs might be more > difficult to spot while reviewing templates. Christian confirmed my > feeling that it was a good idea though, as he told me th

[RFC] Support column alignment in (multi)select debconf questions

2007-10-13 Thread Jérémy Bobbio
Hi! Attached is a patch adding a new capability to cdebconf named "align", following Anton's proposal [1] and as discussed during the september IRC meeting [2]. cdebconf/src/commands.c |5 cdebconf/src/frontend.c |6 cdebc