Skip to content

[doxygen] Add alternative download URL#9708

Merged
vicroms merged 1 commit intomicrosoft:masterfrom
c72578:2020-01-16_doxygen_Add_alternative_download_URL
Jan 22, 2020
Merged

[doxygen] Add alternative download URL#9708
vicroms merged 1 commit intomicrosoft:masterfrom
c72578:2020-01-16_doxygen_Add_alternative_download_URL

Conversation

@c72578
Copy link
Contributor

@c72578 c72578 commented Jan 16, 2020

Only the current and the previous version of doxygen can be downloaded
from: http://doxygen.nl
Sourceforge provides the current version and any previously released
versions for download: https://sourceforge.net/projects/doxygen/files/
Add sourceforge as alternative download URL. This avoids situations,
when the version of doxygen in vcpkg is n-2 and cannot be downloaded
any more, see e.g. commit: 526f937 and the related PR: #9574

Describe the pull request

  • What does your PR fix? Fixes issue #
    This PR fixes potential download issues of doxygen.
  • Which triplets are supported/not supported? Have you updated the CI baseline?
    N/A
  • Does your PR follow the maintainer guide?
    Yes

@PhoebeHui PhoebeHui self-assigned this Jan 17, 2020
@JackBoosY
Copy link
Contributor

These fake failures will resolve in #9726.

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 9708 in repo microsoft/vcpkg

@c72578 c72578 force-pushed the 2020-01-16_doxygen_Add_alternative_download_URL branch from 21ff39d to c362659 Compare January 18, 2020 09:36
@JackBoosY
Copy link
Contributor

/azp run

@PhoebeHui
Copy link
Contributor

Need rerun CI on windows after PR #9752 merged.

Only the current and the previous version of doxygen can be downloaded
from: http://doxygen.nl
Sourceforge provides the current version and any previously released
versions for download: https://sourceforge.net/projects/doxygen/files/
Add sourceforge as alternative download URL. This avoids situations,
when the version of doxygen in vcpkg is n-2 and cannot be downloaded
any more, see e.g. commit: 526f937
@c72578 c72578 force-pushed the 2020-01-16_doxygen_Add_alternative_download_URL branch from c362659 to d9c0e0a Compare January 20, 2020 22:41
@c72578
Copy link
Contributor Author

c72578 commented Jan 20, 2020

OK, PR #9752 has been merged in the meantime and CI has passed.

@PhoebeHui PhoebeHui added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jan 21, 2020
@vicroms vicroms merged commit f992ddb into microsoft:master Jan 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants