#
7d91d5ba |
| 21-Jul-2024 |
Dimitri Papadopoulos <3234522+DimitriPapadopoulos@users.noreply.github.com> |
Fix typos found by codespell Reviewed-by: Kurt Roeckx <kurt@roeckx.be> Reviewed-by: Tim Hudson <tjh@openssl.org> (Merged from https://github.com/openssl/openssl/pull/24949)
|
#
7821b7b9 |
| 16-Jul-2024 |
Richard Levitte |
fix: util/check-format-commit.sh to handle one-line diff hunks For multi-line hunks, 'git diff -U0' outputs a pair of START,COUNT indicators to show where the hunk starts and ends. Howe
fix: util/check-format-commit.sh to handle one-line diff hunks For multi-line hunks, 'git diff -U0' outputs a pair of START,COUNT indicators to show where the hunk starts and ends. However, if the hunk is just one line, only START is output, with the COUNT of 1 being implied. Typically, this happens for copyright change hunks, like this: --- a/crypto/evp/evp_err.c +++ b/crypto/evp/evp_err.c @@ -3 +3 @@ - * Copyright 1995-2023 The OpenSSL Project Authors. All Rights Reserved. + * Copyright 1995-2024 The OpenSSL Project Authors. All Rights Reserved. This is normal unified diff output, and our script must adapt. Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> Reviewed-by: Neil Horman <nhorman@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/24900)
show more ...
|
#
42230f29 |
| 12-Jul-2024 |
Neil Horman |
Convert check-format-commits.sh to use allowlist Initially check-format-commits.sh tried to check everything, using a banlist to exlude files not appropriate for checking. Its b
Convert check-format-commits.sh to use allowlist Initially check-format-commits.sh tried to check everything, using a banlist to exlude files not appropriate for checking. Its becoming clear that that approach isn't workable, given that the number of files that we should not check far outweighs the number of files that we should check. Ideally we should be checking .c files, .h files and their .in counterparts, everything else should be excluded (at least for now) convert the script to using an allowlist, only checking the above list, and ignoring everything else Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Merged from https://github.com/openssl/openssl/pull/24865)
show more ...
|
#
c2083f4e |
| 11-Jul-2024 |
Richard Levitte |
feat: enhance util/check-format-commit.sh to be able to handle a commit range Additionally, the 'git diff' call is modified to not show context lines, as it's confusing to have style nit
feat: enhance util/check-format-commit.sh to be able to handle a commit range Additionally, the 'git diff' call is modified to not show context lines, as it's confusing to have style nits displayed on lines the author of the commits hasn't touched. Reviewed-by: Neil Horman <nhorman@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from https://github.com/openssl/openssl/pull/24856)
show more ...
|
#
861e1f45 |
| 10-Jul-2024 |
Neil Horman |
ignore various files in commit checker Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Paul Dale <ppzgs1@gmail.com> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Mer
ignore various files in commit checker Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Paul Dale <ppzgs1@gmail.com> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Merged from https://github.com/openssl/openssl/pull/24845)
show more ...
|
#
acae12eb |
| 08-Jul-2024 |
Neil Horman |
Add a check-format-commit.sh script Add a wrapper script to check-format.pl, which is capable of analyzing commits rather than just a file. for a provided commit this script: 1
Add a check-format-commit.sh script Add a wrapper script to check-format.pl, which is capable of analyzing commits rather than just a file. for a provided commit this script: 1) runs check-format.pl on the files changed in the provided commit 2) filters the output of check-format.pl, only producing lines that match ranges of changed lines in those files Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Merged from https://github.com/openssl/openssl/pull/24806)
show more ...
|