-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add env var for specifying custom BDWGC path #108
Conversation
This is very useful for debugging Alloy.
library/boehm/build.rs
Outdated
@@ -5,8 +5,8 @@ use std::process::Command; | |||
|
|||
const BOEHM_REPO: &str = "https://github.com/softdevteam/bdwgc.git"; | |||
const BOEHM_ATOMICS_REPO: &str = "https://github.com/ivmai/libatomic_ops.git"; | |||
const BOEHM_DIR: &str = "bdwgc"; | |||
const BUILD_DIR: &str = "lib"; | |||
const BOEHM_DEFAULT_SRC_DIR: &str = "bdwgc"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we consistently call this BDWGC
(including the env var etc)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so really yeah. I'll push an update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this PR is really just a bunch of miscellaneous fixes, I've decided to go ahead and fix this across the entire codebase: 6cfff56
Do you want to squash? |
No, I think these commits should be separate. If you're ok with that, this is ready to merge. |
No description provided.