Skip to content

rofi: fix mkTarget usage#1593

Merged
0xda157 merged 1 commit intonix-community:masterfrom
0xda157:rofi-mkTarget-config
Jul 4, 2025
Merged

rofi: fix mkTarget usage#1593
0xda157 merged 1 commit intonix-community:masterfrom
0xda157:rofi-mkTarget-config

Conversation

@0xda157
Copy link
Copy Markdown
Contributor

@0xda157 0xda157 commented Jul 4, 2025 •
edited
Loading 

       … while evaluating the option `home-manager.users.azalea.programs.rofi.theme.*.active-text.value':

       … while evaluating definitions from `/nix/store/y0v88m35c78ih1mbkrcdnlnfmb4pbgw6-source/modules/rofi/hm.nix':

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: stylix: unguareded `config.lib.stylix.colors` accessed while using mkTarget

Fixes: dea0337e0b ("stylix: restrict access to config while using mkTarget (#1368)")
Fixes: 713f8dae31 ("rofi: use mkTarget (#1550)")

Things done

Notify maintainers

@0xda157 0xda157 requested a review from trueNAHO July 4, 2025 17:28
@stylix-automation stylix-automation bot added the topic: home-manager Home Manager target label Jul 4, 2025
@0xda157 0xda157 force-pushed the rofi-mkTarget-config branch from 8d53918 to 8644c72 Compare July 4, 2025 17:30
@0xda157 0xda157 force-pushed the rofi-mkTarget-config branch from 8644c72 to 34e525e Compare July 4, 2025 17:35
@0xda157 0xda157 merged commit 606944b into nix-community:master Jul 4, 2025
6 checks passed
@0xda157 0xda157 deleted the rofi-mkTarget-config branch July 4, 2025 19:17
@stylix-automation
Copy link
Copy Markdown
Contributor

stylix-automation bot commented Jul 4, 2025

Successfully created backport PR for release-25.05:

stylix-automation bot pushed a commit that referenced this pull request Jul 4, 2025
@0xda157 @github-actions
rofi: fix mkTarget usage (#1593)
68b3e0c 
Fixes: dea0337 ("stylix: restrict access to config while using mkTarget (#1368)")
Fixes: 713f8da ("rofi: use mkTarget (#1550)")
(cherry picked from commit 606944b)
@stylix-automation stylix-automation bot mentioned this pull request Jul 4, 2025
Merged
0xda157 added a commit that referenced this pull request Jul 4, 2025
@0xda157
rofi: fix mkTarget usage (#1593)
bb6f349 
Fixes: dea0337 ("stylix: restrict access to config while using mkTarget (#1368)")
Fixes: 713f8da ("rofi: use mkTarget (#1550)")
(cherry picked from commit 606944b)
LunaCOLON3 pushed a commit to LunaCOLON3/stylix that referenced this pull request Jul 14, 2025
@0xda157 @LunaCOLON3
rofi: fix mkTarget usage (nix-community#1593)
b69a1be 
Fixes: dea0337 ("stylix: restrict access to config while using mkTarget (nix-community#1368)")
Fixes: 713f8da ("rofi: use mkTarget (nix-community#1550)")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@trueNAHO trueNAHO Awaiting requested review from trueNAHO

Assignees

No one assigned

Labels

topic: home-manager Home Manager target

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@0xda157