-
-
Notifications
You must be signed in to change notification settings - Fork 181
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
fails on travis with ASLR disable failed: EPERM #77
Comments
Closing because it seems like a issue of travis. |
Possible workaround is running with |
12: Use sudo inside travis r=kdy1 a=kdy1 See xd009642/tarpaulin#77 (comment)
Oh.. that worked. Thank you. |
Added `sudo: required` following #77
@philipc thanks for the help, I'll make an issue for looking into using findshlibs to improve tarpaulin. Any input you might have would be appreciated 😄 I've also added sudo:required to the example travis.yml in the README so new users don't hit this problem as well. |
@xd009642, I setup cargo-tarpaulin using the README on crates.io and ran into this issue since the crates.io README doesn't yet have this change. Please consider bumping the version to update the crates.io README. BTW, I discovered cargo-tarpaulin from your recent reddit post. I was looking for a coverage solution and was turned off by the complex setup of kcov. Cargo-tarpaulin is so much slicker! |
Thanks for the compliment 😄 I did consider releasing a patch so crates.io is up-to-date but latest stable is out Feb 4th iirc and I'm hoping to actually have some more features to merge in. And that way I don't have to do another release to get rid of the warning about stable Rust! |
Seeing this issue on travis as well https://travis-ci.org/Marwes/combine/jobs/356101326#L946 I believe a fix for this hasn't been released(?) |
@rfdonnelly Yeah I saw that but #77 (comment) and #77 (comment) seems to indicate that there is a solution that does not require sudo. (Going to add |
@Marwes My understanding is that there is no solution for Travis that does not require sudo.
|
Agree with @rfdonnelly sudo is required sorry about that |
This is unfortunate, trying to use tarpaulin in my Jenkins environment, cargo does not play nice with sudo |
This solves an issue when running code coverage: xd009642/tarpaulin#77
I encountered the same error. |
@new-commer You shouldn't be using sudo to run tarpaulin. The workaround is to put |
@philipc Thanks for reply . I'm not use |
travis is like gitlab ci, so the problem is the same, but the fix will be different. The problem is that containers (such as docker) normally have ASLR enabled, but tarpaulin can't work with ASLR. You'll need to figure out either how to run tarpaulin without docker, or how to configure docker to allow tarpaulin to disable ASLR (maybe there's a seccomp setting for this, I'm not familiar with it though). |
Just a note that CircleCI fails the same way: https://circleci.com/gh/mgeisler/textwrap/58. It has worked very well for me until recently. I've raised the issue in their discussion forum: https://discuss.circleci.com/t/cargo-tarpaulin-fails/30215. Perhaps they'll be able to fix this on their side, which would give us an easy way to run Tarpaulin again. |
Workaround on CircleCI is to use An example of this can be found here: https://github.com/bittrd/rust-hello-world/blob/f8840e431009843d7bee652265230cc3fa0f3002/.circleci/config.yml#L38 |
@rdaniels6813 oh thanks for that I'll add a note to the README 👍 |
I ended up using the following CircleCI configuration for textwrap:
This way only the |
Travis log: https://travis-ci.org/swc-project/swc/builds/328637807#L2967
I use nightly
The text was updated successfully, but these errors were encountered: