sirula: 1.0.0-unstable -> 1.1.0

[atagen]

Things done

• updated source to 1.1.0
• updated Cargo.lock

• 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:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• 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/)
• 25.05 Release Notes (or backporting 24.11 and 25.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.

[l1npengtul]

Use of pname binding here is discouraged.

[atagen]

depends on #372016 now for maintainer update

[l1npengtul]

You can add yourself to maintainers in the same PR, just make sure the commit that touches the maintainers file comes before the commit that changes the package.

[l1npengtul]

Also, the package isn't formatted properly. Please make sure you are using nixfmt, as alejandra is not RFC compliant (I know from experience).

Here is the diff for reference:

-{ nix-update-script, lib, fetchFromGitHub, rustPlatform, pkg-config, gtk3
-, gtk-layer-shell, }:
+{
+  nix-update-script,
+  lib,
+  fetchFromGitHub,
+  rustPlatform,
+  pkg-config,
+  gtk3,
+  gtk-layer-shell,
+}:
 rustPlatform.buildRustPackage {
   pname = "sirula";
   version = "1.1.0";
@@ -13,14 +20,16 @@ rustPlatform.buildRustPackage {
   cargoLock = {
     lockFile = ./Cargo.lock;
     outputHashes = {
-      "osstrtools-0.2.2" =
-        "sha256-Co4pcikfN4vtIVK7ZsRGCWMAhMJWNNVZe/AdN1nMlmQ=";
+      "osstrtools-0.2.2" = "sha256-Co4pcikfN4vtIVK7ZsRGCWMAhMJWNNVZe/AdN1nMlmQ=";
     };
   };
 
   nativeBuildInputs = [ pkg-config ];
 
-  buildInputs = [ gtk3 gtk-layer-shell ];
+  buildInputs = [
+    gtk3
+    gtk-layer-shell
+  ];
 
   passthru.updateScript = nix-update-script { };

[atagen]

oh well, it's needed across a few PRs for me to become maintainer, better not to pin it to any one of them

[atagen]

Also, the package isn't formatted properly. Please make sure you are using nixfmt, as alejandra is not RFC compliant (I know from experience).

I use it like , nixfmt pkgs/by-name/si/sirula/package.nix, maybe comma is not using nixfmt-rfc-style for this but nixfmt-classic or something?

[l1npengtul]

Also, the package isn't formatted properly. Please make sure you are using nixfmt, as alejandra is not RFC compliant (I know from experience).

I use it like , nixfmt pkgs/by-name/si/sirula/package.nix, maybe comma is not using nixfmt-rfc-style for this but nixfmt-classic or something?

Thats odd, should be the correct usage...
Perhaps try using nix-shell -p nixfmt-rfc-style to ensure you're getting the correct one?

[l1npengtul]

oh well, it's needed across a few PRs for me to become maintainer, better not to pin it to any one of them

Ah, ok.

[l1npengtul]

Please format this file again.

[atagen]

oops, I didn't mean to ping you. was trying to figure out what's failing in these tests/how to rerun them since it seems the CI itself was broken at that time? sorry

[Eveeifyeve]

Use of pname binding here is discouraged.

Related Resource: #277994

[Eveeifyeve]

@NixOS/nixpkgs-vet could you re-run the ci as it broke at curl stage which is not this pr fault.

[philiptaron]

@NixOS/nixpkgs-vet could you re-run the ci as it broke at curl stage which is not this pr fault.

Done. Passed.

[niklaskorz]

Please change the PR title to "sirula: 1.0.0-unstable -> 1.1.0"

[Eveeifyeve]

The code looks good, I just need to test on my end with both nixpkgs review and local bin build then I will approve it.

[niklaskorz]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 371977

---

x86_64-linux

✅ 1 package built:

• sirula

---

aarch64-linux

✅ 1 package built:

• sirula