Skip to content

libcurl3 still exists on Ubuntu 18.04 servers#2929

Closed
krknopp wants to merge 2 commits intomicrosoft:masterfrom
krknopp:master
Closed

libcurl3 still exists on Ubuntu 18.04 servers#2929
krknopp wants to merge 2 commits intomicrosoft:masterfrom
krknopp:master

Conversation

@krknopp
Copy link

@krknopp krknopp commented Apr 17, 2020

on our systems, libcurl3 is still available on Ubuntu 18.04 servers, so the script installs libcurl3, which forces curl and libcurl4 to be removed.

This change checks to see which version of Ubuntu the script is running on and then installs the appropriate version of libcurl.

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request May 1, 2020
…ure Spot

* Adds scripts to generate scale sets for testing Linux.
    * Note workaround for microsoft/azure-pipelines-agent#2929
* Switches Windows validation to 'Spot' VMs.
* Opens the git port 9418.
* Removes provisioning of the no longer used 'logs' file share.
* Changes Azure region to 'westus2', which is cheaper.
* Adds +x to all the scripts in scripts/azure-pipelines.
* Use 'xml-results' for all platforms instead of 'raw xml results' on Windows.
BillyONeal added a commit to microsoft/vcpkg that referenced this pull request May 1, 2020
…ure Spot

* Adds scripts to generate scale sets for testing Linux.
    * Note workaround for microsoft/azure-pipelines-agent#2929
* Switches Windows validation to 'Spot' VMs.
* Opens the git port 9418.
* Removes provisioning of the no longer used 'logs' file share.
* Changes Azure region to 'westus2', which is cheaper.
* Adds +x to all the scripts in scripts/azure-pipelines.
* Use 'xml-results' for all platforms instead of 'raw xml results' on Windows.
# ubuntu 18 uses libcurl4
# ubuntu 14, 16 and other linux use libcurl3
apt install -y libcurl3 || apt install -y libcurl4
if [ $(lsb_release -rs) = "18.04" ]
Copy link
Member

Choose a reason for hiding this comment

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

Should this check for 14 and 16 instead because future versions are going to use 4?

Copy link

Choose a reason for hiding this comment

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

There's a workaround btw. If you install libcurl4 first, then do: apt-mark hold libcurl4
That will prevent the crapfest that follows.

Copy link

@azilber azilber May 21, 2020

Choose a reason for hiding this comment

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

Also, why not just reverse it? Otherwise you're going to have to continually keep adding future versions:
apt-get install -y libcurl4 || apt-get install -y libcurl3

Copy link
Author

@krknopp krknopp left a comment

Choose a reason for hiding this comment

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

updated code to reflect comments.

@mjroghelia
Copy link
Contributor

@krknopp @BillyONeal Thanks for bringing this to our attention. I've reversed the order of the installs (#2998) to address the immediate issue on Ubuntu 18. I'm holding off on more sophisticated use of lsb_release as part of a larger refactoring effort. Sorry for the delay on this. The fix will be in the next release of the agent, which I'll begin the process of releasing shortly.

@mjroghelia mjroghelia closed this Jun 3, 2020
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.

4 participants