1 |
>>>>> On Tue, 21 Nov 2017, Michał Górny wrote: |
2 |
|
3 |
>> > It is an error for a single file to be matched by multiple entries |
4 |
>> > of different semantics, file size or checksum values. It is an error |
5 |
>> > to specify another entry for a file matching ``IGNORE``, or one of its |
6 |
>> > subdirectories. |
7 |
>> |
8 |
>> What about regular files in a directory (or subdirectory) matched |
9 |
>> by IGNORE? Looks like this case is not covered (?). |
10 |
|
11 |
> Ignored regular files must not have any other (e.g. DATA) entries. |
12 |
> Otherwise the expected behavior is unclear -- are we supposed to |
13 |
> verify the file or ignore it? |
14 |
|
15 |
I still believe that the wording doesn't convey that. Maybe an example |
16 |
will clarify what I mean. |
17 |
|
18 |
There is a directory foo/bar and a regular file foo/bar/quux in it. |
19 |
Now in Manifest there are these entries: |
20 |
|
21 |
IGNORE foo/bar |
22 |
DATA foo/bar/quux <size> <checksums> |
23 |
|
24 |
The spec says: "It is an error to specify another entry for a file |
25 |
matching ``IGNORE``, or one of its subdirectories." However, file |
26 |
foo/bar/quux neither matches IGNORE nor is a subdirectory of it. |
27 |
|
28 |
Ulrich |