danth / stylix

System-wide colorscheming and typography for NixOS
https://stylix.danth.me/
MIT License
1.16k stars 143 forks source link

error while evaluating attribute 'kdeglobals' of derivation 'stylix-kde' #125

Closed TotalChaos05 closed 1 year ago

TotalChaos05 commented 1 year ago

Not sure if this is stylix, but it worked when i disabled it. full trace: https://pastebin.com/1Sj8gYk9 (not sure if i should paste the whole thing here)

it mentions kde at line 421, not sure whats happening with it.

arguablykomodo commented 1 year ago

Getting a very similar error. It seems that base16.nix is trying to parse a string with a " character as a hexadecimal number, but I'm not familiar enough with the codebase to properly track down the cause.

TotalChaos05 commented 1 year ago

@arguablykomodo neither base16.nix or base16-schemes have been updated recently. is it a nixpkgs issue? it seems to be an issue with setting a custom theme from base16-schemes what theme are you using?

TotalChaos05 commented 1 year ago

related issue https://github.com/SenchoPens/base16.nix/issues/4

arguablykomodo commented 1 year ago

I think i found the source of the error, it seems to stem from calls to config.lib.stylix.colors when using a yaml base16 theme which contains comments (for example, helios, gruvbox-dark-hard, and hardcore throw the error attribute '"' missing, while solarized-dark, gruber, and cupcake don't). There must be some issue in the yaml parsing code?

TotalChaos05 commented 1 year ago

i "fixed" it by adding base16-schemes = { url = github:TotalChaos05/base16-schemes; flake = false; }; to my flake inputs and defining the scheme with stylix.base16Scheme = "${base16-schemes}/catppuccin-mocha.yaml";

all i changed in my base16-schemes fork was removing all comments

gekoke commented 1 year ago

Duplicate of https://github.com/danth/stylix/issues/104

SenchoPens commented 1 year ago

Hi, maintainer of base16.nix here! I fixed the parsing of the strings with comments after them. Before @danth updates the flake inputs of stylix, you can try it out by adding stylix.inputs.base16.follows = "github:SenchoPens/base16.nix"; to your flake inputs.

@gekoke, this isn't a duplicate of #104, but I hope I fixed that one too, though I can't test it myself.

TotalChaos05 commented 1 year ago

@SenchoPens thank you !

danth commented 1 year ago

Stylix should be using the fixed version by default now :)