Wireshark · Wireshark-dev: Re: [Wireshark-dev] checkhf.pl fails only on pre-commit hook

[Hadriel Kaplan <hadriel.kaplan@xxxxxxxxxx>]

On Apr 4, 2014, at 11:26 PM, Hadriel Kaplan <hadriel.kaplan@xxxxxxxxxx> wrote:

> I barely know perl, but checkhf.pl line 564 does indeed look bogus to me. (the double slashes are what it thinks is starting a regex search pattern that's not terminated I believe)

Actually, no it's fine. I don't know why your computer's choking on it.

I even tried running it with the debug option for all the valid numbers and it's working fine for me.

Weird.


> Having said that, that function shouldn't be being called/invoked, afaict.  It's only supposed to be invoked if the command to run that script uses a "--debug=<number>" or "-d <number>" option I think. And the pre-commit script does not do that. Unless $FILE is somehow a string that appears to be that option?  Try putting a 'echo $FILE' in the pre-commit script before the "./tools/checkhf.pl $FILE" line and see what it's printing out when you do 'git commit'.  Maybe something weird is going on with the 'git diff-index ... | grep ...' result?
> 
> -hadriel
> 
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe