Skip to content

zotify, onthespot: drop#446400

Closed
eclairevoyant wants to merge 2 commits into
NixOS:masterfrom
eclairevoyant:drop-1
Closed

zotify, onthespot: drop#446400
eclairevoyant wants to merge 2 commits into
NixOS:masterfrom
eclairevoyant:drop-1

Conversation

@eclairevoyant

@eclairevoyant eclairevoyant commented Sep 26, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Things done

closes #446383
closes #356878

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@eclairevoyant eclairevoyant mentioned this pull request Sep 26, 2025
Closed
13 tasks
@nixpkgs-ci nixpkgs-ci Bot added 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 Sep 26, 2025
@nix-owners
nix-owners Bot requested review from bwkam and onny September 26, 2025 16:54
@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 26, 2025
ryand56
ryand56 approved these changes Sep 26, 2025
View reviewed changes

@ryand56 ryand56 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

onthespot relies on broken dependencies as of the latest version, so I'm in favor to drop both.

@ryand56 ryand56 added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Sep 26, 2025
@qweered

qweered commented Sep 27, 2025

Copy link
Copy Markdown
Contributor

Aur builds from appimage https://aur.archlinux.org/packages/onthespot-appimage

@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 3+ This PR was reviewed and approved by three or more persons. and removed 12.approvals: 2 This PR was reviewed and approved by two persons. labels Sep 27, 2025
@mdaniels5757 mdaniels5757 added the 8.has: clean-up This PR removes packages or removes other cruft label Sep 27, 2025
@eclairevoyant
eclairevoyant deleted the drop-1 branch September 29, 2025 15:52
@qweered

qweered commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

Why closing PR?

@henrispriet henrispriet mentioned this pull request Oct 6, 2025
Merged
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SuperSandro2000 SuperSandro2000 SuperSandro2000 approved these changes
@onny onny Awaiting requested review from onny
@bwkam bwkam Awaiting requested review from bwkam
+2 more reviewers
@ryand56 ryand56 ryand56 approved these changes
@qweered qweered qweered approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

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: 3+ This PR was reviewed and approved by three or more persons.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

zotify: package is outdated and does not work

5 participants

@eclairevoyant @qweered @SuperSandro2000 @ryand56 @mdaniels5757