Skip to content

heads: Improve updateDepsScript#1293

Merged
OPNA2608 merged 1 commit into
ngi-nix:mainfrom
OPNA2608:fix-heads-updateScript-must-apply-patches
Jul 3, 2025
Merged

heads: Improve updateDepsScript#1293
OPNA2608 merged 1 commit into
ngi-nix:mainfrom
OPNA2608:fix-heads-updateScript-must-apply-patches

Conversation

@OPNA2608
Copy link
Copy Markdown
Contributor

@OPNA2608 OPNA2608 commented Jul 3, 2025

  • Fix leftover ftpmirror.gnu.org URLs
    We still can't rely on all the mirrors in there giving correct results
  • Exclude acpica-unix2-20220331.tar.gz from being fetched by the script
    Original source is dead, unsure which mirror might be correct & safe due to hash differences
  • Don't throw away everything if a single URL fails to fetch
    Just give an empty hash and move on. A human can manually re-attempt this later.

This closes #1256, technically. But more in a "letter of the law" way. We apply the coreboot patches now, but the failing ACPICA source download still doesn't work, and we just consistently exclude it now. Even the URL from the patch files is dead.

There is technically a different script that we should maybe be using for getting crossgcc dependencies, but it too gives a dead URL for specifically this package: https://github.com/linuxboot/heads/blob/d0350e02f4f88fbc16eb26893c2edc8e7cdae441/bin/fetch_coreboot_crossgcc_archive.sh#L105-L115

There are some possible candidates for where we could fetch the archive from instead, but we don't need to fix this right now. The board that needs this is marked untested, so it shouldn't be a big deal to keep this excluded.

@OPNA2608 OPNA2608 enabled auto-merge (squash) July 3, 2025 18:08
- Fix leftover ftpmirror.gnu.org URLs
  We still can't rely on all the mirrors in there giving correct results
- Exclude acpica-unix2-20220331.tar.gz from being fetched by the script
  Original source is dead, unsure which mirror might be correct & safe due to hash differences
- Don't throw away everything if a single URL fails to fetch
  Just give an empty hash and move on. A human can manually re-attempt this later.
@OPNA2608 OPNA2608 force-pushed the fix-heads-updateScript-must-apply-patches branch from 7f3132c to 2cdd102 Compare July 3, 2025 18:09
@OPNA2608 OPNA2608 merged commit d2f2907 into ngi-nix:main Jul 3, 2025
14 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Nix@NGI Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Heads: Apply coreboot patches during updateDepsScript

3 participants