catgirl: init at 1.6

[xfnw]

Motivation for this change

package the catgirl irc client

Things done

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
• Built on platform(s)
  • NixOS
  • macOS
  • other Linux distributions
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Ensured that relevant documentation is up to date
• Fits CONTRIBUTING.md.

[xfnw]

oops, forgot to add myself to the maintainers list

fixed

[fabaff]

Result of nixpkgs-review pr 115159 run on x86_64-linux 1

1 package built:

• catgirl

[SuperSandro2000]

please squash the catgirl: init at 1.6 commits.

[fogti]

It might be a good idea to sort the package lists (e.g. argument list in the header, nativeBuildInputs, buildInputs) alphabetically.

[xfnw]

It might be a good idea to sort the package lists (e.g. argument list in the header, nativeBuildInputs, buildInputs) alphabetically.

@zseri hmm, i can't seem to find any other packages that sort the build inputs, is that a normal thing to do?

[fogti]

The last few PRs that I reviewed all had sorted build inputs, and I think it is a normal thing to do.
This does not apply to the argument list in the header: it is often not "simply" sorted, and there exist multiple conflicting conventions applied to different packages in nixpkgs.

[xfnw]

@SuperSandro2000 could you re-review?

[fogti]

builds successfully with Nix (2.3.10) on Gentoo.

[SuperSandro2000]

other than that LGTM