HomePhabricator

Merge #11435: build: Make "make clean" remove all files created when running…

Description

Merge #11435: build: Make "make clean" remove all files created when running "make check"

Summary:
f35d033 build: Make "make clean" remove all files created when running "make check" (practicalswift)

Pull request description:

Make `make clean` remove all files created when running `make check`. More specifically: remove also `obj/build.h` and `bench/data/block413567.raw.h` as part of `make clean`.

Before this patch:

```bash
$ git clone https://github.com/bitcoin/bitcoin.git
$ cd bitcoin/
$ ./autogen.sh
$ ./configure
$ cp -r ../bitcoin ../bitcoin-before-make
$ make check
$ make clean
$ cp -r ../bitcoin ../bitcoin-after-make-and-make-clean
$ cd ..
$ diff -rq bitcoin-before-make/ bitcoin-after-make-and-make-clean/ | grep -E "^Only in bitcoin-after-make-and-make-clean/" | grep -v dirstamp
Only in bitcoin-after-make-and-make-clean/src/bench/data: block413567.raw.h
Only in bitcoin-after-make-and-make-clean/src/obj: build.h
$
```

After this patch:

```bash
$ git clone https://github.com/bitcoin/bitcoin.git
$ cd bitcoin/
$ ./autogen.sh
$ ./configure
$ cp -r ../bitcoin ../bitcoin-before-make
$ make check
$ make clean
$ cp -r ../bitcoin ../bitcoin-after-make-and-make-clean
$ cd ..
$ diff -rq bitcoin-before-make/ bitcoin-after-make-and-make-clean/ | grep -E "^Only in bitcoin-after-make-and-make-clean/" | grep -v dirstamp
$
```

Tree-SHA512: 953e8423485ffd415f0ade6abe0b4c407454f67c332140ef019d89db425bb4a831327b3f634b8d69b17325dcfc6e3ac72dc2ba1ce5462158eecc3c05645e93ba

Backport of Core PR11435
https://github.com/bitcoin/bitcoin/pull/11435/files

Test Plan:

# on master
make
make clean
ls src/bench/data
# block413567.raw.h
ls src/obj
# build.h

# with this patch applied
make
make clean
ls src/bench/data
# <nothing>
ls src/obj
# <nothing>

Reviewers: #bitcoin_abc, deadalnix, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D3223

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Oct 4 2017, 13:07
jasonbcoxCommitted on Jun 6 2019, 21:03
jasonbcoxPushed on Jun 6 2019, 21:03
Reviewer
Restricted Project
Differential Revision
D3223: Merge #11435: build: Make "make clean" remove all files created when running "make check"
Parents
rABC8ab8989e9d36: Merge #12877: doc: Use bitcoind in Tor documentation
Branches
Unknown
Tags
Unknown