-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: something is off with EXECGEN_TARGETS and logictest-bins #49676
Labels
A-build-system
C-bug
Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Comments
yuzefovich
added
C-bug
Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
A-build-system
labels
May 29, 2020
Thanks for the repro steps, @yuzefovich. I'm taking a look. |
craig bot
pushed a commit
that referenced
this issue
May 29, 2020
49696: Makefile: fix handling of non-existent *.eg.go files r=petermattis a=petermattis When a `bin/*.d` file references an `*.eg.go` file, that file was required to exist or be buildable by the Makefile. Any commit which removed a `*.eg.go` file would violate this requirement causing the build to fail until the offending `bin/*.d` file was removed. In order to prevent this badness, a catchall `%.eg.go` rule is added which will force the target dependent on the `bin/%.d` file to be rebuilt. Fixes #49676 Release note: None Co-authored-by: Peter Mattis <petermattis@gmail.com>
jbowens
pushed a commit
to jbowens/cockroach
that referenced
this issue
Jun 1, 2020
When a `bin/*.d` file references an `*.eg.go` file, that file was required to exist or be buildable by the Makefile. Any commit which removed a `*.eg.go` file would violate this requirement causing the build to fail until the offending `bin/*.d` file was removed. In order to prevent this badness, a catchall `%.eg.go` rule is added which will force the target dependent on the `bin/%.d` file to be rebuilt. Fixes cockroachdb#49676 Release note: None
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-build-system
C-bug
Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Here is a repro of some weird behavior of logic test binary:
The failure is
Note that the commit in question introduced these two
EXECGEN_TARGETS
and removed another one (namely, it removedhashtable.eg.go
and addedhashtable_full.eg.go
andhashtable_distinct.eg.go
).The text was updated successfully, but these errors were encountered: