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

doxgyen failing to install on Appveyor #4000

Closed
derekbruening opened this issue Dec 31, 2019 · 4 comments · Fixed by #4105
Closed

doxgyen failing to install on Appveyor #4000

derekbruening opened this issue Dec 31, 2019 · 4 comments · Fixed by #4105

Comments

@derekbruening
Copy link
Contributor

We have had many problems installing chocolately packages on Appveyor, causing us to downgrade installation failures to just warnings we continue past. We noticed recently that doxygen has been failing to install every time:

https://ci.appveyor.com/project/DynamoRIO/dynamorio/builds/29690836?fullLog=true

choco install --timeout 600 -y doxygen.portable imagemagick.tool ghostscript || true
Chocolatey v0.10.8
Installing the following packages:
doxygen.portable;imagemagick.tool;ghostscript
By installing you accept licenses for the packages.
Progress: Downloading doxygen.portable 1.8.11... 100%

doxygen.portable v1.8.11 [Approved] - Likely broken for FOSS users (due to download location changes)
doxygen.portable package files install completed. Performing other installation steps.
Attempt to get headers for http://ftp.stack.nl/pub/users/dimitri/doxygen-1.8.11.windows.x64.bin.zip failed.
  The remote file either doesn't exist, is unauthorized, or is forbidden for url 'http://ftp.stack.nl/pub/users/dimitri/doxygen-1.8.11.windows.x64.bin.zip'. Exception calling "GetResponse" with "0" argument(s): "The remote name could not be resolved: 'ftp.stack.nl'"
Downloading doxygen.portable 64 bit
  from 'http://ftp.stack.nl/pub/users/dimitri/doxygen-1.8.11.windows.x64.bin.zip'
ERROR: The remote file either doesn't exist, is unauthorized, or is forbidden for url 'http://ftp.stack.nl/pub/users/dimitri/doxygen-1.8.11.windows.x64.bin.zip'. Exception calling "GetResponse" with "0" argument(s): "The remote name could not be resolved: 'ftp.stack.nl'" 
This package is likely not broken for licensed users - see https://chocolatey.org/docs/features-private-cdn.
The install of doxygen.portable was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\doxygen.portable\tools\chocolateyInstall.ps1'.
 See log for details.

We could try the doxygen package instead of doxygen.portable.

@derekbruening
Copy link
Contributor Author

Xref #2616

@derekbruening
Copy link
Contributor Author

This is blocking #2861, #3194, DynamoRIO/drmemory#2255 where we need the docs for package building. We may also have to solve the ghostscript and fig2dev problems.

@derekbruening
Copy link
Contributor Author

#3235 is also blocking docs from running

@derekbruening
Copy link
Contributor Author

I can get this to work by bypassing choco and directly downloading.

derekbruening added a commit that referenced this issue Feb 15, 2020
Removes build-time transformation from FIG to PNG or EPS and instead
checks in the PNG files for our handful of figures.  This then removes
the need for Fig2Dev, Ghostscript, and ImageMagick.

Fixes Doxygen installation on Appveyor by directly downloading it
rather than trying to use Chocolatey.

Fixes identification of whether Doxygen is Cygwin or not by changing
the test to try to write to a UNIX path as the old test failed on
newer Doxygen versions.

Fixes #3235
Fixes #4000
derekbruening added a commit that referenced this issue Feb 15, 2020
Removes build-time transformation from FIG to PNG or EPS and instead
checks in the PNG files for our handful of figures.  This then removes
the need for Fig2Dev, Ghostscript, and ImageMagick altogether.

Fixes Doxygen installation on Appveyor by directly downloading it
rather than trying to use Chocolatey.

Fixes identification of whether Doxygen is Cygwin or not by changing
the test to try to write to a UNIX path as the old test failed on
newer Doxygen versions.

Fixes a few x86 INSTR_CREATE Doxygen errors reported by the new Doxygen.

Suppresses documenting the many AArch64 INSTR_CREATE macros missing
headers: i#4106 covers adding the headers and remove the exclusions.

Issue: #4106
Fixes #3235
Fixes #4000
derekbruening added a commit that referenced this issue Dec 30, 2020
The doxygen.nl URL breaks every few months when they put out their
latest release and remove the old release.  We switch to their
sourceforge site which archives old releases, so we won't break in the
near future.  (Xref #4000 where we used to use chocolatey to avoid
specifying the precise version: but choco was flaky.)
derekbruening added a commit that referenced this issue Dec 30, 2020
The doxygen.nl URL breaks every few months when they put out their
latest release and remove the old release.  We switch to their
sourceforge site which archives old releases, so we won't break in the
near future.  (Xref #4000 where we used to use chocolatey to avoid
specifying the precise version: but choco was flaky.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant