[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[dannysmith at gcc dot gnu dot org]

--- Comment #7 from dannysmith at gcc dot gnu dot org  2009-01-13 07:41 
---
Subject: Bug 38580

Author: dannysmith
Date: Tue Jan 13 07:40:51 2009
New Revision: 143330

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143330
Log:
PR bootstrap/38580
* gcc.c (process_command): Replace call to execvp with calls
to pex_one and exit.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/gcc.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[dannysmith at users dot sourceforge dot net]

--- Comment #8 from dannysmith at users dot sourceforge dot net  2009-01-13 
07:43 ---
Fixed on trunk.


-- 

dannysmith at users dot sourceforge dot net changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[jakub at gcc dot gnu dot org]

--- Comment #5 from jakub at gcc dot gnu dot org  2009-01-09 09:36 ---
Seems to work on x86_64-linux.  The error message is different if the program
doesn't exist:
./xgcc -b nonexistent-target -B ./ -v
./xgcc: error trying to exec './nonexistent-target-gcc-4.4.0': execvp: No such
file or directory
from:
./xgcc -b nonexistent-target -B ./ -v
xgcc: couldn't run './nonexistent-target-gcc-4.4.0': No such file or directory
but I guess that's not a big deal.  When the program exists, it works just
fine.  If it works on mingw, can you post it to gcc-patches?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[ktietz at gcc dot gnu dot org]

--- Comment #6 from ktietz at gcc dot gnu dot org  2009-01-09 09:55 ---
(In reply to comment #4)
 Created an attachment (id=17052)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17052action=view) [edit]
 Replace execvp with pex_one in process_command
 Patch uses pex_one as per Ian Taylor suggestion , but the error reporting may
 need expansion.
 Danny

Ok, I agree that the error message shouldn't be here a big problem. Your patch
works for mingw-w64, so I have no objections.
Kai


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[dannysmith at users dot sourceforge dot net]

--- Comment #4 from dannysmith at users dot sourceforge dot net  2009-01-08 
08:41 ---
Created an attachment (id=17052)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17052action=view)
Replace execvp with pex_one in process_command

Patch uses pex_one as per Ian Taylor suggestion , but the error reporting may
need expansion.
Danny


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[jakub at gcc dot gnu dot org]

--- Comment #3 from jakub at gcc dot gnu dot org  2009-01-08 00:05 ---
Ian Taylor said on IRC that -b/-V should use pex_one/exit instead of execvp.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580

[Bug bootstrap/38580] [4.4 Regression] Bootstrap broken on mingw32

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

Summary|Bootstrap broken on mingw32 |[4.4 Regression] Bootstrap
   ||broken on mingw32
   Target Milestone|--- |4.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38580