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

Fix Travis CI nightly build #38

Merged
merged 1 commit into from
May 23, 2018
Merged

Conversation

hcpl
Copy link
Contributor

@hcpl hcpl commented May 23, 2018

That was easier than I thought: https://travis-ci.org/hcpl/derive-new/builds/382464446.

Also, would you mind if I added a Rust 1.15 build? The README makes it sound like derive-new should work in every Rust version that has stable #[derive] which means every version since 1.15.

Because both regular test and `testcrate` test create their own
`target/debug/deps/libderive_new-${hash}.so` with a unique hash,
`compiletest_rs` gets confused since it doesn't know how to tell which
one is the right one.
@nrc nrc merged commit 0a2bd16 into nrc:master May 23, 2018
@nrc
Copy link
Owner

nrc commented May 23, 2018

Nice, thanks!

Also, would you mind if I added a Rust 1.15 build?

Sure, although I'm not sure if it will actually work since I think we use some of the newer proc macro stuff as well as just custom derive, in which case I guess we should update the README.

@hcpl
Copy link
Contributor Author

hcpl commented May 23, 2018

I did check with my local 1.15 and all test pass, so I'll prepare yet another PR now.

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.

2 participants