Page MenuHomePhabricator

Add src/test/ to .gitignore

Authored by sickpig on May 24 2018, 12:05.


Group Reviewers
Restricted Project

see title

Test Plan

try to git-add src/test/ . the command should add rise an error along with a message similar to this one:
The following paths are ignored by one of your .gitignore files:
Use -f if you really want to add them.

Diff Detail

rABC Bitcoin ABC
gitignore (branched from master)
Lint OK
No Unit Test Coverage
Build Status
Buildable 2635
Build 3384: Bitcoin ABC Buildbot (legacy)
Build 3383: arc lint + arc unit

Event Timeline

sickpig created this revision.May 24 2018, 12:05
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 24 2018, 12:05

What is this file, why do we want to ignore it ?

sickpig updated this revision to Diff 3963.May 24 2018, 13:43

Format comment

sickpig added a comment.May 24 2018, 13:47

last change wasn't supposed to update this diff.. fixing

sickpig updated this revision to Diff 3965.May 24 2018, 13:54

remove a spurios commit

sickpig added a comment.May 24 2018, 14:25

What is this file, why do we want to ignore it ?

It is generated by make check and it's an helper file that contains the extension to be used for binaries aiding the execution of the unit tests. When you run on Linux it's content is something like:


whereas when you're cross compiling for win32/64:


I've never seen this file. My build generates test/config.ini which is read by the test harness. I'm fairly sure this is legacy -- we backported a commit a long time ago related to this.

sickpig abandoned this revision.May 25 2018, 07:22

​src/test/ has been generated by an old in-tree build from late 2017, it is not used anymore. sorry for the noise.