Skip to content

travis: only install required apt packages - #3721

Closed
MarcelRaad wants to merge 7 commits into
curl:masterfrom
MarcelRaad:travis_apt_cleanup
Closed

travis: only install required apt packages#3721
MarcelRaad wants to merge 7 commits into
curl:masterfrom
MarcelRaad:travis_apt_cleanup

Conversation

@MarcelRaad

Copy link
Copy Markdown
Member

This allows to selectively update some clang jobs to xenial as the LLVM repository name is different for xenial than for trusty. Also, it saves a little time for most Linux jobs.

This reduces the GCC job runtimes a little and it's needed to
selectively update clang builds to xenial.

Closes
Reduces the time needed for the other jobs a little.

Closes
Reduces the time needed for the other jobs a little.

Closes
Reduces the time needed for the other jobs a little.

Closes
Reduces the time needed for the other jobs a little.

Closes
Reduces the time needed for the other jobs a little.

Closes
Reduces the time needed for the other jobs a little.

Closes

@danielgustafsson danielgustafsson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it possible to also only compile dependencies such as MesaLink for the mesalink jobs? This is currently done in before_script but my Travis skills aren't up to scratch for figuring out if that can be made job dependent.

@MarcelRaad

Copy link
Copy Markdown
Member Author

I wondered about that too. Probably @bagder had a good reason to do that starting in 277d3cd for mbedtls (and then for wolfssl and mesalink) as the prior brotli and BoringSSL builds were only executed for those jobs?

@MarcelRaad

Copy link
Copy Markdown
Member Author

FYI: I hope we can get rid of some of those manually built dependencies. I have another branch based on this one to update some jobs to Xenial (not all though because Xenial's Valgrind doesn't like Xenial's OpenSSL, which is why this PR is needed). Xenial contains some of the dependencies missing in Trusty (but at least wolfSSL's version is too old for test 311).

@bagder

bagder commented Apr 5, 2019

Copy link
Copy Markdown
Member

@bagder had a good reason to do that starting in 277d3cd for mbedtls (and then for wolfssl and mesalink)

Who really knows how travis works? I certainly don't and mostly try every possible combination until it seems to behave... My intentions were that the builds of those dependencies should end up in directories that are cached between invokes so that only 'make install' would be necessary for most builds when the cache is used. We can of course polish the logic so that we only 'make install' dependencies that are actually used.

MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Apr 11, 2019
Reduces the time needed for the other jobs a little.

Closes curl#3721
@MarcelRaad
MarcelRaad deleted the travis_apt_cleanup branch April 11, 2019 19:17
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants