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

switch to trusty build in travis ci and fix some test cases #1155

Merged
merged 1 commit into from May 25, 2018

Conversation

Projects
None yet
2 participants
@bruce30262
Contributor

bruce30262 commented May 25, 2018

For #1133 , #1021 and #1022 .

This PR including switching travis build to trusty ( Ubuntu 14.04 ) and some fixes to make the build pass all the test cases. The build works on the latest stable branch ( build log )

Note that this also works on the latest dev branch, for details check out my fix-travis branch( which base on the dev branch) and its build log

Hope this fix will let us make a big step forward to the release of pwntools 3.13 :)

@zachriggle

This comment has been minimized.

Contributor

zachriggle commented May 25, 2018

Ignoring Codacy / Coveralls checks to merge CI fixes.

@zachriggle zachriggle merged commit 5ae13e8 into Gallopsled:stable May 25, 2018

1 of 3 checks passed

Codacy/PR Quality Review Codacy was unable to analyse your pull request.
Details
coverage/coveralls Coverage pending from Coveralls.io
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment