| line |
stmt |
bran |
cond |
sub |
pod |
time |
code |
|
1
|
|
|
|
|
|
|
package Test::Portability::Files; |
|
2
|
|
|
|
|
|
|
$Test::Portability::Files::VERSION = '0.09'; |
|
3
|
|
|
|
|
|
|
# ABSTRACT: Check file names portability |
|
4
|
3
|
|
|
3
|
|
14497
|
use strict; |
|
|
3
|
|
|
|
|
9
|
|
|
|
3
|
|
|
|
|
103
|
|
|
5
|
3
|
|
|
3
|
|
20
|
use warnings; |
|
|
3
|
|
|
|
|
6
|
|
|
|
3
|
|
|
|
|
114
|
|
|
6
|
3
|
|
|
3
|
|
1787
|
use ExtUtils::Manifest qw(maniread); |
|
|
3
|
|
|
|
|
31182
|
|
|
|
3
|
|
|
|
|
270
|
|
|
7
|
3
|
|
|
3
|
|
29
|
use File::Basename; |
|
|
3
|
|
|
|
|
8
|
|
|
|
3
|
|
|
|
|
193
|
|
|
8
|
3
|
|
|
3
|
|
22
|
use File::Find; |
|
|
3
|
|
|
|
|
7
|
|
|
|
3
|
|
|
|
|
122
|
|
|
9
|
3
|
|
|
3
|
|
19
|
use File::Spec; |
|
|
3
|
|
|
|
|
7
|
|
|
|
3
|
|
|
|
|
53
|
|
|
10
|
3
|
|
|
3
|
|
16
|
use Test::Builder; |
|
|
3
|
|
|
|
|
7
|
|
|
|
3
|
|
|
|
|
277
|
|
|
11
|
|
|
|
|
|
|
require Exporter; |
|
12
|
|
|
|
|
|
|
|
|
13
|
|
|
|
|
|
|
our @ISA = qw(Exporter); |
|
14
|
|
|
|
|
|
|
our @EXPORT = qw(&options &run_tests); |
|
15
|
|
|
|
|
|
|
our @EXPORT_OK = @EXPORT; |
|
16
|
|
|
|
|
|
|
|
|
17
|
|
|
|
|
|
|
my $Test = Test::Builder->new; |
|
18
|
|
|
|
|
|
|
|
|
19
|
|
|
|
|
|
|
sub import { |
|
20
|
1
|
|
|
1
|
|
10
|
my $self = shift; |
|
21
|
1
|
|
|
|
|
17
|
my $caller = caller; |
|
22
|
|
|
|
|
|
|
|
|
23
|
|
|
|
|
|
|
{ |
|
24
|
|
|
|
|
|
|
## no critic |
|
25
|
3
|
|
|
3
|
|
19
|
no strict 'refs'; |
|
|
3
|
|
|
|
|
6
|
|
|
|
3
|
|
|
|
|
3864
|
|
|
|
1
|
|
|
|
|
3
|
|
|
26
|
1
|
|
|
|
|
2
|
*{ $caller . '::options' } = \&options; |
|
|
1
|
|
|
|
|
7
|
|
|
27
|
1
|
|
|
|
|
3
|
*{ $caller . '::run_tests' } = \&run_tests; |
|
|
1
|
|
|
|
|
3
|
|
|
28
|
|
|
|
|
|
|
## use critic |
|
29
|
|
|
|
|
|
|
} |
|
30
|
|
|
|
|
|
|
|
|
31
|
1
|
|
|
|
|
7
|
$Test->exported_to($caller); |
|
32
|
1
|
50
|
|
|
|
14
|
$Test->plan( tests => 1 ) unless $Test->has_plan; |
|
33
|
|
|
|
|
|
|
} |
|
34
|
|
|
|
|
|
|
|
|
35
|
|
|
|
|
|
|
my %options = ( use_file_find => 0, ); |
|
36
|
|
|
|
|
|
|
|
|
37
|
|
|
|
|
|
|
my %tests = ( |
|
38
|
|
|
|
|
|
|
ansi_chars => 1, |
|
39
|
|
|
|
|
|
|
one_dot => 1, |
|
40
|
|
|
|
|
|
|
dir_noext => 1, |
|
41
|
|
|
|
|
|
|
special_chars => 1, |
|
42
|
|
|
|
|
|
|
space => 1, |
|
43
|
|
|
|
|
|
|
mac_length => 0, |
|
44
|
|
|
|
|
|
|
amiga_length => 0, |
|
45
|
|
|
|
|
|
|
vms_length => 1, |
|
46
|
|
|
|
|
|
|
dos_length => 0, |
|
47
|
|
|
|
|
|
|
case => 1, |
|
48
|
|
|
|
|
|
|
'symlink' => 1, |
|
49
|
|
|
|
|
|
|
windows_reserved => 1, |
|
50
|
|
|
|
|
|
|
); |
|
51
|
|
|
|
|
|
|
|
|
52
|
|
|
|
|
|
|
my %errors_text = |
|
53
|
|
|
|
|
|
|
( # wrap the text at this column --------------------------------> | |
|
54
|
|
|
|
|
|
|
ansi_chars => |
|
55
|
|
|
|
|
|
|
"These files does not respect the portable filename characters\n" |
|
56
|
|
|
|
|
|
|
. "as defined by ANSI C and perlport:\n", |
|
57
|
|
|
|
|
|
|
|
|
58
|
|
|
|
|
|
|
one_dot => "These files contain more than one dot in their name:\n", |
|
59
|
|
|
|
|
|
|
|
|
60
|
|
|
|
|
|
|
dir_noext => "These directories have an extension in their name:\n", |
|
61
|
|
|
|
|
|
|
|
|
62
|
|
|
|
|
|
|
special_chars => |
|
63
|
|
|
|
|
|
|
"These files contain special characters that may break on\n" |
|
64
|
|
|
|
|
|
|
. "several systems, please correct:\n", |
|
65
|
|
|
|
|
|
|
|
|
66
|
|
|
|
|
|
|
space => "These files contain space in their name, which is not well\n" |
|
67
|
|
|
|
|
|
|
. "handled on several systems:\n", |
|
68
|
|
|
|
|
|
|
|
|
69
|
|
|
|
|
|
|
mac_length => |
|
70
|
|
|
|
|
|
|
"These files have a name more than 31 characters long, which\n" |
|
71
|
|
|
|
|
|
|
. "will be truncated on Mac OS Classic and old AmigaOS:\n", |
|
72
|
|
|
|
|
|
|
|
|
73
|
|
|
|
|
|
|
amiga_length => |
|
74
|
|
|
|
|
|
|
"These files have a name more than 107 characters long, which\n" |
|
75
|
|
|
|
|
|
|
. "will be truncated on recent AmigaOS:\n", |
|
76
|
|
|
|
|
|
|
|
|
77
|
|
|
|
|
|
|
vms_length => |
|
78
|
|
|
|
|
|
|
"These files have a name or extension too long for VMS (both\n" |
|
79
|
|
|
|
|
|
|
. "are limited to 39 characters):\n", |
|
80
|
|
|
|
|
|
|
|
|
81
|
|
|
|
|
|
|
dos_length => |
|
82
|
|
|
|
|
|
|
"These files have a name too long for MS-DOS and compatible\n" |
|
83
|
|
|
|
|
|
|
. "systems:\n", |
|
84
|
|
|
|
|
|
|
|
|
85
|
|
|
|
|
|
|
case => "The name of these files differ only by the case, which can\n" |
|
86
|
|
|
|
|
|
|
. "cause real problems on case-insensitive filesystems:", |
|
87
|
|
|
|
|
|
|
|
|
88
|
|
|
|
|
|
|
'symlink' => "The following files are symbolic links, which are not\n" |
|
89
|
|
|
|
|
|
|
. "supported on several operating systems:", |
|
90
|
|
|
|
|
|
|
); |
|
91
|
|
|
|
|
|
|
|
|
92
|
|
|
|
|
|
|
my %bad_names = (); |
|
93
|
|
|
|
|
|
|
my %lc_names = (); |
|
94
|
|
|
|
|
|
|
|
|
95
|
|
|
|
|
|
|
|
|
96
|
|
|
|
|
|
|
sub options { |
|
97
|
1
|
|
|
1
|
1
|
7
|
my %opts = @_; |
|
98
|
1
|
|
|
|
|
21
|
for my $test ( keys %tests ) { |
|
99
|
12
|
50
|
|
|
|
27
|
$tests{$test} = $opts{"test_$test"} if exists $opts{"test_$test"}; |
|
100
|
|
|
|
|
|
|
} |
|
101
|
1
|
|
|
|
|
4
|
for my $opt ( keys %options ) { |
|
102
|
1
|
50
|
|
|
|
3
|
$options{$opt} = $opts{$opt} if exists $opts{$opt}; |
|
103
|
|
|
|
|
|
|
} |
|
104
|
|
|
|
|
|
|
@tests{ keys %tests } = ( $opts{all_tests} ) x ( keys %tests ) |
|
105
|
1
|
50
|
|
|
|
9
|
if exists $opts{all_tests}; |
|
106
|
|
|
|
|
|
|
} |
|
107
|
|
|
|
|
|
|
|
|
108
|
|
|
|
|
|
|
|
|
109
|
|
|
|
|
|
|
sub test_name_portability { |
|
110
|
45
|
|
|
45
|
1
|
5209
|
my ( $file, $file_name, $file_path, $file_ext ); |
|
111
|
|
|
|
|
|
|
|
|
112
|
|
|
|
|
|
|
# extract path, base name and extension |
|
113
|
45
|
50
|
|
|
|
99
|
if ( $options{use_file_find} ) { # using Find::File |
|
114
|
|
|
|
|
|
|
# skip generated files |
|
115
|
0
|
0
|
0
|
|
|
0
|
return if $_ eq File::Spec->curdir or $_ eq 'pm_to_blib'; |
|
116
|
0
|
|
|
|
|
0
|
my $firstdir = ( |
|
117
|
|
|
|
|
|
|
File::Spec->splitdir( File::Spec->canonpath($File::Find::name) ) ) |
|
118
|
|
|
|
|
|
|
[0]; |
|
119
|
0
|
0
|
0
|
|
|
0
|
return if $firstdir eq 'blib' or $firstdir eq '_build'; |
|
120
|
|
|
|
|
|
|
|
|
121
|
0
|
|
|
|
|
0
|
$file = $File::Find::name; |
|
122
|
0
|
|
|
|
|
0
|
( $file_name, $file_path, $file_ext ) = |
|
123
|
|
|
|
|
|
|
fileparse( $file, '\\.[^.]+?' ); |
|
124
|
|
|
|
|
|
|
|
|
125
|
|
|
|
|
|
|
} |
|
126
|
|
|
|
|
|
|
else { # only check against MANIFEST |
|
127
|
45
|
|
|
|
|
78
|
$file = shift; |
|
128
|
45
|
|
|
|
|
1059
|
( $file_name, $file_path, $file_ext ) = |
|
129
|
|
|
|
|
|
|
fileparse( $file, '\\.[^.]+?' ); |
|
130
|
|
|
|
|
|
|
|
|
131
|
|
|
|
|
|
|
#for my $dir (File::Spec->splitdir(File::Spec->canonpath($file_path))) { |
|
132
|
|
|
|
|
|
|
# test_name_portability($dir) |
|
133
|
|
|
|
|
|
|
#} |
|
134
|
|
|
|
|
|
|
|
|
135
|
45
|
|
|
|
|
138
|
$_ = $file_name . $file_ext; |
|
136
|
|
|
|
|
|
|
} |
|
137
|
|
|
|
|
|
|
|
|
138
|
|
|
|
|
|
|
#print STDERR "file $file\t=> path='$file_path', name='$file_name', ext='$file_ext'\n"; |
|
139
|
|
|
|
|
|
|
|
|
140
|
|
|
|
|
|
|
# After this point, the following variables are expected to hold these semantics |
|
141
|
|
|
|
|
|
|
# $file must contain the path to the file (t/00load.t) |
|
142
|
|
|
|
|
|
|
# $_ must contain the full name of the file (00load.t) |
|
143
|
|
|
|
|
|
|
# $file_name must contain the base name of the file (00load) |
|
144
|
|
|
|
|
|
|
# $file_path must contain the path to the directory containing the file (t/) |
|
145
|
|
|
|
|
|
|
# $file_ext must contain the extension (if any) of the file (.t) |
|
146
|
|
|
|
|
|
|
|
|
147
|
|
|
|
|
|
|
# check if the name only uses portable filename characters, as defined by ANSI C |
|
148
|
45
|
50
|
|
|
|
115
|
if ( $tests{ansi_chars} ) { |
|
149
|
45
|
100
|
|
|
|
171
|
/^[A-Za-z0-9._][A-Za-z0-9._-]*$/ or $bad_names{$file} .= 'ansi_chars,'; |
|
150
|
|
|
|
|
|
|
} |
|
151
|
|
|
|
|
|
|
|
|
152
|
|
|
|
|
|
|
# check if the name is a Windows Reserved Filename |
|
153
|
45
|
50
|
|
|
|
95
|
if ( $tests{'windows_reserved'} ) { |
|
154
|
|
|
|
|
|
|
$file_name =~ /^(CON|PRN|AUX|NUL|COM[1-9]|LPT[1-9])$/i |
|
155
|
45
|
50
|
|
|
|
148
|
and $bad_names{$file} .= 'windows_reserved,'; |
|
156
|
|
|
|
|
|
|
} |
|
157
|
|
|
|
|
|
|
|
|
158
|
|
|
|
|
|
|
# check if the name contains more than one dot |
|
159
|
45
|
50
|
|
|
|
93
|
if ( $tests{one_dot} ) { |
|
160
|
45
|
100
|
|
|
|
115
|
tr/.// > 1 and $bad_names{$file} .= 'one_dot,'; |
|
161
|
|
|
|
|
|
|
} |
|
162
|
|
|
|
|
|
|
|
|
163
|
|
|
|
|
|
|
# check if the name contains special chars |
|
164
|
45
|
50
|
|
|
|
96
|
if ( $tests{special_chars} ) { |
|
165
|
|
|
|
|
|
|
m-[!"#\$%&'\(\)\*\+/:;<>\?@\[\\\]^`\{\|\}~]- |
|
166
|
45
|
100
|
|
|
|
102
|
and $bad_names{$file} .= 'special_chars,'; |
|
167
|
|
|
|
|
|
|
} |
|
168
|
|
|
|
|
|
|
|
|
169
|
|
|
|
|
|
|
# check if the name contains a space char |
|
170
|
45
|
50
|
|
|
|
87
|
if ( $tests{space} ) { |
|
171
|
45
|
100
|
|
|
|
98
|
m/ / and $bad_names{$file} .= 'space,'; |
|
172
|
|
|
|
|
|
|
} |
|
173
|
|
|
|
|
|
|
|
|
174
|
|
|
|
|
|
|
# check the length of the name, compared to Mac OS Classic max length |
|
175
|
45
|
100
|
|
|
|
83
|
if ( $tests{mac_length} ) { |
|
176
|
14
|
100
|
|
|
|
41
|
length > 31 and $bad_names{$file} .= 'mac_length,'; |
|
177
|
|
|
|
|
|
|
} |
|
178
|
|
|
|
|
|
|
|
|
179
|
|
|
|
|
|
|
# check the length of the name, compared to AmigaOS max length |
|
180
|
45
|
100
|
|
|
|
83
|
if ( $tests{amiga_length} ) { |
|
181
|
14
|
100
|
|
|
|
30
|
length > 107 and $bad_names{$file} .= 'amiga_length,'; |
|
182
|
|
|
|
|
|
|
} |
|
183
|
|
|
|
|
|
|
|
|
184
|
|
|
|
|
|
|
# check the length of the name, compared to VMS max length |
|
185
|
45
|
50
|
|
|
|
82
|
if ( $tests{vms_length} ) { |
|
186
|
|
|
|
|
|
|
( length($file_name) <= 39 and length($file_ext) <= 40 ) |
|
187
|
45
|
100
|
100
|
|
|
168
|
or $bad_names{$file} .= 'vms_length,'; |
|
188
|
|
|
|
|
|
|
} |
|
189
|
|
|
|
|
|
|
|
|
190
|
|
|
|
|
|
|
# check the length of the name, compared to DOS max length |
|
191
|
45
|
100
|
|
|
|
84
|
if ( $tests{dos_length} ) { |
|
192
|
|
|
|
|
|
|
( length($file_name) <= 8 and length($file_ext) <= 4 ) |
|
193
|
14
|
100
|
100
|
|
|
52
|
or $bad_names{$file} .= 'dos_length,'; |
|
194
|
|
|
|
|
|
|
} |
|
195
|
|
|
|
|
|
|
|
|
196
|
|
|
|
|
|
|
# check if the name is unique on case-insensitive filesystems |
|
197
|
45
|
50
|
|
|
|
86
|
if ( $tests{case} ) { |
|
198
|
45
|
100
|
66
|
|
|
172
|
if ( not $lc_names{$file} and $lc_names{ lc $file } ) { |
|
199
|
1
|
|
|
|
|
4
|
$bad_names{$file} .= 'case,'; |
|
200
|
1
|
|
|
|
|
4
|
$bad_names{ $lc_names{ lc $file } } .= 'case,'; |
|
201
|
|
|
|
|
|
|
} |
|
202
|
|
|
|
|
|
|
else { |
|
203
|
44
|
|
|
|
|
107
|
$lc_names{ lc $file } = $file; |
|
204
|
|
|
|
|
|
|
} |
|
205
|
|
|
|
|
|
|
} |
|
206
|
|
|
|
|
|
|
|
|
207
|
|
|
|
|
|
|
# check if the file is a symbolic link |
|
208
|
45
|
50
|
|
|
|
89
|
if ( $tests{'symlink'} ) { |
|
209
|
45
|
50
|
|
|
|
688
|
-l $file and $bad_names{$file} .= 'symlink,'; |
|
210
|
|
|
|
|
|
|
} |
|
211
|
|
|
|
|
|
|
|
|
212
|
|
|
|
|
|
|
# if it's a directory, check that it has no extension |
|
213
|
45
|
50
|
|
|
|
163
|
if ( $tests{'dir_noext'} ) { |
|
214
|
45
|
100
|
66
|
|
|
630
|
-d $file and tr/.// > 0 and $bad_names{$file} .= 'dir_noext,'; |
|
215
|
|
|
|
|
|
|
} |
|
216
|
|
|
|
|
|
|
} |
|
217
|
|
|
|
|
|
|
|
|
218
|
|
|
|
|
|
|
|
|
219
|
|
|
|
|
|
|
sub run_tests { |
|
220
|
1
|
|
|
1
|
1
|
999
|
fileparse_set_fstype('Unix'); |
|
221
|
|
|
|
|
|
|
|
|
222
|
1
|
50
|
|
|
|
6
|
if ( $options{use_file_find} ) { |
|
223
|
|
|
|
|
|
|
|
|
224
|
|
|
|
|
|
|
# check all files found using File::Find |
|
225
|
0
|
|
|
|
|
0
|
find( \&test_name_portability, File::Spec->curdir ); |
|
226
|
|
|
|
|
|
|
|
|
227
|
|
|
|
|
|
|
} |
|
228
|
|
|
|
|
|
|
else { |
|
229
|
|
|
|
|
|
|
# check only against files listed in MANIFEST |
|
230
|
1
|
|
|
|
|
4
|
my $manifest = maniread(); |
|
231
|
1
|
|
|
|
|
525
|
map { test_name_portability($_) } keys %$manifest; |
|
|
31
|
|
|
|
|
89
|
|
|
232
|
|
|
|
|
|
|
} |
|
233
|
|
|
|
|
|
|
|
|
234
|
|
|
|
|
|
|
# check the results |
|
235
|
1
|
|
|
|
|
6
|
my $bad_names = _bad_names(); |
|
236
|
1
|
50
|
|
|
|
5
|
if ( keys %$bad_names ) { |
|
237
|
0
|
|
|
|
|
0
|
$Test->ok( 0, "File names portability" ); |
|
238
|
|
|
|
|
|
|
|
|
239
|
0
|
|
|
|
|
0
|
my %errors_list = (); |
|
240
|
0
|
|
|
|
|
0
|
for my $file ( keys %$bad_names ) { |
|
241
|
0
|
|
|
|
|
0
|
for my $error ( split ',', $bad_names->{$file} ) { |
|
242
|
0
|
0
|
|
|
|
0
|
$errors_list{$error} = [] if not ref $errors_list{$error}; |
|
243
|
0
|
|
|
|
|
0
|
push @{ $errors_list{$error} }, $file; |
|
|
0
|
|
|
|
|
0
|
|
|
244
|
|
|
|
|
|
|
} |
|
245
|
|
|
|
|
|
|
} |
|
246
|
|
|
|
|
|
|
|
|
247
|
0
|
|
|
|
|
0
|
for my $error ( sort keys %errors_list ) { |
|
248
|
0
|
|
|
|
|
0
|
$Test->diag( $errors_text{$error} ); |
|
249
|
|
|
|
|
|
|
|
|
250
|
0
|
|
|
|
|
0
|
for my $file ( sort @{ $errors_list{$error} } ) { |
|
|
0
|
|
|
|
|
0
|
|
|
251
|
0
|
|
|
|
|
0
|
$Test->diag(" $file"); |
|
252
|
|
|
|
|
|
|
} |
|
253
|
|
|
|
|
|
|
|
|
254
|
0
|
|
|
|
|
0
|
$Test->diag(' '); |
|
255
|
|
|
|
|
|
|
} |
|
256
|
|
|
|
|
|
|
} |
|
257
|
|
|
|
|
|
|
else { |
|
258
|
1
|
|
|
|
|
7
|
$Test->ok( 1, "File names portability" ); |
|
259
|
|
|
|
|
|
|
} |
|
260
|
|
|
|
|
|
|
} |
|
261
|
|
|
|
|
|
|
|
|
262
|
|
|
|
|
|
|
|
|
263
|
|
|
|
|
|
|
sub _bad_names { |
|
264
|
15
|
|
|
15
|
|
78
|
return \%bad_names; |
|
265
|
|
|
|
|
|
|
} |
|
266
|
|
|
|
|
|
|
|
|
267
|
|
|
|
|
|
|
|
|
268
|
|
|
|
|
|
|
1; |
|
269
|
|
|
|
|
|
|
|
|
270
|
|
|
|
|
|
|
__END__ |