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

[WIP] Build with meson #89

Closed
wants to merge 5 commits into from
Closed

[WIP] Build with meson #89

wants to merge 5 commits into from

Conversation

strega-nil
Copy link
Contributor

@strega-nil strega-nil commented Oct 21, 2018

Since I'm a VS2017 user, I also made sure to get it compiling with VS2017.

This is just an initial pass. I think it's a useful comparison with #88, as I think meson is far simpler and easy to read compared to CMake.

@strega-nil
Copy link
Contributor Author

strega-nil commented Oct 22, 2018

You'll also notice that the git versions will update every build, as opposed to every run of CMake, like in #88 (aiui).

How it's written right now is also quite fast - it only requires rebuilding the one file, not every single file where the defines have changed.

I'm probably not going to do the downloading and building of rustc
unless @thepowersgang gives the go-ahead, but here's how it might start
to look
@thepowersgang
Copy link
Owner

Could you separate the source changes out from the new build system?

@strega-nil
Copy link
Contributor Author

Sure, I just wanted to make sure it built.

@thepowersgang
Copy link
Owner

I'm closing this PR in favour of using CMake (sorry).
However, I'd appreciate the VS2017 fixes in a separate PR.

@strega-nil
Copy link
Contributor Author

np! I'll see if I can do the thing.

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

Successfully merging this pull request may close these issues.

None yet

3 participants