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

nixos/systemd-boot: Avoid remote mypy executions #263397

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

nbraud
Copy link
Contributor

@nbraud nbraud commented Oct 25, 2023

Description of changes

In the systemd-boot NixOS module, set preferLocalBuilds when type-checking systemd-boot-builder.py, to avoid the overhead of remote builds.

This is a second go at this, #261640 wrongly used runCommandLocal instead, breaking a NixOS test and blocking the nixos-unstable channel.

Many thanks to @vcunat for pointing out runCommandLocal also sets allowSubstitutes = false;

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
  • Tested, as applicable:
    • NixOS test installer.simpleUefiSystemdBoot
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@nbraud
Copy link
Contributor Author

nbraud commented Apr 29, 2024

Rebased to resolve merge conflict

@nbraud nbraud merged commit d8e31f6 into NixOS:master Apr 30, 2024
21 checks passed
@nbraud nbraud deleted the runLocal/systemd-boot branch April 30, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants