search

danth / stylix

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

doesn't work with latest tinted-theming #571

Closed nazarewk closed 2 weeks ago

nazarewk commented 2 months ago

Not sure what is going on here, but seems to be some changed standards and missing coordination between projects?

trueNAHO commented 2 months ago

Not sure what is going on here

What exactly is the problem?

It seems that https://github.com/tinted-theming renamed some of their repositories. I will submit a Stylix PR for this.

nazarewk commented 2 months ago

they also:

trueNAHO commented 2 months ago

Until Stylix updates this input, this should not be a problem already, right?

trueNAHO commented 2 months ago

@nazarewk, I have not looked into this yet. Could you clarify what exactly is the problem with the current tinted-foot verision?

danth commented 1 month ago

572 has been merged; has this improved the situation?

nazarewk commented 1 month ago

572 didn't change anything, but updating tinted-foot to latest will result in:

trace: Warning (base16.nix): failed to parse YAML file without an IFD.
trace: Warning (base16.nix): A config.yaml does not follow base16 conventions or was incorrectly parsed.
Defaulting to an empty extension.
Please consult https://github.com/SenchoPens/base16.nix/tree/main#%EF%B8%8F-troubleshooting
trace: Warning (base16.nix): failed to parse YAML file without an IFD.
trace: Warning (base16.nix): A config.yaml does not follow base16 conventions or was incorrectly parsed.
Defaulting to an empty extension.
Please consult https://github.com/SenchoPens/base16.nix/tree/main#%EF%B8%8F-troubleshooting
base16-nix-parse-check> Running phase: checkPhase
base16-nix-parse-check> Output of "jd /nix/store/w0j2i02sfmpl19jmnprrd5ll4l068yj0-fromYAML /nix/store/nzxhrh19vbjzsnz1wb896w45y0wfdjnm-parsed-yaml-as-json":
base16-nix-parse-check> @ ["base16","supported-systems"]
base16-nix-parse-check> - ["base16"]
base16-nix-parse-check> + "[base16]"
base16-nix-parse-check> @ ["base24","supported-systems"]
base16-nix-parse-check> - ["base24"]
base16-nix-parse-check> + "[base24]"
base16-nix-parse-check> Error (base16.nix): /nix/store/rqh827dzqi5n11vwfv21gggwaaf24338-source/templates/config.yaml was parsed incorrectly during nix evaluation.
base16-nix-parse-check> Please consult https://github.com/SenchoPens/base16.nix/tree/main#%EF%B8%8F-troubleshooting
home-manager-path> created 4071 symlinks in user environment

In https://github.com/tinted-theming/tinted-foot/issues/8 I tried fixing the first part (parsing) by modifying the yaml file in tinted-foot, but the structure still prevents from successfully building stylix config.

aacebedo commented 1 month ago

updated stylix today and having the same issue with nord now with kitty and tmux

trueNAHO commented 1 month ago

updated stylix today and having the same issue with nord now with kitty and tmux

Is this caused by https://github.com/danth/stylix/pull/588?

aacebedo commented 1 month ago

updated stylix today and having the same issue with nord now with kitty and tmux

Is this caused by #588?

it seems. I reverted stylix to 63426a59e714c4389c5a8e559dee05a0087a3043 and I can complie

trueNAHO commented 1 month ago

I am already looking into this. I will try to properly resolve this today.

aacebedo commented 1 month ago

Thanks!

trueNAHO commented 1 month ago

I am already looking into this. I will try to properly resolve this today.

Feel free to test https://github.com/danth/stylix/pull/589 and leave a comment on whether it works.

noahfraiture commented 1 month ago

Hello, I have the same issue even on https://github.com/danth/stylix/commit/63426a59e714c4389c5a8e559dee05a0087a3043 However this is my first time using nixos, I might have done something wrong. This is how I declare it in flake.nix stylix.url = "github:danth/stylix?rev=63426a59e714c4389c5a8e559dee05a0087a3043"; And I try to use it simply by setting enable to true and give it an image

aacebedo commented 1 month ago

I am already looking into this. I will try to properly resolve this today.

Feel free to test #589 and leave a comment on whether it works.

I confirm it works