On Fri, 13 Dec 2024 04:06:47 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> As requested in [SKARA-1357](https://bugs.openjdk.org/browse/SKARA-1357), 
>> now skara bot is able to check the copyright header format.
>> Now I am going to update the jcheck configuration to configure copyright 
>> check as an error.
>
> .jcheck/conf line 39:
> 
>> 37: 
>> 38: [checks "copyright"]
>> 39: 
>> files=^(?!LICENSE|license\.txt|.*\.bin|.*\.gif|.*\.jpg|.*\.png|.*\.patch|.*\.wav|.*\.class|.*-header|.*\.jar|).*
> 
> IIUC there are other special files that could be excluded here: .icon, .tiff, 
> .dat
> 
> But, again IIUC, this is just an optimization as the check would be "if the 
> file has what appears to be an Oracle copyright, then validate the format". 
> So binary files would not have it and so would not be validated.

Yes, your understanding is correct. Additionally, Skara won't attempt to check 
binary files.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22721#discussion_r1884233125

Reply via email to