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] sudoless build on trusty #3280

Closed
wants to merge 2 commits into from
Closed

[WIP] sudoless build on trusty #3280

wants to merge 2 commits into from

Conversation

springmeyer
Copy link
Contributor

Testing branch for how our builds work on the new trusty images: https://blog.travis-ci.com/2016-11-08-trusty-container-public-beta/

@springmeyer
Copy link
Contributor Author

Currently broken with:

Disallowing packages: libstdc\+\+-5-dev, libstxxl-dev, libstxxl1, libzip-dev, lua5.1, liblua5.1-0-dev, libtbb-dev, libgdal-dev, libluabind-dev
If you require these packages, please review the package approval process at: https://github.com/travis-ci/apt-package-whitelist#package-approval-process

Starting with request for libstdc++-5 at travis-ci/apt-package-safelist#3597

@oxidase
Copy link
Contributor

oxidase commented Nov 10, 2016

@springmeyer for asan builds binutils 2.26 is required with some bug fixes for

[100%] Linking CXX executable osrm-example
/usr/bin/ld: unrecognized option '--push-state'

I did not find a better way than add ppa:jonathonf/binutils via sudo that provides 2.26 binutils for trusty. Once it will be in the whitelist than before_install.x86_64-asan.sh can be removed.

@springmeyer
Copy link
Contributor Author

@oxidase - thank you for alerting me of the sudo usage inside the scripts at https://github.com/Project-OSRM/osrm-backend/blob/master/scripts/travis/. No action needed on your part: getting support for building on sudo:false boxes is not urgent. I propose we wait for travis to add whitelist support for the basic packages and then we can revisit this PR.

@daniel-j-h
Copy link
Member

daniel-j-h commented Nov 23, 2016

Just issued a Travis whitelisting request: travis-ci/apt-source-safelist#335

@springmeyer
Copy link
Contributor Author

Just issues a Travis whitelisting request: travis-ci/apt-source-safelist#335

@daniel-j-h FWIW, binutils 2.27 is available via mason for linux: https://github.com/mapbox/mason/tree/master/scripts/binutils/2.27. Example using it for -flto: https://github.com/mapbox/vector-tile/blob/bench/.travis.yml#L48-L56

@daniel-j-h
Copy link
Member

Interesting - so what's blocking your sudoless PR here?

@springmeyer
Copy link
Contributor Author

Interesting - so what's blocking your sudoless PR here?

@daniel-j-h whitelist for libstdc++-5-dev: #3280 (comment).

Note: this PR is trying to test building sudoless on trusty. If we want to run sudoless on precise this PR would not be blocked by travis-ci/apt-package-safelist#3597.

@daniel-j-h
Copy link
Member

Closing - @TheMarex tested this in #3852

@daniel-j-h daniel-j-h closed this Mar 29, 2017
@springmeyer springmeyer deleted the sudoless branch March 29, 2017 18:14
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