Skip to content
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

Always rebuilding #2

Closed
est31 opened this issue May 29, 2019 · 4 comments
Closed

Always rebuilding #2

est31 opened this issue May 29, 2019 · 4 comments
Assignees

Comments

@est31
Copy link

est31 commented May 29, 2019

For some reason, when I turn on bundling, it's always rebuilding the entire library for me. There is no caching involved. That's obviously bad.

@onelson
Copy link
Owner

onelson commented May 29, 2019

Yeah. I'm betting this is also somehow related to the outstanding issue where if you build with or without the bundled feature, you have to cargo clean between switching. The build.rs needs some work.

@onelson onelson self-assigned this Jun 1, 2019
@onelson
Copy link
Owner

onelson commented Jun 1, 2019

So, I have been looking at this from a couple different ways. First and foremost, it looks like I had a series of cargo:rerun-if-s in the build script, originally designed to help avoid redundant rebuilds. If I remove them all, things seem to cache mostly the way we'd expect. I think maybe those instructions might have made sense when I had been depending on a jq-src crate, but no longer. Leveraging a git submodule just felt a lot more straightforward than having a 3rd crate in the mix.

The issue I mentioned above where alternating between a non-bundled build and a bundled build can break the build script still persists (#3). This mystery will likely be solved another day. Still, I think just having the cache working properly when not alternating the feature is a big enough win for now.

If you are watching the clock - I'd expect to cut releases for this and onelson/jq-rs#1 sometime this weekend. I'll plan to spend a little more time working towards #3, but I won't wait on it if it doesn't seem obvious with a little extra research.

@onelson
Copy link
Owner

onelson commented Jun 2, 2019

I think this should be fixed in v0.2.1.

@onelson onelson closed this as completed Jun 2, 2019
@est31
Copy link
Author

est31 commented Jun 2, 2019

Thanks for looking into it @onelson !

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

No branches or pull requests

2 participants