Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-24 Thread Georg Baum
Abdelrazak Younes wrote: The simple fact that utils in package.C can _exit_ lyx is an interface violation so please don't try to give me lesson about good interface. Sigh. There are more interface violations already, e.g. in src/support/filetools.C // FIXME Interface violation #include

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-24 Thread Georg Baum
Abdelrazak Younes wrote: > The simple fact that utils in package.C can _exit_ lyx is an interface > violation so please don't try to give me lesson about good interface. Sigh. There are more interface violations already, e.g. in src/support/filetools.C // FIXME Interface violation #include

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
younes == younes [EMAIL PROTECTED] writes: younes Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: younes 16823 younes URL: http://www.lyx.org/trac/changeset/16823 Log: With this younes commit, errors and warnings can use message box even before younes LyX is properly initialized.

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: younes == younes [EMAIL PROTECTED] writes: younes Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: younes 16823 younes URL: http://www.lyx.org/trac/changeset/16823 Log: With this younes commit, errors and warnings can use message box even before younes

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: This would be a regression. Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. So when the debian installer that runs as root uses export to create

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: This would be a regression. Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. So when the debian installer that runs

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Michael Gerz
Abdelrazak Younes schrieb: Jean-Marc Lasgouttes wrote: Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: This would be a regression. Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. So when the

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread José Matos
On Tuesday 23 January 2007 1:26:49 pm Abdelrazak Younes wrote: Jean-Marc Lasgouttes wrote: younes == younes [EMAIL PROTECTED] writes: younes Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: younes 16823 younes URL: http://www.lyx.org/trac/changeset/16823 Log: With this

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
José Matos wrote: I propose a compromise, write the warnings both to lyxerr and to the dialog. OK After all you raise a dialog because you thing the issue is sufficiently important to get one. Yes. If the mode is nogui don't raise the dialog. This will please everyone. :-)

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Michael Gerz wrote: Abdelrazak Younes schrieb: Jean-Marc Lasgouttes wrote: Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: This would be a regression. Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: Abdelrazak I am just trying to improve the experience of the typical Abdelrazak user. The typical user won't open a console to launch lyx Abdelrazak (ether on Windows, Linux or Mac), he will just Abdelrazak double-click an icon. If there

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Georg Baum
Abdelrazak Younes wrote: Jean-Marc Lasgouttes wrote: Abdelrazak == Abdelrazak Younes [EMAIL PROTECTED] writes: This would be a regression. Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. So when

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Georg Baum
Abdelrazak Younes wrote: I didn't know that tex2lyx is using the same package.C.in as LyX. Could we change that? No, we should not. tex2lyx should use the same package.C, otherwise we either get a maintenance nightmare or we risk that it uses other directories. The fact that you use something

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Bo Peng
Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. Just let you know that I use 'lyx --export pdf2 my.lyx' quite often on command line, which may or may not have a $DISPLAY (an X-server). Bo

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Georg Baum wrote: Abdelrazak Younes wrote: I didn't know that tex2lyx is using the same package.C.in as LyX. Could we change that? No, we should not. tex2lyx should use the same package.C, otherwise we either get a maintenance nightmare or we risk that it uses other directories. The fact

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Bo Peng wrote: Abdelrazak batch mode is only for export up to now. Here there is no Abdelrazak regression as the export function don't need any dialog. Just let you know that I use 'lyx --export pdf2 my.lyx' quite often on command line, which may or may not have a $DISPLAY (an X-server).

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Bo Peng
As I said, my commit did not break that at all. Then I have no opinion about your changes. :-) Bo

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
> "younes" == younes <[EMAIL PROTECTED]> writes: younes> Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: younes> 16823 younes> URL: http://www.lyx.org/trac/changeset/16823 Log: With this younes> commit, errors and warnings can use message box even before younes> LyX is properly

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: "younes" == younes <[EMAIL PROTECTED]> writes: younes> Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: younes> 16823 younes> URL: http://www.lyx.org/trac/changeset/16823 Log: With this younes> commit, errors and warnings can use message box even before

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: >> This would be a regression. Abdelrazak> batch mode is only for export up to now. Here there is no Abdelrazak> regression as the export function don't need any dialog. So when the debian installer that runs as root uses

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: This would be a regression. Abdelrazak> batch mode is only for export up to now. Here there is no Abdelrazak> regression as the export function don't need any dialog. So when the debian installer that

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Michael Gerz
Abdelrazak Younes schrieb: Jean-Marc Lasgouttes wrote: "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: This would be a regression. Abdelrazak> batch mode is only for export up to now. Here there is no Abdelrazak> regression as the export function don't need any dialog. So

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread José Matos
On Tuesday 23 January 2007 1:26:49 pm Abdelrazak Younes wrote: > Jean-Marc Lasgouttes wrote: > >> "younes" == younes <[EMAIL PROTECTED]> writes: > > > > younes> Author: younes Date: Tue Jan 23 13:38:20 2007 New Revision: > > younes> 16823 > > > > younes> URL:

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
José Matos wrote: I propose a compromise, write the warnings both to lyxerr and to the dialog. OK After all you raise a dialog because you thing the issue is sufficiently important to get one. Yes. If the mode is nogui don't raise the dialog. This will please everyone. :-)

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Michael Gerz wrote: Abdelrazak Younes schrieb: Jean-Marc Lasgouttes wrote: "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: This would be a regression. Abdelrazak> batch mode is only for export up to now. Here there is no Abdelrazak> regression as the export function don't

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Jean-Marc Lasgouttes
> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: Abdelrazak> I am just trying to improve the experience of the typical Abdelrazak> user. The typical user won't open a console to launch lyx Abdelrazak> (ether on Windows, Linux or Mac), he will just Abdelrazak> double-click an

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Georg Baum
Abdelrazak Younes wrote: > Jean-Marc Lasgouttes wrote: >>> "Abdelrazak" == Abdelrazak Younes >>> <[EMAIL PROTECTED]> writes: >> This would be a regression. >> >> Abdelrazak> batch mode is only for export up to now. Here there is no >> Abdelrazak> regression as the export function

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Georg Baum
Abdelrazak Younes wrote: > I didn't know that tex2lyx is using the same package.C.in as LyX. Could > we change that? No, we should not. tex2lyx should use the same package.C, otherwise we either get a maintenance nightmare or we risk that it uses other directories. The fact that you use

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Bo Peng
>> Abdelrazak> batch mode is only for export up to now. Here there is no >> Abdelrazak> regression as the export function don't need any dialog. Just let you know that I use 'lyx --export pdf2 my.lyx' quite often on command line, which may or may not have a $DISPLAY (an X-server). Bo

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Georg Baum wrote: Abdelrazak Younes wrote: I didn't know that tex2lyx is using the same package.C.in as LyX. Could we change that? No, we should not. tex2lyx should use the same package.C, otherwise we either get a maintenance nightmare or we risk that it uses other directories. The fact

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Abdelrazak Younes
Bo Peng wrote: >> Abdelrazak> batch mode is only for export up to now. Here there is no >> Abdelrazak> regression as the export function don't need any dialog. Just let you know that I use 'lyx --export pdf2 my.lyx' quite often on command line, which may or may not have a $DISPLAY (an

Re: [Cvslog] r16823 - in /lyx-devel/trunk/src: frontends/Alert.C front...

2007-01-23 Thread Bo Peng
As I said, my commit did not break that at all. Then I have no opinion about your changes. :-) Bo