build2-packaging / google-benchmark

Build2 package for Google-Benchmark
Other
0 stars 1 forks source link

Fix nondeterministic target setup order #6

Closed helmesjo closed 2 years ago

helmesjo commented 2 years ago

It was noticed that occasionally (or always for GCC?), the order of target setup (that is, targets include/ & src/ respectively) were not setup correctly. For GCC a patched version is required, which is generated by the include/ target. This must happen before setting up/building the /src target but this was not the case.

This PR just makes it explicit in what order things are setup. Verified multiple times switching back and forth between master & this branch (and bdep deinit @gcc && bdep init @gcc inbetween).

Tested with GCC 10, GCC 11 & Clang 13.