Processed: Re: Reworking the GTK+ cdebconf frontend

2007-07-04 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]: > tags 402126 pending Bug#402126: GTK frontend should not reload keymap at every frontend_go() call There were no tags set. Tags added: pending > thanks Stopping processing here. Please contact me if you need assistance. Debian bug tracking system admi

Bug#402126: Reworking the GTK+ cdebconf frontend

2007-07-04 Thread Attilio Fiandrotti
tags 402126 pending thanks Attilio Fiandrotti wrote: 4) As suggested by Jeremy on IRC, reading d-i/keymap while keeping track of last known value and looking for changes. This is, IMHO, the easiest solution available ATM, and morover doesn't require touching other parts of the installer th

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Jérémy Bobbio
On Mon, Jul 02, 2007 at 10:45:47AM +0200, Attilio Fiandrotti wrote: > We must decide anyway how kbd-chooser and the GTK frontend should > communicate (although it's one-way signaling, from kbd-chooser to the > GTK frontend). > > Options are > > 1) by mean of a debconf question (debconf abuse?)

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Attilio Fiandrotti
Attilio Fiandrotti wrote: Frans Pop wrote: On Monday 02 July 2007 09:34, Attilio Fiandrotti wrote: A reasonable improvement over current situation could prbably be setting TRUE a boolean "d-i/keymap_changed" question from within the keymap-chooser everytime the console keymap is updated. This

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Attilio Fiandrotti
Frans Pop wrote: On Monday 02 July 2007 09:34, Attilio Fiandrotti wrote: A reasonable improvement over current situation could prbably be setting TRUE a boolean "d-i/keymap_changed" question from within the keymap-chooser everytime the console keymap is updated. This could be an option, but us

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Attilio Fiandrotti
Loïc Minier wrote: On Mon, Jul 02, 2007, Attilio Fiandrotti wrote: The core of the problem is not detecting the keymap change at the toolkit level (which is unimportant in our case) but rather informing the windowing system (DirectFB in our case) that the keymap has to be updated to match the

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Loïc Minier
On Mon, Jul 02, 2007, Attilio Fiandrotti wrote: > The core of the problem is not detecting the keymap change at the toolkit > level (which is unimportant in our case) but rather informing the windowing > system (DirectFB in our case) that the keymap has to be updated to match > the console one s

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Frans Pop
On Monday 02 July 2007 09:34, Attilio Fiandrotti wrote: > A reasonable improvement over current situation could prbably be > setting TRUE a boolean "d-i/keymap_changed" question from within the > keymap-chooser everytime the console keymap is updated. This could be an option, but using debconf for

Re: Reworking the GTK+ cdebconf frontend

2007-07-02 Thread Attilio Fiandrotti
Loïc Minier wrote: Because keymap reloading may be a common issue for other gtk/dfb based apps, i think a reasonable way to proceed is adding a gdk_directfb_keymap_reload() Would there be a X11 equivalent for this? I'm not sure I agree Gdk needs to gain a directfb specific function conc

Re: Reworking the GTK+ cdebconf frontend

2007-06-28 Thread Loïc Minier
On Thu, Jun 28, 2007, Attilio Fiandrotti wrote: [...] > The cleanest solution would probably be delegating keymap reloading > (acually, the calling of dfb_input_device_reload_keymap() ) to an > external application, called by the a debconf client only when needed, > and not by the frontend. Hm

Re: Reworking the GTK+ cdebconf frontend

2007-06-28 Thread Attilio Fiandrotti
Colin Watson wrote: On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: * r47575 - There are plans for having cdebconf replacing debconf someday, this means the gtk frontend should build against gtk/x11 too [1]. Including more directfb private includes files goes the oppposite w

Re: Reworking the GTK+ cdebconf frontend

2007-06-27 Thread Otavio Salvador
Colin Watson <[EMAIL PROTECTED]> writes: > On Fri, Jun 22, 2007 at 11:52:56AM +0200, Attilio Fiandrotti wrote: >> Apropos of the way of performing commits, after the g-i became part of >> official builds i became very prudent about committing huge changes to >> the sourcecode. >> >> If you look

Re: Reworking the GTK+ cdebconf frontend

2007-06-27 Thread Colin Watson
On Fri, Jun 22, 2007 at 11:52:56AM +0200, Attilio Fiandrotti wrote: > Apropos of the way of performing commits, after the g-i became part of > official builds i became very prudent about committing huge changes to > the sourcecode. > > If you look into last months d-boot archives, you'll see tha

Re: Reworking the GTK+ cdebconf frontend

2007-06-27 Thread Colin Watson
On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: > * r47575 > - There are plans for having cdebconf replacing debconf someday, this > means the gtk frontend should build against gtk/x11 too [1]. > Including more directfb private includes files goes the oppposite way, > so this

Re: Reworking the GTK+ cdebconf frontend

2007-06-23 Thread Attilio Fiandrotti
Jérémy Bobbio wrote: On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: I must evantually say i'm disappointed of the way the gtk frontend code was nonchalantly modified, without any patch being posted and discussed on d-boot previously, moreover proving to ignore many design d

Re: Reworking the GTK+ cdebconf frontend

2007-06-23 Thread Attilio Fiandrotti
Jérémy Bobbio wrote: On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: I must evantually say i'm disappointed of the way the gtk frontend code was nonchalantly modified, without any patch being posted and discussed on d-boot previously, moreover proving to ignore many design d

Re: Reworking the GTK+ cdebconf frontend

2007-06-22 Thread Attilio Fiandrotti
Jérémy Bobbio wrote: On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: I must evantually say i'm disappointed of the way the gtk frontend code was nonchalantly modified, without any patch being posted and discussed on d-boot previously, moreover proving to ignore many design d

Re: Reworking the GTK+ cdebconf frontend

2007-06-21 Thread Jérémy Bobbio
On Thu, Jun 21, 2007 at 03:35:09PM +0200, Attilio Fiandrotti wrote: > I must evantually say i'm disappointed of the way the gtk frontend code > was nonchalantly modified, without any patch being posted and discussed > on d-boot previously, moreover proving to ignore many design decisions > behin

Re: Reworking the GTK+ cdebconf frontend

2007-06-21 Thread Attilio Fiandrotti
Jérémy Bobbio wrote: On Thu, Jun 21, 2007 at 10:39:24AM +0200, Attilio Fiandrotti wrote: Jérémy Bobbio wrote: The whole frontend is currently in one huge single C file. I'm proofreading the whole thing for refactoring, memory leaks, potentiel segfauts, etc. I'd like to know What's the problem

Re: Reworking the GTK+ cdebconf frontend

2007-06-21 Thread Jérémy Bobbio
On Thu, Jun 21, 2007 at 10:39:24AM +0200, Attilio Fiandrotti wrote: > Jérémy Bobbio wrote: > >The whole frontend is currently in one huge single C file. I'm > >proofreading the whole thing for refactoring, memory leaks, potentiel > >segfauts, etc. > > I'd like to know What's the problem in having

Re: Reworking the GTK+ cdebconf frontend

2007-06-21 Thread Attilio Fiandrotti
Jérémy Bobbio wrote: Hi! Just a short notice to tell everyone that I have started to work on improving the code of the GTK+ frontend for cdebconf. The whole frontend is currently in one huge single C file. I'm proofreading the whole thing for refactoring, memory leaks, potentiel segfauts, etc.

Reworking the GTK+ cdebconf frontend

2007-06-20 Thread Jérémy Bobbio
Hi! Just a short notice to tell everyone that I have started to work on improving the code of the GTK+ frontend for cdebconf. The whole frontend is currently in one huge single C file. I'm proofreading the whole thing for refactoring, memory leaks, potentiel segfauts, etc. This should keep me p