On Sat, Apr 08, 2006 at 09:45:21PM +0200, Georg Baum wrote: > Am Samstag, 8. April 2006 18:14 schrieb Enrico Forestieri: > > All the necessary machinery to activate it is already present in > > configure and configure.py, so this simple patch to os_win32.C > > allows using the cygwin tetex with a native win32 LyX. > > > > Log: > > * src/support/os_win32.C: activate cygwin-tetex support > > > > Do with it whatever you think fits better ;-) > > I don't like the hardcoded "/cygdrive/", because that would fail for > people who changed that name. IMO if somebody wants to use cygwin tetex > he may as well use cygwin LyX. We don't need to support all possible > combinations.
You are right, of course. However, apart from the fact that it would work in 99.9% of cases, I bet, it could be easily fixed by getting that info directly from cygwin. Anyway, I don't want to make things unnecessarily complicated. Consider that that code would be actually used only when the user has the cygwin Bourne shell (or python) as only in that case (and independently of this patch) the cygwin_path_fix_needed switch gets written to lyxrc.defaults (and thus cygwin_path_fix_ can be true). In my eyes it would be a nice zero-cost shield in case someone uses a native win32 LyX but with cygwin tools. Then, you are right again that he who uses cygwin tetex may as well use cygwin LyX, so I only mildly endorse this patch, which I put together in less than ten minutes. For this reason I said to do with it whatever you think fits better. -- Enrico
