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 "get

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 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 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 that

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 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 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 do

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 ico

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 nee

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. :-) OK.

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/

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 whe

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 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

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
> "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 i