Skip to content

hypr{launcher,pwcenter,toolkit,wire}: init at 0.1.0#453264

Merged
JohnRTitor merged 5 commits intoNixOS:masterfrom
SchweGELBin:hyprupdate
Nov 18, 2025
Merged

hypr{launcher,pwcenter,toolkit,wire}: init at 0.1.0#453264
JohnRTitor merged 5 commits intoNixOS:masterfrom
SchweGELBin:hyprupdate

Conversation

@SchweGELBin
Copy link
Member

@SchweGELBin SchweGELBin commented Oct 18, 2025 •
edited
Loading

https://github.com/hyprwm/hyprlauncher/releases/tag/v0.1.0
https://github.com/hyprwm/hyprpwcenter/releases/tag/v0.1.0
https://github.com/hyprwm/hyprtoolkit/releases/tag/v0.1.0
https://github.com/hyprwm/hyprwire/releases/tag/v0.1.0

hyprtoolkit and hyprwire are libraries used for hyprlauncher and hyprpwcenter
These are the initial releases for those packages and these packages are kinda buggy in the current state.

All of those packages don't build with the standard stdenv but with gcc15Stdenv.

Things done

  • 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.

@SchweGELBin
Copy link
Member Author

SchweGELBin commented Oct 18, 2025

Another package called hyprlauncher existed before and already got removed.
The alias was still there and throws an error in the test, even though I removed the alias in this pr.

@Gliczy
Copy link
Member

Gliczy commented Oct 21, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 453264
Commit: ad4d18247f8a4291d08a2445b7a5b27cc525b644

x86_64-linux

✅ 5 packages built:
  • hyprlauncher
  • hyprpwcenter
  • hyprtoolkit
  • hyprwire
  • nixpkgs-manual

Gliczy
Gliczy reviewed Oct 21, 2025
View reviewed changes
@Gliczy
Copy link
Member

Gliczy commented Oct 21, 2025

hyprpwcenter seem to work as it should.
As for hyprlauncher, it opens but only list a few entries. I'm not sure if this is an upstream issue.

Pinging @NixOS/hyprland since it adds the hyprland team as maintainer.

@SchweGELBin
Copy link
Member Author

SchweGELBin commented Oct 21, 2025

hyprpwcenter seem to work as it should. As for hyprlauncher, it opens but only list a few entries. I'm not sure if this is an upstream issue.

hyprlauncher searches for deskop items by default. (In my case only the Steam Games)
You can also search for unicode characters with "." and calcuate with "=" as a prefix.

@SchweGELBin SchweGELBin force-pushed the hyprupdate branch 2 times, most recently from 4ab1588 to aa90c29 Compare October 21, 2025 17:22
fufexan
fufexan reviewed Oct 21, 2025
View reviewed changes
@SchweGELBin
Copy link
Member Author

SchweGELBin commented Oct 21, 2025

Thank you for your review @fufexan!
I will implement your changes tomorrow (depending on time zones).

@SchweGELBin SchweGELBin force-pushed the hyprupdate branch 2 times, most recently from 338e6c8 to b6e4bc7 Compare October 22, 2025 09:33
@SchweGELBin

This comment was marked as resolved.

Sign in to view
@JohnRTitor

This comment was marked as resolved.

Sign in to view
@wolfgangwalther

This comment was marked as resolved.

Sign in to view
@wolfgangwalther wolfgangwalther mentioned this pull request Oct 22, 2025
Merged
1 task
wolfgangwalther
wolfgangwalther reviewed Oct 22, 2025
View reviewed changes
@khaneliman khaneliman reopened this Oct 22, 2025
@nixpkgs-ci nixpkgs-ci bot added 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. labels Oct 22, 2025
@wolfgangwalther

This comment was marked as resolved.

Sign in to view
MattSturgeon
MattSturgeon reviewed Oct 22, 2025
View reviewed changes
@SchweGELBin SchweGELBin force-pushed the hyprupdate branch 2 times, most recently from c0d5f42 to d8e8dcc Compare October 27, 2025 22:12
@SchweGELBin
Copy link
Member Author

SchweGELBin commented Oct 27, 2025 •
edited
Loading

  • hyprtoolkit got updated to 0.2.0
  • hyprpwcenter got updated to 0.1.1 and depends on hyprtoolkit 0.2.0
  • hyprlauncher got updated to 0.1.1 and depends on hyprtoolkit 0.2.0

hyprlauncher 0.1.1 was broken for me (something like undeclared variable "options").
The other package updates worked for me, but hyprlauncher 0.1.0 doesn't work with hyprtoolkit 0.2.0 either.

So I won't update these packages for now and would like to keep this pr as is.

Edit:
Sorry for the force-pushes, I was trying to get the tests working again:
Server Error: Sorry, this diff is taking too long to generate.

Edit 2:
I removed the mainProgram option from hyprtoolkit as it is actually only a library.

@SchweGELBin SchweGELBin force-pushed the hyprupdate branch 4 times, most recently from 7aaa7eb to 81c0ccf Compare October 27, 2025 23:01
@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Oct 30, 2025
@nixpkgs-ci nixpkgs-ci bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Nov 18, 2025
@SchweGELBin
Copy link
Member Author

SchweGELBin commented Nov 18, 2025

Hey, I've rebased this pr to the current master.
The old hyprlauncher package got removed, so I removed it here too.

I won't update the packages here as these are the original versions and it's a good base for working packages.
I'm planning to create a follow-up pr, where I bump the versions when possible.

Can someone review this please, so I can create a follow-up?

@qweered
Copy link
Contributor

qweered commented Nov 18, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 453264
Commit: a7a7793ddc846ea35118f654850b112dda7752a4 (subsequent changes)
Merge: 79c01a89104a1824c1edc20d75cd7ccf93605c25

Logs: https://github.com/qweered/nixpkgs-review-gha/actions/runs/19451445003

x86_64-linux

✅ 4 packages built:
  • hyprlauncher
  • hyprpwcenter
  • hyprtoolkit
  • hyprwire

aarch64-linux

✅ 4 packages built:
  • hyprlauncher
  • hyprpwcenter
  • hyprtoolkit
  • hyprwire

qweered
qweered approved these changes Nov 18, 2025
View reviewed changes
Copy link
Contributor

@qweered qweered left a comment

Choose a reason for hiding this comment

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

Approved automatically following the successful run of nixpkgs-review.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Nov 18, 2025
@JohnRTitor JohnRTitor added this pull request to the merge queue Nov 18, 2025
Merged via the queue into NixOS:master with commit 9eebc47 Nov 18, 2025
28 of 31 checks passed
@SchweGELBin SchweGELBin deleted the hyprupdate branch November 18, 2025 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@khaneliman khaneliman khaneliman left review comments

@MattSturgeon MattSturgeon MattSturgeon left review comments

@wolfgangwalther wolfgangwalther wolfgangwalther left review comments

+3 more reviewers

@fufexan fufexan fufexan left review comments

@Gliczy Gliczy Gliczy left review comments

@qweered qweered qweered approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@SchweGELBin @Gliczy @JohnRTitor @wolfgangwalther @qweered @khaneliman @MattSturgeon @fufexan