-
Notifications
You must be signed in to change notification settings - Fork 32
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
the compile rule should be smarter about when it can avoid rebuilds. #28
Comments
Is that behavior wrong? I'm imagining that changes to libkj would dirty the capnp compiler binary, and then any generated code output by the capnp compiler is dirtied (since it was one of the inputs to that rule), and then anything that depends on that generated code is also dirtied, which covers the rest of the stuff in capnp and sandstorm. |
Ah, perhaps you're right; I hadn't been thinking about it too carefully.
Quoting Drew Fisher (2020-02-20 03:37:14)
… Is that behavior wrong? I'm imagining that changes to libkj would dirty
the capnp compiler binary, and then any generated code output by the
capnp compiler is dirtied (since it was one of the inputs to that
rule), and then anything that depends on that generated code is also
dirtied, which covers the rest of the stuff in capnp and sandstorm.
--
You are receiving this because you authored the thread.
Reply to this email directly, [1]view it on GitHub, or [2]unsubscribe.
Verweise
1. #28?email_source=notifications&email_token=AAGXYPQEX474KLPL2AY4T3DRDY6LVA5CNFSM4KYJHNDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEML6P4Q#issuecomment-588769266
2. https://github.com/notifications/unsubscribe-auth/AAGXYPWJFQJEUTY5K3WOMU3RDY6LVANCNFSM4KYJHNDA
|
Yes, this is probably what's going on. It is admittedly very annoying. Maybe A better solution would be for |
In particular, right now I'm doing some print debugging and tracing an issue I'm having in the bridge down into stuff defined in libkj. Adding a simple print statement to a .c++ file should only require re-building the one file, and re-linking anything that uses the resulting object. Instead, ekam is re-compiling a whole bunch of stuff downstream in capnp and in sandstorm.
The text was updated successfully, but these errors were encountered: