In perl.git, the branch smoke-me/nicholas/vms-filespec has been created <http://perl5.git.perl.org/perl.git/commitdiff/ee34ec23c6dae39a1a354e4c77822d17aba376d1?hp=0000000000000000000000000000000000000000>
at ee34ec23c6dae39a1a354e4c77822d17aba376d1 (commit) - Log ----------------------------------------------------------------- commit ee34ec23c6dae39a1a354e4c77822d17aba376d1 Author: Nicholas Clark <n...@ccl4.org> Date: Mon Jun 17 11:49:39 2013 +0200 Move VMS::Filespec from vms/ext to ext/ This simplifies the VMS Makefile. It would have simplified the VMS Makefile further if it had had the correct rules to delete the build products now eliminated :-) M Configure M MANIFEST M Porting/Maintainers.pl M Porting/corelist.pl A ext/VMS-Filespec/lib/VMS/Filespec.pm A ext/VMS-Filespec/t/filespec.t M vms/descrip_mms.template D vms/ext/Filespec.pm D vms/ext/filespec.t M win32/FindExt.pm M write_buildcustomize.pl commit d60184cd73fb4ab2aef56d097a61d7e88e26557c Author: Nicholas Clark <n...@ccl4.org> Date: Mon Jun 17 10:37:50 2013 +0100 vms/ext/filespec.t does not need to be +x The mode of this file has rattled back and forth between +x and -x since it was first added. It makes no difference which it is, so remove -x and hence 1 special case. M Porting/exec-bit.txt M vms/ext/filespec.t commit c3af8b653646464bb1e1f1b80a6ecbf5980a419c Author: Nicholas Clark <n...@ccl4.org> Date: Mon Jun 17 11:27:16 2013 +0200 Refactor FindExt, merging scan_ext() and find_ext(). The return value of FindExt::scan_ext() has never been used, since FindExt was first added by commit 8e2329934bcca9c5 in April 2001. The call to FindExt::extensions() has no side effects, so it can be eliminated. Hence FindExt::scan_ext() is a trivial wrapper around FindExt::find_ext(), and the two can be merged. Also, simplify the logic for "known" extensions. The complexity of checking the hash first was needed when extension directories were nested. It should have been removed as part of commit 1f8a0b38638b171c in Feb 2009. M win32/FindExt.pm commit bb6aa89b683f62dc169632e2ed9e1ab9ebc0da38 Author: Nicholas Clark <n...@ccl4.org> Date: Mon Jun 17 10:50:33 2013 +0200 Skip most of FindExt's tests for troublesome configurations. There are various various things that break the test's assumptions. 1) If Encode is a static extension, then Configure has special case logic to add Encode/* as static extensions 2) -Uusedl causes Encode to be a static extension, and drops building XS::APItest and XS::Typemap 3) Any use of -Dnoextensions to choose not to build a extension is not known by the test If any of these are true, FindExt::extensions() and $Config{extensions} will differ, and most of the tests are going to fail. Moreover, failure doesn't tell us anything interesting. So don't run those tests. M t/porting/FindExt.t commit 22a39589ad04f575cdf8d644aff8b2a8aedaa3c6 Author: Nicholas Clark <n...@ccl4.org> Date: Mon Jun 17 10:19:48 2013 +0200 In FindExt.t, move the main loop's comparison logic into a subroutine. M t/porting/FindExt.t commit 848b9eaba769ce9869f01093b7884e18f895f5a0 Author: Nicholas Clark <n...@ccl4.org> Date: Fri Jun 14 11:28:31 2013 +0200 Minor refactors to FindExt's test, removing code duplication. Use a ternary instead of if/unless on the same $^O test. Use a loop to call FindExt::scan_ext() As FindExt exports nothing, we can require it rather than using it. M t/porting/FindExt.t commit ff6b34375ec86913c0f65d3815c8ce5d3f4aa235 Author: Nicholas Clark <n...@ccl4.org> Date: Wed Jun 5 21:19:12 2013 +0200 Thanks to FindExt::apply_config() we're now able to test dynamic extensions. FindExt::apply_config() mimic's Configure's logic, which means that FindExt's idea of which extensions should be built is consistent with Configure's. M t/porting/FindExt.t commit e78f76d1d8d6f5f562ff1b8a60b0b87ead3ba4d9 Author: Nicholas Clark <n...@ccl4.org> Date: Thu Jun 6 10:51:39 2013 +0200 FindExt::apply_config() needs to skip Sys::Syslog on Win32. Without this, the test expectations on Win32 are inconsistent with what actually got built. M win32/FindExt.pm commit a58bfd55f93b7a4517183a21eaf5298814e2c17a Author: Nicholas Clark <n...@ccl4.org> Date: Wed Jun 5 20:15:12 2013 +0200 Correct a type in FindExt::apply_config in the "I18N-Langinfo" code. Commit 557ab4cb986767c7 (Feb 2011) which added this routine had $config->{i_nl_langinfo} not $config->{d_nl_langinfo} ('i' should be 'd'). The logic for I18N::Langinfo should now be correct. M win32/FindExt.pm ----------------------------------------------------------------------- -- Perl5 Master Repository