mainGNU Libtool - Support: sr #109514, ltmain.sh: [_MSC_VER] should be...

 
 

sr #109514: ltmain.sh: [_MSC_VER] should be [_WIN32] in one place

Submitter:  None
Submitted:  Mon 04 Jun 2018 02:21:41 PM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  3 - Normal Status:  None
Privacy:  Public Assigned to:  None
Originator Email:  -email is unavailable- Open/Closed:  Closed
Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 07 Mar 2024 03:33:29 PM UTC, comment #1: 

Thank you! A similar patch was submitted and applied [1].

[1] https://savannah.gnu.org/patch/index.php?9678

Ileana Dumitrescu <ildumi>
Group administrator
Mon 04 Jun 2018 02:21:41 PM UTC, original submission:  

diff:

 #include <stdio.h>
 #include <stdlib.h>
-#ifdef _MSC_VER
+#if defined (WIN32) && !defined (__GNUC_)
 # include <direct.h>
 # include <process.h>
 # include <io.h>
 #else
 # include <unistd.h>
 # include <stdint.h>
 # ifdef _CYGWIN_
 #  include <io.h>
 # endif
 #endif

The reason is that there are much more compilers for win32 than the proprietary MSVC. All I'm aware of that are not GCC "based" need process.h instead of unistd.h.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ildumi (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2024-03-07 ildumi Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code