emersion / kanshi

Dynamic display configuration (mirror)
https://wayland.emersion.fr/kanshi/
MIT License
652 stars 46 forks source link

Add '!include' statement #68

Closed ghost closed 3 years ago

ghost commented 4 years ago

Replaces #53. Could use a check on the path resolution logic to see if it's correct.

emersion commented 4 years ago

Right now profile names make this ambiguous (a profile could be called !include). Potential solutions:

emersion commented 4 years ago

Add a mandatory profile prefix: profile my-profile { … }

I think I'm going to go with this solution. See https://github.com/emersion/kanshi/issues/75

emersion commented 4 years ago

We should now be able to use the top-level include directive safely.

emersion commented 3 years ago

Superseded by #87