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

Default to little endian on windows #110

Merged
merged 1 commit into from
Jan 21, 2016
Merged

Conversation

Orvid
Copy link
Contributor

@Orvid Orvid commented Aug 4, 2015

Because MSVC doesn't provide any way for us to detect which we are actually on.

Because MSVC doesn't provide any way for us to detect which we are actually on.
juchem added a commit that referenced this pull request Jan 21, 2016
Default to little endian on windows
@juchem juchem merged commit 422b34a into facebook:master Jan 21, 2016
juchem pushed a commit that referenced this pull request Jan 22, 2016
This reverts commit 422b34a, reversing
changes made to a73eae1.
ghost pushed a commit that referenced this pull request Feb 2, 2016
Summary:
Because MSVC doesn't provide any way for us to detect which we are actually on.
Closes #110

Reviewed By: juchem, yfeldblum

Differential Revision: D2853371

Pulled By: Orvid

fb-gh-sync-id: edfde892bdef573590a6c10c1728ccf7417d2c4a
facebook-github-bot pushed a commit that referenced this pull request Feb 16, 2022
Summary:
Pull Request resolved: facebook/sapling#110

Pull Request resolved: facebookexperimental/rust-shed#27

Make it so that changes to rust-shed or other common rust source are used locally vendored, so they don't need to be pushed to github before they are visible in a build.

There was already some support for cargo vendoring in getdeps, but it was limited to dependencies between manifests built with cargo builder.  This wasn't enough to build something like eden (cmake is main entry point, with later calls cargo) or eden_scm (make is main entry point, with later calls to cargo), so this diff adds a cargo prepare step for getdeps other primary build systems.

The cargo vendoring is done by using a cargo config file to point to the source files used by getdeps.  It has two modes:

1. per crate, existing mode which is already automatic for cargo to cargo manifest dependencies.  To use it for a non cargo build manifest, add crate.pathmap
2. per git url, existing mode which was only use for crates.io third-party crates, now can be enabled by setting cargo.cargo_config_file

Reviewed By: yancouto

Differential Revision: D33895469

fbshipit-source-id: 7b13c0b679532492a336ce217de875c25fe1be90
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants