On Fri, Mar 24, 2006 at 02:04:53PM +0100, Georg Baum wrote:
> No, it does not need posix style paths: It simply cannot cope with
> backslashes for obvious reasons. AFAIK miktex can handle paths like
> "C:/temp/test.tex". Not a big difference, but not exactly posix either.
This path style is calle
On Fri, Mar 24, 2006 at 01:43:55PM +0100, Georg Baum wrote:
> Enrico Forestieri wrote:
>
> > I have a comment on this patch. I see that you get rid of the rtrim call
> > in latex_path. This has the following effect: in all cases the argument
> > of [EMAIL PROTECTED] will have two '/' appended, exc
Enrico Forestieri wrote:
> On Fri, Mar 24, 2006 at 11:09:00AM +0100, Georg Baum wrote:
>> So what do you think about the patch? Should it go in, or should we first
>> research the "use cygwin_path_fix in external_path or not" issue?
>
> In my experience, if a cygwin application does not try to ma
Enrico Forestieri wrote:
> I have a comment on this patch. I see that you get rid of the rtrim call
> in latex_path. This has the following effect: in all cases the argument
> of [EMAIL PROTECTED] will have two '/' appended, except on cygwin when
> cygwin_path_fix_ is true, in which case only one
On Fri, Mar 24, 2006 at 12:51:17PM +0100, Georg Baum wrote:
> Jean-Marc Lasgouttes wrote:
>
> > I think I agree with it. Could you repost it please? I am lost in the
> > thread :)
>
> Sure, here it is.
I have a comment on this patch. I see that you get rid of the rtrim call
in latex_path. This h
On Fri, Mar 24, 2006 at 11:09:00AM +0100, Georg Baum wrote:
> Jean-Marc Lasgouttes wrote:
>
> > I thought the idea of cygwin_path_fix was that it is only needed for
> > latex. I am not so sure anymore.
Why do you say so?
> The lines
>
> //No backslashes in LaTeX files
> dos_path
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:
Georg> Jean-Marc Lasgouttes wrote:
>> I think I agree with it. Could you repost it please? I am lost in
>> the thread :)
Georg> Sure, here it is.
Yes, that's the one I saw. Apply it to trunk and branch.
JMarc
Jean-Marc Lasgouttes wrote:
> I think I agree with it. Could you repost it please? I am lost in the
> thread :)
Sure, here it is.
GeorgIndex: src/support/os_unix.C
===
--- src/support/os_unix.C (Revision 13467)
+++ src/support/os_u
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:
Georg> Anyway, the patch is somewehat orthogonal to the question
Georg> whether cygwin_path_fix should be used in external_path or not.
Georg> We would need to change the implementation of latex_path() if
Georg> that should be true, but the e
Jean-Marc Lasgouttes wrote:
> I thought the idea of cygwin_path_fix was that it is only needed for
> latex. I am not so sure anymore.
The lines
//No backslashes in LaTeX files
dos_path = subst(dos_path,'\\','/');
would support the argument that external_path() is intended for la
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:
Georg> Here you basically revert external_path on non-cygwin windows.
Georg> This is a bit ugly, so I moved this into a new function
Georg> lyx::support::os::latex_path(). This is a noop except on
Georg> cygwin, where it is identical to exter
Georg Baum <[EMAIL PROTECTED]> writes:
> > Many thanks for your helpful and appreciated comments.
>
> I hope they are not too disappointing
Oh, not at all...
> Here you basically revert external_path on non-cygwin windows. This is a
> bit ugly, so I moved this into a new function
> lyx::supp
Am Donnerstag, 23. März 2006 21:26 schrieb Enrico Forestieri:
> Your theory is right and I wasn't fearless enough, not having the time
> to study the code. So, the attached is apparently the right fix. While
> testing this patch I run up against another bug.
>
> 1) File->New
> 2) Insert->File->Ch
Georg Baum <[EMAIL PROTECTED]> writes:
> I think that you can safely call it unconditionally. I looked it up, and
> latex_path() is only called on filenames that are written to .tex files. I
> would be very surprised if a TeX compiler exists on windows that needs win
> style paths in \inputa path
Enrico Forestieri wrote:
> The problem is that I have not a full grasp of the overall code and don't
> want introduce changes without fully knowing the consequences. I repeat,
> my idea is of only fixing what is broken, making changes that I fully
> understand. As a problem arised with [EMAIL PROT
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
>
> > "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:
>
> Georg> Now I don't like either anymore. The problem of both fixes is
> Georg> that they use a trailing '/' as flag whether external_path() is
> Georg> needed or not. This will strike b
Georg Baum <[EMAIL PROTECTED]> writes:
> What I meant was: Do we need for one setting of cygwin_path_fix_needed both
> windos/posix styles, or does cygwin_path_fix_needed simply switch
> external_path() between the windows and posix version?
As it is now, cygwin_path_fix_needed simply switch exte
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:
Georg> Now I don't like either anymore. The problem of both fixes is
Georg> that they use a trailing '/' as flag whether external_path() is
Georg> needed or not. This will strike back sooner or later.
Sure, this is not good. Is the problem t
Lars Gullik Bjønnes wrote:
> I feel the exact opposite. the cygwin_path_fix pollutes the "portable"
> code, pollutes the "not cygwin files"
But it makes it explicit why external_path() is needed.
> The second patch is the nice one IMO.
Now I don't like either anymore. The problem of both fixes
Georg Baum <[EMAIL PROTECTED]> writes:
| Enrico Forestieri wrote:
|
| > The attached patch fixes the bug reported here:
| > http://thread.gmane.org/gmane.editors.lyx.general/29227
| >
| > In my intention this is the first of a series of patches aimed at
| > polishing the Cygwin target, which I t
Enrico Forestieri wrote:
> Trailing, not heading /. But it seems I was misunderstanding the code.
And I was misunderstanding your post :-(
>> I am not really sure how things work on cygwin. Can we always assume that
>> all external paths are either windows or posix style? Or do we need posix
>>
Georg Baum <[EMAIL PROTECTED]> writes:
>
> Enrico Forestieri wrote:
>
> > Really? I had thought the second one was more cleaner. Actually I had to
> > figure out that when latex_path() is called for generating the argument
> > of \input path it is assumed that a trailing / is present, and the
>
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
>
> > "Enrico" == Enrico Forestieri <[EMAIL PROTECTED]> writes:
>
> Enrico> Really? I had thought the second one was more cleaner.
> Enrico> Actually I had to figure out that when latex_path() is called
> Enrico> for generating the argument of
Enrico Forestieri wrote:
> Really? I had thought the second one was more cleaner. Actually I had to
> figure out that when latex_path() is called for generating the argument
> of [EMAIL PROTECTED] it is assumed that a trailing / is present, and the
> second
> patch makes this clear not only for c
> "Enrico" == Enrico Forestieri <[EMAIL PROTECTED]> writes:
Enrico> Really? I had thought the second one was more cleaner.
Enrico> Actually I had to figure out that when latex_path() is called
Enrico> for generating the argument of [EMAIL PROTECTED] it is assumed that
Enrico> a trailing / is p
Georg Baum <[EMAIL PROTECTED]> writes:
>
> Enrico Forestieri wrote:
>
> > The attached patch fixes the bug reported here:
> > http://thread.gmane.org/gmane.editors.lyx.general/29227
> >
> > In my intention this is the first of a series of patches aimed at
> > polishing the Cygwin target, which I
Martin Vermeer <[EMAIL PROTECTED]> writes:
>
> On Wed, Mar 22, 2006 at 09:29:35PM +, Enrico Forestieri wrote:
> > The attached patch fixes the bug reported here:
> > http://thread.gmane.org/gmane.editors.lyx.general/29227
> >
> > In my intention this is the first of a series of patches aimed
Enrico Forestieri wrote:
> The attached patch fixes the bug reported here:
> http://thread.gmane.org/gmane.editors.lyx.general/29227
>
> In my intention this is the first of a series of patches aimed at
> polishing the Cygwin target, which I think is lagging behind. Please, tell
> me if this ok w
On Wed, Mar 22, 2006 at 09:29:35PM +, Enrico Forestieri wrote:
> The attached patch fixes the bug reported here:
> http://thread.gmane.org/gmane.editors.lyx.general/29227
>
> In my intention this is the first of a series of patches aimed at polishing
> the Cygwin target, which I think is laggi
Enrico Forestieri <[EMAIL PROTECTED]> writes:
>
> The attached patch fixes the bug reported here:
> http://thread.gmane.org/gmane.editors.lyx.general/29227
>
> In my intention this is the first of a series of patches aimed at polishing
> the Cygwin target, which I think is lagging behind. Please
The attached patch fixes the bug reported here:
http://thread.gmane.org/gmane.editors.lyx.general/29227
In my intention this is the first of a series of patches aimed at polishing
the Cygwin target, which I think is lagging behind. Please, tell me if this
ok with you. I have no problem in maintain
31 matches
Mail list logo
