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

awless: avoid building on aarch64-linux #173792

Closed
wants to merge 1 commit into from

Conversation

swdunlop
Copy link
Contributor

Description of changes

Disables build of Awless on aarch64-linux, leaving it enabled for other platforms where it currently builds in 21.11 and master. The upstream project has not updated since 2018 and needs to update its version of x/sys/unix to build on aarch64-linux.

We could probably patch its go.mod and go.sum, but since I do not actively use aarch64-linux, I cannot assume it is backward compatible. (Especially since x/sys/unix does not guarantee backward compatibility.)

Relevant discussions and issues:

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.05 Release Notes (or backporting 21.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@swdunlop swdunlop added the 0.kind: build failure A package fails to build label May 20, 2022
@swdunlop
Copy link
Contributor Author

Pinging @NixOS/nixos-release-managers for review.

@ofborg ofborg bot added 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels May 20, 2022
@06kellyjac
Copy link
Member

We could probably patch its go.mod and go.sum

It's so old it's using still using dep so there is no go.mod or go.sum

I really think unless you or someone else takes up maintainership of awless this should be dropped.

@@ -16,7 +16,7 @@ buildGoPackage rec {
meta = with lib; {
homepage = "https://github.com/wallix/awless/";
description = "A Mighty CLI for AWS";
platforms = with platforms; linux ++ darwin;
platforms = with platforms; [ "i686-linux" "x86_64-linux" ]++ darwin;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use meta.broken instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point -- this was suggested in the larger #173671 PR which marks various packages as broken.

@swdunlop swdunlop closed this May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: build failure A package fails to build 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants