On Sun, 10 Nov 2024, Jakub Jelinek wrote: > On Sun, Nov 10, 2024 at 01:30:06PM -0300, Alexandre Oliva wrote: > > On Nov 9, 2024, Jakub Jelinek <ja...@redhat.com> wrote: > > > > > r15-4998 and r15-5004 had wrong commit message, add those to > > > ignored commits. > > > > Ugh, sorry and thanks. > > Was that .c vs .cc only, or was there anything else? > > I think so. > > > I'm surprised the commit-time checker didn't catch them. > > I'm surprised too, but don't want to try to push further broken commits just > to double check that. ;)
I think this is the known loophole where branch-specific checks only get applied to commits that are new to the repository, not to commits that were previously pushed to a branch with laxer checks. This has been fixed - see https://github.com/AdaCore/git-hooks/issues/20 - but we need to update the copy of the hooks used by GCC (there are multiple copies on sourceware, I think), which means checking for any incompatible changes since the version currently in use that might require updates to the hook configuration. (And then we'd need to use the new facilities introduced by the fix for that issue to ensure the branch-specific checks are applied to all commits added to relevant branches, not just ones new to the repository; it's not automatic even with updated hooks.) -- Joseph S. Myers josmy...@redhat.com
