bazel - needs to show git describe in splash
Description
The CMake generates a useful version for use to use, which the bazel build should do as well. This allows for easy debug and if the log files are a concern it can already be disabled with -no_splash
Suggested Solution
No response
Additional Context
No response
I disagree with this feature request: this breaks bazel hashes and would cause a re-run of all tests when it is not necessary, for instance, changing a git comment or modifying a comment in the OpenROAD source code.
Changing a comment in the source code does not change the commit id unless you commit the change.
I think this is a useful feature. Knowing what version of the tool generated some result is often helpful.
Changing a comment in the source code does not change the commit id unless you commit the change.
Yes: I want to avoid rebuilds/retests in this case, such as a no-op merge commit.
I think this is a useful feature. Knowing what version of the tool generated some result is often helpful.
Yes, but it needs to be done in a different way.
I am thinking a build.txt file that is only read in runtime so that bazel does not consider it a dependency.
This has to be a solved problem.
@hzeller Any experience on how to implement this without the downsides?
You do it with the build tagging system --stamp I mentioned earlier. You just run --nostamp in everything except a special --config=release mode.
Given that we don't do formal releases I'm not sure how helpful that is. If the CI has stamping on then developers will see no cache benefit. If the CI has it off then likely nobody will ever turn it on.
Yes there are ways and I can take care of it when I have a minute (but next two weeks: no minutes left)
@QuantamHD @hzeller do you have time to address this?