Skip to content

[release-25.11] treewide: remove redundant stylix.image escaping and string coercion#2143

Merged
trueNAHO merged 3 commits intonix-community:release-25.11from
trueNAHO:backport-2134-to-release-25.11
Jan 15, 2026
Merged

[release-25.11] treewide: remove redundant stylix.image escaping and string coercion#2143
trueNAHO merged 3 commits intonix-community:release-25.11from
trueNAHO:backport-2134-to-release-25.11

Conversation

@trueNAHO
Copy link
Copy Markdown
Member

Manual backport of #2134.

The only difference with the original commit is in /modules/hyprpaper/hm.nix.


trueNAHO and others added 3 commits January 14, 2026 16:09
Co-authored-by: Matt Sturgeon <matt@sturgeon.me.uk>
(cherry picked from commit 3f6651a)
Coerce derivations to store paths to ensure non-null values are
stringified store paths with valid string contexts.

Fixes: 838df8b ("stylix: improve `stylix.image` type (nix-community#1414)")

Co-authored-by: Matt Sturgeon <matt@sturgeon.me.uk>
(cherry picked from commit ca1bc32)
Remove redundant shell escaping and string coercion as stylix.image is
guaranteed to be a stringified store path with a valid string context
and no special characters.

Fixes: 3499e3e ("treewide: properly quote stylix.image when used as a shell argument")
(cherry picked from commit 61c9f4d)
@stylix-automation stylix-automation bot added topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: stylix /stylix/ subsystem labels Jan 14, 2026
Copy link
Copy Markdown
Contributor

@MrSom3body MrSom3body left a comment

Choose a reason for hiding this comment

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

LGTM :)

@trueNAHO trueNAHO enabled auto-merge (rebase) January 15, 2026 15:36
@trueNAHO trueNAHO disabled auto-merge January 15, 2026 15:50
@trueNAHO trueNAHO merged commit c1a01c5 into nix-community:release-25.11 Jan 15, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: stylix /stylix/ subsystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants