Skip to content

aws-env: remove#318566

Closed
msanft wants to merge 1 commit intoNixOS:masterfrom
msanft:aws-env/buildgomodule
Closed

aws-env: remove#318566
msanft wants to merge 1 commit intoNixOS:masterfrom
msanft:aws-env/buildgomodule

Conversation

@msanft
Copy link
Contributor

@msanft msanft commented Jun 9, 2024

Description of changes

Remove aws-env, as GH code search doesn't yield any users, and the project seems to be abandoned upstream.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 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
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@msanft msanft requested a review from katexochen June 9, 2024 16:21
@katexochen katexochen mentioned this pull request Jun 9, 2024
35 tasks
@ofborg ofborg bot requested a review from srhb June 9, 2024 16:59
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Jun 9, 2024
@srhb
Copy link
Contributor

srhb commented Jun 10, 2024

Are you an active user of aws-env? I notice that there's not been any maintenance done to upstream in a long while (which I guess your fetchpatch of your own PR also reveals)

@msanft
Copy link
Contributor Author

msanft commented Jun 10, 2024

Are you an active user of aws-env? I notice that there's not been any maintenance done to upstream in a long while (which I guess your fetchpatch of your own PR also reveals)

I'm not. Just migrated it for #318069. If you're fine with removal, I'd vouch for that as well.

@msanft msanft force-pushed the aws-env/buildgomodule branch from e4370bc to dc2823f Compare June 10, 2024 13:43
@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 10, 2024
@srhb
Copy link
Contributor

srhb commented Jun 11, 2024

@msanft I'm going to vote removal, to be honest. Though I appreciate your work, and if anyone complains we can resurrect it. How does that sound to you? I can take care of the removal if need be. :)

@srhb srhb mentioned this pull request Jun 11, 2024
13 tasks
@msanft msanft force-pushed the aws-env/buildgomodule branch from dc2823f to fd00528 Compare June 11, 2024 06:05
@msanft msanft changed the title aws-env: migrate to buildGoModule aws-env: remove Jun 11, 2024
@ofborg ofborg bot added 8.has: clean-up This PR removes packages or removes other cruft 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. and removed 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: clean-up This PR removes packages or removes other cruft 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. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants