Skip to content

Move to Ubuntu Trusty on Travis CI and remove apt add-on#428

Merged
jackycute merged 1 commit intohackmdio:masterfrom
PeterDaveHello:travis-ci-optimize
Apr 19, 2017
Merged

Move to Ubuntu Trusty on Travis CI and remove apt add-on#428
jackycute merged 1 commit intohackmdio:masterfrom
PeterDaveHello:travis-ci-optimize

Conversation

@PeterDaveHello
Copy link
Copy Markdown
Contributor

Two reasons here:

  1. Ubuntu Precise is reaching EOL, though Travis CI may take care of it.
    we still can move to Trusty earlier as I didn't see any dependency of
    Precise here.

  2. g++4.8 is built-in in Travis CI's Ubuntu Trusty environment, which
    means we don't need to spend time on adding ppa repository, apt update
    and apt install, could save time on the CI.

Two reasons here:

1) Ubuntu Precise is reaching EOL, though Travis CI may take care of it.
we still can move to Trusty earlier as I didn't see any dependency of
Precise here.

2) g++4.8 is built-in in Travis CI's Ubuntu Trusty environment, which
means we don't need to spend time on adding ppa repository, apt update
and apt install, could save time on the CI.
@PeterDaveHello PeterDaveHello changed the title Move to Ubuntu Trusty on Travis CI Move to Ubuntu Trusty on Travis CI and remove apt add-on Apr 18, 2017
@jackycute jackycute requested a review from a60814billy April 18, 2017 05:40
Copy link
Copy Markdown
Member

@a60814billy a60814billy left a comment

Choose a reason for hiding this comment

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

It seems good!

@jackycute
Copy link
Copy Markdown
Member

Thanks @PeterDaveHello

@jackycute jackycute merged commit 57b670d into hackmdio:master Apr 19, 2017
@PeterDaveHello PeterDaveHello deleted the travis-ci-optimize branch April 19, 2017 13:44
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.

3 participants