Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-07 Thread Enrico Forestieri
On Wed, Apr 05, 2006 at 09:27:13PM +0200, Georg Baum wrote: > Am Mittwoch, 5. April 2006 03:39 schrieb Enrico Forestieri: > > I also fixed the bug about the checkbox status in QPrefs.C, as it is > > very annoying that it does the contrary of what its label says > > (inverted logic). > > What abou

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-05 Thread Enrico Forestieri
On Wed, Apr 05, 2006 at 09:27:13PM +0200, Georg Baum wrote: > Am Mittwoch, 5. April 2006 03:39 schrieb Enrico Forestieri: > > I also fixed the bug about the checkbox status in QPrefs.C, as it is > > very annoying that it does the contrary of what its label says > > (inverted logic). > > What about

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-05 Thread Enrico Forestieri
On Wed, Apr 05, 2006 at 09:27:13PM +0200, Georg Baum wrote: > Am Mittwoch, 5. April 2006 03:39 schrieb Enrico Forestieri: > > I also fixed the bug about the checkbox status in QPrefs.C, as it is > > very annoying that it does the contrary of what its label says > > (inverted logic). > > What about

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-05 Thread Georg Baum
Am Mittwoch, 5. April 2006 03:39 schrieb Enrico Forestieri: > I also fixed the bug about the checkbox status in QPrefs.C, as it is > very annoying that it does the contrary of what its label says > (inverted logic). What about qt4, xforms, gtk? Please send a patch if one of them needs this fix, t

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-04 Thread Enrico Forestieri
On Tue, Apr 04, 2006 at 10:13:40PM +0200, Georg Baum wrote: > Am Montag, 3. April 2006 11:58 schrieb Enrico Forestieri: > > Only a small glitch, Georg. In os_cygwin.C, the test "p.length() < 1" > > in is_posix_path() and is_windows_path() should be changed to > > "p.length() <= 1" > > I forgot th

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-04 Thread Georg Baum
Am Dienstag, 4. April 2006 22:18 schrieb Enrico Forestieri: > On Tue, Apr 04, 2006 at 09:50:15PM +0200, Georg Baum wrote: > > Please add a comment why this is necessary. > > I tried to explain this one here: > http://bugzilla.lyx.org/show_bug.cgi?id=2344 > > Does it suffice mentioning bug 2344 in

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-04 Thread Enrico Forestieri
On Tue, Apr 04, 2006 at 09:50:15PM +0200, Georg Baum wrote: > > Index: src/support/environment.C > > === > > --- src/support/environment.C   (revision 13547) > > +++ src/support/environment.C   (working copy) > > @@ -95,7 +95,11 @@ vo

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-04 Thread Georg Baum
Am Montag, 3. April 2006 11:58 schrieb Enrico Forestieri: > Only a small glitch, Georg. In os_cygwin.C, the test "p.length() < 1" > in is_posix_path() and is_windows_path() should be changed to > "p.length() <= 1" I forgot that, please fix thsi together with the comment. Georg

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-04 Thread Georg Baum
Am Montag, 3. April 2006 10:15 schrieb Enrico Forestieri: > On Sun, Apr 02, 2006 at 10:20:35PM +0200, Georg Baum wrote: > > > Am Sonntag, 2. April 2006 17:57 schrieb Enrico Forestieri: > > > enum path_target { > > > DEFAULT, > > > LATEX > > > } > > > > > > std::string internal_path(std::s

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-03 Thread Enrico Forestieri
On Mon, Apr 03, 2006 at 11:40:03AM +0200, Jean-Marc Lasgouttes wrote: > > "Georg" == Georg Baum <[EMAIL PROTECTED]> writes: > > >> Georg, cygwin runs on windows and it understands both posix and > >> win-style paths. So, what is external and what is internal? In the > >> win32 version C:/xxx

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-03 Thread Enrico Forestieri
On Mon, Apr 03, 2006 at 11:22:48AM +0200, Georg Baum wrote: > Enrico Forestieri wrote: > > > No, it is not because of that. The point is that > > lyx::support::latex_path() calls ChangeExtensions, which in turn > > calls internal_path(), which would turn the path into posix in > > cygwin. > > I

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-03 Thread Jean-Marc Lasgouttes
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes: >> Georg, cygwin runs on windows and it understands both posix and >> win-style paths. So, what is external and what is internal? In the >> win32 version C:/xxx is the internal representation, in the *nix >> version /c/xxx is the internal rep

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-03 Thread Georg Baum
Enrico Forestieri wrote: > No, it is not because of that. The point is that > lyx::support::latex_path() calls ChangeExtensions, which in turn > calls internal_path(), which would turn the path into posix in > cygwin. I see. Calling internal_path() in ChangeExtension seems wrong to me. I'll have

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-03 Thread Enrico Forestieri
On Sun, Apr 02, 2006 at 10:20:35PM +0200, Georg Baum wrote: > Am Sonntag, 2. April 2006 17:57 schrieb Enrico Forestieri: > > Well, I am facing a problem related to it now. Please, help this > > C++ newbie to solve the following problem. > > > > I had already the need to add an enum argument to in

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-02 Thread Georg Baum
Am Sonntag, 2. April 2006 17:57 schrieb Enrico Forestieri: > Well, I am facing a problem related to it now. Please, help this > C++ newbie to solve the following problem. > > I had already the need to add an enum argument to internal_path. > Its natural declaration was in "support/os.h" in the > l

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-02 Thread Enrico Forestieri
On Sun, Apr 02, 2006 at 04:54:44PM +, Angus Leeming wrote: > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > In C I would simply "#include " within > > "support/filetools.h", but I don't think this is the correct C++ > > solution. > > You can't forward declare enums, so your > proposed solu

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-02 Thread Angus Leeming
Enrico Forestieri <[EMAIL PROTECTED]> writes: > In C I would simply "#include " within > "support/filetools.h", but I don't think this is the correct C++ > solution. You can't forward declare enums, so your proposed solution is fine for now. An alternative would be to define a class that is conve

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-02 Thread Enrico Forestieri
On Sun, Apr 02, 2006 at 01:14:09PM +0200, Georg Baum wrote: > Am Samstag, 1. April 2006 21:17 schrieb Enrico Forestieri: > > On Sat, Apr 01, 2006 at 05:55:07PM +0200, Georg Baum wrote: > > > Almost. No boolean please (that would circumvent the whole idea of > > > internal/external path). Simply h

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-02 Thread Georg Baum
Am Samstag, 1. April 2006 21:17 schrieb Enrico Forestieri: > On Sat, Apr 01, 2006 at 05:55:07PM +0200, Georg Baum wrote: > > Almost. No boolean please (that would circumvent the whole idea of > > internal/external path). Simply hardcode the decision in internal_path > > and external_path. Later w

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-01 Thread Enrico Forestieri
On Sat, Apr 01, 2006 at 05:55:07PM +0200, Georg Baum wrote: > Am Samstag, 1. April 2006 17:07 schrieb Enrico Forestieri: > > Understood. I'll leave the checkbox thing as it is now, i.e., the user > > can subvert the configure script decisions (a really bad idea) and > > arrange things such that L

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-01 Thread Georg Baum
Am Samstag, 1. April 2006 17:07 schrieb Enrico Forestieri: > Understood. I'll leave the checkbox thing as it is now, i.e., the user > can subvert the configure script decisions (a really bad idea) and > arrange things such that LyX internally uses the format decided by > internal_path(). My idea i

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-01 Thread Enrico Forestieri
On Sat, Apr 01, 2006 at 01:46:39PM +0200, Georg Baum wrote: > I fear we have been talking past each other. I never wanted a "dirty" > patch. I wanted a correct one, but split up in smaller chunks. For > example, the additional check box could be left out in the first version, > hardcoding win s

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-04-01 Thread Georg Baum
Am Freitag, 31. März 2006 21:56 schrieb Enrico Forestieri: > Here is a minimal patch for cygwin. It fixes bug 2344 and the following > two bugs: > > - Export to latex fails when cygwin_path_fix is true, because >   prefixIs(p1,p2) in copyFile() fails to recognize that p2 is prefix of >   p1 when t

Re: [patch] fix bug 2344 and paths issues on cygwin

2006-03-31 Thread Enrico Forestieri
On Fri, Mar 31, 2006 at 09:56:41PM +0200, Enrico Forestieri wrote: > The correct fix to this problem is my other patch in the thread > "Cygwin polishing", which however, I admit tries to solve too much > things in a single shot. This is to reaffirm what I said. I just discovered another bug relat

[patch] fix bug 2344 and paths issues on cygwin

2006-03-31 Thread Enrico Forestieri
Here is a minimal patch for cygwin. It fixes bug 2344 and the following two bugs: - Export to latex fails when cygwin_path_fix is true, because prefixIs(p1,p2) in copyFile() fails to recognize that p2 is prefix of p1 when they are in different styles. - When a file is exported to latex and t