LMS-Community / slimserver-vendor

Third-party software used with Lyrion Music Server
https://lyrion.org
42 stars 68 forks source link

bugfix: Correct errors in Perl's CC detection #40

Closed fsbruva closed 6 years ago

fsbruva commented 6 years ago

The PERL_CC test currently uses ARCHPERL, and thus assumes that ARCHPERL is the same as PERL_BIN. This is a logic error, because PERL_BIN (the version of Perl being used by the build) may differ from ARCHPERL, giving erroneous feedback to the user.

Avoid a logic error from an early call to a potentially different Perl than the one buildme.sh will ultimately use. Remove erroneous use of ARCHPERL for compiler difference tests. Add PERL_CC compiler tests using validated PERL_BIN. Invert the supported OS check logic and declare ARCH using PERL_BIN.

Closes #39