diff options
author | Nicholas Clark <nick@ccl4.org> | 2013-07-18 15:10:29 +0200 |
---|---|---|
committer | Nicholas Clark <nick@ccl4.org> | 2013-07-24 09:00:50 +0200 |
commit | 32388cc3cb865653dde65b60b7ef1b606123c64a (patch) | |
tree | 060c8373062427cb3115ac005e0321c6c9b8c63c /regen.pl | |
parent | d9be4ae33dc13699f7f93e606f177f2ae43c81ab (diff) | |
download | perl-32388cc3cb865653dde65b60b7ef1b606123c64a.tar.gz |
Syntax check regen/uconfig_h.pl using t/porting/utils.t
It's the only regen script that we can't run as part of the tests (because it
requires a Unix shell), but can syntax check (because it only uses core
modules).
In theory we could make it skip with --tap if $Config{sh} is not what we
expect, but to be robust this looks to be a problem. Firstly, $Config{sh}
can be undef, or something "non-Unix". To be useful a whitelist needs to be
(at least) (?:/usr)?/bin/sh, and potentially also ksh. But the output is not
valid TAP:
$ ./perl -Ilib regen/uconfig_h.pl --tap
Extracting uconfig.h-new (with variable substitutions)
ok - regen/uconfig_h.pl uconfig.h
and t/TEST would choke, so we'd need to capture it or otherwise comment out
that "Extracting" line which just adds both complexity and fragility.
So the right trade off appears to be just to syntax check it.
Diffstat (limited to 'regen.pl')
0 files changed, 0 insertions, 0 deletions