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

Re: Bug#220868: Patch to implement a CLEARSEEN command



Denis Barbier wrote:
On Mon, Nov 17, 2003 at 12:50:48PM +0100, Sebastian Ley wrote:
[...]

But I am quite certain that this whole stuff is a bad idea, how to abort
and thus go back to main-menu should be handled directly by cdebconf,
and not by config scripts.

I suppose that is impossible (correct me if I'm wrong) since cdebconf
knows nothing of the question hierarchy and neither of main-menu.


When cdebconf runs a config script, questions are not immediately flagged
as being seen, but pushed into a stack containing all seen questions.
When the script ends, the database is updated, and when user goes back
the question is removed from the stack.
great
i don't know if this option exists in debconf, but i think it will be useful and avoid big config.dat with
/apt-setup/baddir
/apt-setup/cd/another
/apt-setup/cd/bad


In his patch, Steinar adds tha ability to purge this stack before
exiting, and thus question status is not updated.  This can be very
useful, my only disagreement being that IMO this purge should not be
requested by config scripts, but by cdebconf itself.  I wrote a patch
yesterday to add an 'Abort' button to progress bars, which kills
the config script (by sending SIGTERM to the right pid, as suggested
by Steinar in another thread) and thus will return to main-menu.
I am now polishing it, and am adding an abort capability the same way
as backup works, i.e. an 'Abort' button will be displayed if it is
supported by the frontend.  It will go into a CVS branch when it is
finished.

Denis







Reply to: