Skip to content

Conversation

maflcko
Copy link
Contributor

@maflcko maflcko commented May 26, 2023

(See commit description for steps I used)

This should not increase the .git folder size, because all inputs already exist in .git. For the same reason it should not increase coverage.

MarcoFalke added 5 commits May 26, 2023 11:11
Command:

FUZZ=process_message ./src/test/fuzz/fuzz -merge=1 ../btc_qa_assets/fuzz_seed_corpus/process_message ../btc_qa_assets/fuzz_seed_corpus/process_message*
…le=1)

FUZZ=process_message ./src/test/fuzz/fuzz -merge=1 -use_value_profile=1 ../btc_qa_assets/fuzz_seed_corpus/process_message ../btc_qa_assets/fuzz_seed_corpus/process_message*
(--with-sanitizers=fuzzer,undefined,integer,address and
-use_value_profile=1)
@maflcko
Copy link
Contributor Author

maflcko commented May 26, 2023

This is done in preparation to delete the specific dirs, but can be done even if the dirs are not deleted.

Copy link
Member

@dergoegge dergoegge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 14913e0

@fanquake fanquake merged commit fda67be into bitcoin-core:main May 28, 2023
@maflcko
Copy link
Contributor Author

maflcko commented Jul 13, 2023

On a second thought, this change doesn't make sense. Instead of duplicating/copying the files it should have moved them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants