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

TravisCI improvements #2460

Merged
merged 5 commits into from Mar 4, 2017
Merged

TravisCI improvements #2460

merged 5 commits into from Mar 4, 2017

Conversation

Chocobo1
Copy link
Contributor

@Chocobo1 Chocobo1 commented Mar 3, 2017

  • cleanup indentation
    2 spaces for indentation
  • make jobs drop to 3
    TravisCI allocates 2 cpus per build instance
  • move package installation to apt addon
  • make pip install to user directory
    Also disable sudo for faster bootup

And the "disable sudo" causes the ccache need to be regenerated.

@mention-bot
Copy link

@Chocobo1, thanks for your PR! By analyzing the history of the files in this pull request, we identified @danilaml, @AniLeo and @Bigpet to be potential reviewers.

@Chocobo1
Copy link
Contributor Author

Chocobo1 commented Mar 3, 2017

Also I noticed this project is using coveralls, but it seems it wasn't configured/hooked-up correctly...

.travis.yml Outdated
- if [ "$TRAVIS_OS_NAME" = "linux" ]; then
sudo apt-get install -y libwxgtk3.0-dev;
fi;
# shutdown services on Travis, which may have a memory impact
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment can be safely deleted since we stopped doing that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, done.

@Nekotekina Nekotekina merged commit ef5225b into RPCS3:master Mar 4, 2017
@Chocobo1 Chocobo1 deleted the travis branch March 4, 2017 13:47
Zangetsu38 pushed a commit to Zangetsu38/rpcs3 that referenced this pull request Mar 4, 2017
* TravisCI: cleanup indentation

2 spaces for indentation

* TravisCI: make jobs drop to 3

TravisCI allocates 2 cpus per build instance

* TravisCI: move package installation to apt addon block

* TravisCI: make pip install to user directory

Also disable sudo for faster bootup

* TravisCI: expand if condition to multiple lines

Also remove outdated comment
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.

None yet

5 participants