bugGNUstep - Bugs: bug #19321, $PATH not set properly under MINGW

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #19321: $PATH not set properly under MINGW

Submitted by:  Xavier Glattard <xgl>
Submitted on:  Fri 16 Mar 2007 09:53:15 AM UTC  
 
Category: MakefilesSeverity: 3 - Normal
Item Group: BugStatus: Fixed
Privacy: PublicAssigned to: Nicola Pero <nico>
Open/Closed: Closed

Fri 16 Mar 2007 10:47:01 AM UTC, comment #1:

Thanks ... your patch makes complete sense to me -- fixpath.sh has explicit if/else for the various options, and obviously the cygwin one is supposed to be for cygwin and the mingw one for mingw. ;-)

So I applied it to trunk. ;-)

Thanks

PS: I'm not on MinGW at the moment though, so I couldn't really test it.

Nicola Pero <nico>
Project MemberIn charge of this item.
Fri 16 Mar 2007 09:53:15 AM UTC, original submission:

Hi

On my XP box with MSYS :

$ echo $PATH
/home/Xavier/GNUstep/Tools:/cygdrive/C/GNUstep/Local/Tools:/cygdri
ve/C/GNUstep/System/Tools:.:/usr/local/bin:/mingw/bin:/bin:/c/WIND
OWS/system32:/c/WINDOWS:/c/WINDOWS/System32/Wbem:/c/Program Files/Fichiers communs/GTK/2.0/bin

As you can see GNUstep paths are translated form C:\GNUstep to /cygdrive/GNUstep. There should be /C/GNUstep

I found out that the CYGWIN variable is set to 'yes'
If I change configure (and congigure.ac) in gnustep-make the path is set properly (see patch below). But is it the good way ?

- Xavier

Index: configure
===================================================================
--- configure (revision 24878)
+++ configure (working copy)
@@ -2780,7 +2780,7 @@
MSWIND=yes;;
mingw32 )
MINGW32=yes
- CYGWIN=yes
+ CYGWIN=no
MSWIND=yes;;
* )
MINGW32=no
Index: configure.ac
===================================================================
--- configure.ac (revision 24878)
+++ configure.ac (working copy)
@@ -120,7 +120,7 @@
MSWIND=yes;;
mingw32 )
MINGW32=yes
- CYGWIN=yes
+ CYGWIN=no
MSWIND=yes;;
* )
MINGW32=no

Xavier Glattard <xgl>
Project Member

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by nico (Posted a comment)
  • -unavailable- added by xgl (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 16 Mar 2007 10:47:01 AM UTCnicoStatusNone=>Fixed
      Assigned toNone=>nico
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1