search

ammaraskar / sphinx-action

Github action that builds docs using sphinx and places errors inline
Apache License 2.0
194 stars 116 forks source link

Cannot run apt update #33

Open maxnoe opened 3 years ago

maxnoe commented 3 years ago

Trying to install additional packages, running apt update --yes fails with an error (presumably something to do with the new debian release?) Config:

  with:
    docs-folder: docs/
    pre-build-command: apt update --yes && apt install --yes git build-essential pandoc

Log:

Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
Get:2 http://security.debian.org/debian-security buster/updates InRelease [65.4 kB]
Get:3 http://deb.debian.org/debian buster-updates InRelease [51.9 kB]
Reading package lists...
E: Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
E: Repository 'http://deb.debian.org/debian buster-updates InRelease' changed its 'Suite' value from 'stable-updates' to 'oldstable-updates'
E: Repository 'http://security.debian.org/debian-security buster/updates InRelease' changed its 'Suite' value from 'stable' to 'oldstable'

Link to a build: https://github.com/cta-observatory/ctapipe/pull/1774/checks?check_run_id=3424315763

ocket8888 commented 3 years ago

I'm wondering if skipping the update would help - otherwise they may have to update the base image to bullseye - since as of 19 days ago it's the new stable and buster is the "oldstable" which seems to be what those messages are about.

ocket8888 commented 3 years ago

Update: Yeah, if you skip the update it seems to be working okay. Build link: https://github.com/apache/trafficcontrol/pull/6158/checks?check_run_id=3495291877

So instead of

  with:
    docs-folder: docs/
    pre-build-command: apt update --yes && apt install --yes git build-essential pandoc

just

  with:
    docs-folder: docs/
    pre-build-command: apt install --yes git build-essential pandoc

should work.

ax3l commented 3 years ago

Just tried this - this only moves the error down to the apt install stage and we will still lack latexmk for the build (#32).

I think we should update to a newer Debian base image to fix.

ocket8888 commented 3 years ago

Might be specific to latexmk, then, because I'm using it successfully to install pandoc. All that really means, though, is that anything else you install may or may not work and could break at any moment.