[Bug bootstrap/39023] New: lto-plugin.c uses mkdtemp unconditionally

[Bug bootstrap/39023] New: lto-plugin.c uses mkdtemp unconditionally

[ro at gcc dot gnu dot org]

lto-plugin.c uses mkdtemp unconditionally, which breaks Solaris 10/x86
bootstrap
where this function is missing (while it was introduced for Solaris 11).  There
should be a replacement, but the fix for PR bootstrap/39022 would avoid this
since gold currently cannot be used on non-GNU/Linux configurations (cf. PR
gold/7024).


-- 
           Summary: lto-plugin.c uses mkdtemp unconditionally
           Product: gcc
           Version: lto
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: dnovillo at gcc dot gnu dot org
        ReportedBy: ro at gcc dot gnu dot org
 GCC build triplet: i386-pc-solaris2.10
  GCC host triplet: i386-pc-solaris2.10
GCC target triplet: i386-pc-solaris2.10


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[bje at gcc dot gnu dot org]

-- 

bje at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-07-08 06:16:20
               date|                            |


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[ro at gcc dot gnu dot org]

------- Comment #1 from ro at gcc dot gnu dot org  2009-07-09 19:35 -------
Despite Diegos fix

2009-05-20  Diego Novillo  <dnovillo@google.com>

        * configure.ac (host_libs): Remove lto-plugin.
        Add --enable-lto.
        Add --with-libelf, --with-libelf-include and --with-libelf-lib.
        If --enable-lto is used, add 'lto' to new_enable_languages.
        If --enable-lto is used and gold is enabled, add
        lto-plugin to configdirs.

I've just submitted the trivial patch to fix this:

http://gcc.gnu.org/ml/gcc-patches/2009-07/msg00525.html
this still doesn't work.


-- 


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[bje at gcc dot gnu dot org]

------- Comment #2 from bje at gcc dot gnu dot org  2009-07-15 02:01 -------
Fixed in r149434.


-- 

bje at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[ro at gcc dot gnu dot org]

------- Comment #3 from ro at gcc dot gnu dot org  2009-07-15 16:05 -------
Sorry, my Comment #1 was directed at the wrong bug.  mkdtemp is still used
unconditionally, and this would e.g. affect Solaris 10 where it is missing.


-- 

ro at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2009-10-03 22:07 -------
Still true.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |lto


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

[Bug bootstrap/39023] lto-plugin.c uses mkdtemp unconditionally

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2010-04-23 14:33 -------
This was fixed by

2009-11-05  Rafael Avila de Espindola  <espindola@google.com>

        * lto-plugin.c (temp_obj_dir_name): Remove.
        (arguments_file_name): New.
        (free_2): Free arguments_file_name instead of temp_obj_dir_name.
        (exec_lto_wrapper):  Create arguments file with make_temp_file.
        (cleanup_handler): Don't remove the temporary directory. Remove the
        arguments file.
        (onload): Don't create the temporary directory.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.5.0


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