This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Remove __GNUC__ / _MSC_VER games from win32/config_H.*
authorSteve Hay <steve.m.hay@googlemail.com>
Sat, 9 Jun 2012 10:57:58 +0000 (11:57 +0100)
committerSteve Hay <steve.m.hay@googlemail.com>
Sat, 9 Jun 2012 16:58:53 +0000 (17:58 +0100)
commit3090e57a7623df3083462032f942f4bb64d46921
treea5d649599f5f17b9f65e0927150257a74cdd0bae
parentc716620072a2a924157c74219f6e6a9b5c3a8d46
Remove __GNUC__ / _MSC_VER games from win32/config_H.*

The code was added by 465b7da985, based on changes in ActivePerl 816, but
it's not working correctly because the current build process winds up with
a config.h in lib/CORE which doesn't contain that code anyway.

(The appropriate canned config_H.* file is copied to config.h in order to
build miniperl.exe, but then miniperl.exe is used to run config_h.PL to
generate the real config.h from config_h.SH and config.sh. The real
config.h is then used to build perl.exe and is what gets copied into
lib/CORE for later use when building extensions, but it doesn't contain
the support for other compilers because that isn't in config_h.SH...)

Removing the code won't affect ActivePerl since it has not been getting
installed anyway, and Jan Dubois has confirmed that it is possible to
build a Perl extension using e.g. Off_t with MinGW even with the VC
config.h header file. Furthermore, anyone building perl from source
themselves will presumably be able to use the same compiler to build
extensions as they used to build perl itself anyway, so there is no
obvious need for this in the core perl distro.

Therefore, removing it rather than fixing it is better for simplicity.
win32/Makefile
win32/config_H.gc
win32/config_H.gc64
win32/config_H.gc64nox
win32/config_H.vc
win32/config_H.vc64
win32/makefile.mk