Skip to content

Conversation

Liam-Weitzel
Copy link

@Liam-Weitzel Liam-Weitzel commented Jul 12, 2025

My first nix package, a maplestory inspired cursor theme. Works on both x11 and wayland. A fork of the popular banana-cursor.

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/)
  • Nixpkgs 25.11 Release Notes (or backporting 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 25.05 NixOS Release notes)
    • (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, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 12.first-time contribution This PR is the author's first one; please be gentle! 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Jul 12, 2025
@nixpkgs-ci nixpkgs-ci bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Jul 12, 2025
@langsjo
Copy link
Contributor

langsjo commented Jul 12, 2025

And welcome to nixpkgs! These are just some small changes to follow conventions, but otherwise looks good in my eyes

@Iamanaws
Copy link
Contributor

Good job addressing issues.

As I mentioned in #423912,

I still want to note that this package may not have sufficient relevance.

Before adding a new package

How realistic is it that it will be used by other people? It's good that nixpkgs caters to various niches, but if it's a niche of 5 people it's probably too small. A good estimate is checking upstream issues and pull requests, or other software repositories. Library packages should have at least one dependent.

Checkout NUR for personal packages

@Liam-Weitzel
Copy link
Author

And welcome to nixpkgs! These are just some small changes to follow conventions, but otherwise looks good in my eyes

Hi! Thanks for the review! I've addressed these issues

@Liam-Weitzel
Copy link
Author

Good job addressing issues.

As I mentioned in #423912,

I still want to note that this package may not have sufficient relevance.

Before adding a new package

How realistic is it that it will be used by other people? It's good that nixpkgs caters to various niches, but if it's a niche of 5 people it's probably too small. A good estimate is checking upstream issues and pull requests, or other software repositories. Library packages should have at least one dependent.

Checkout NUR for personal packages

Hi, yes, apologies for making a new PR. I usually work collaboratively on gitlab. That is a valid concern and honestly I don't know if more than 5 people will use this. The parent package 'banana-cursor' has had some success (281 stars), and this plugin builds on top of it, so it's fair to say the audience is quite small. I submitted it here because it seemed like a natural extension of what's already in nixpkgs & i'd hoped others would appreciate it. But I totally understand if it doesn't meet the bar for inclusion.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jul 12, 2025
@Liam-Weitzel
Copy link
Author

Hi, yes, apologies for making a new PR. I usually work collaboratively on gitlab. That is a valid concern and honestly I don't know if more than 5 people will use this. The parent package 'banana-cursor' has had some success (281 stars), and this plugin builds on top of it, so it's fair to say the audience is quite small. I submitted it here because it seemed like a natural extension of what's already in nixpkgs & i'd hoped others would appreciate it. But I totally understand if it doesn't meet the bar for inclusion.

That said, I’ve noticed that there are other packages recently merged into nixpkgs that seem similarly niche or limited in audience - for example, #422351, which appears to be a classroom project with a fairly minimal upstream presence (12 stars). I think it would be super cool if this package was merged and I'm sure I can convince my 12 friends to use this cursor theme 😜

@Liam-Weitzel
Copy link
Author

Hi, how long is the usual queue time to get a runner for the x86-64-darwin maintainers ofborg test? It's been > 1 month.

@Iamanaws
Copy link
Contributor

Iamanaws commented Aug 18, 2025

Hi, how long is the usual queue time to get a runner for the x86-64-darwin maintainers ofborg test? It's been > 1 month.

All checks have run fine. The maintainers on Darwin checks have been skip as this is Linux only, nothing to worry about.

@Liam-Weitzel
Copy link
Author

Liam-Weitzel commented Aug 22, 2025

All checks have run fine. The maintainers on Darwin checks have been skip as this is Linux only, nothing to worry about.

Nice! What does the usual process look like for the final merge?

@Iamanaws
Copy link
Contributor

Nice! What does the usual process look like for the final merge?

Usually, like a long wait until a committer give it a look. If running with luck you can get it merge in like 2 weeks, but it's been more than that, so this one seems like the long wait ones. (from some weeks to a few months 🥲)

Too many PRs and too little committers. But hopefully you will get it soon

@Iamanaws
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 424590
Commit: 4bc4426ef0f46460a3631837f1fcb3faa202e501 (subsequent changes)
Merge: e85438b4a5734e497f17ac808584669debf1e446

Logs: https://github.com/Iamanaws/nixpkgs-review-gha/actions/runs/17148273880


x86_64-linux

✅ 1 package built:
  • maplestory-cursor

aarch64-linux

✅ 1 package built:
  • maplestory-cursor

@nixpkgs-ci nixpkgs-ci bot 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 Aug 22, 2025
@Liam-Weitzel
Copy link
Author

Usually, like a long wait until a committer give it a look. If running with luck you can get it merge in like 2 weeks, but it's been more than that, so this one seems like the long wait ones. (from some weeks to a few months 🥲)

Too many PRs and too little committers. But hopefully you will get it soon

Ah alright, no worries at all. Thank you for all the guidance :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 0 This PR does not cause any packages 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. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants