Make .gitignore ignore src/test/fuzz/fuzz.exe #21336

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:210302-ignore changing 1 files +1 −2

hebasto commented at 1:33 PM on March 2, 2021: member

No description provided.
fanquake commented at 1:34 PM on March 2, 2021: member

Shouldn't this be included by *.exe ?
hebasto commented at 1:40 PM on March 2, 2021: member

Shouldn't this be included by *.exe ?

Did not dig into the depth, but it does not work.
laanwj commented at 1:42 PM on March 2, 2021: member
Strange, as we don't mention any of the other .exe explicitly. What's different about the fuzz one?

Edit: it's this rule:
```
!src/test/fuzz/*.*
```
Why do we need that?
hebasto commented at 1:59 PM on March 2, 2021: member
Strange, as we don't mention any of the other .exe explicitly. What's different about the fuzz one?

Edit: it's this rule:

!src/test/fuzz/*.*
Right.
laanwj commented at 2:04 PM on March 2, 2021: member

Would it work to move that rule above the *..exe one?
Make .gitignore ignore src/test/fuzz/fuzz.exe 35da623574
hebasto force-pushed on Mar 2, 2021
hebasto commented at 2:19 PM on March 2, 2021: member
Edit: it's this rule:

!src/test/fuzz/*.*

Why do we need that?
Appears that we do not need it :)

Also s/src/test/fuzz/*/src/test/fuzz/fuzz/ to be explicit.
hebasto renamed this:
~~Add src/test/fuzz/fuzz.exe to .gitignore~~
Make .gitignore ignore src/test/fuzz/fuzz.exe
on Mar 2, 2021
practicalswift commented at 3:21 PM on March 2, 2021: contributor

Concept ACK
MarcoFalke commented at 6:46 PM on March 2, 2021: member

cr ACK 35da623574ccae99a4bf1ecc215311a0936a417a
practicalswift commented at 9:28 PM on March 2, 2021: contributor

cr ACK 35da623574ccae99a4bf1ecc215311a0936a417a
fanquake merged this on Mar 3, 2021
fanquake closed this on Mar 3, 2021
fanquake locked this on Mar 3, 2021
hebasto deleted the branch on Mar 3, 2021
laanwj commented at 10:08 AM on March 5, 2021: member

Posthumous ACK, this is a much better solution

Contributors