-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add configuration file evaluation #49
Comments
Regarding the first requirement, would an implementation similar to StumpWM's load-rc be okay? |
Ya, that should work for now. I'd like to get rid of the old-school .stumpwmrc and .stumpwm.d style configuration locations, or at the very least not have them be prioritized over anything in the XDG conf directory. My main concern would be error handling. I guess I'd try to eval the file before starting the rendering loop, and just crash if there's an error? |
Perfect! Also, I just saw this PR which seems to be addressing this, so I'll follow the progress there. |
@Arian-D I haven't had the time to finish completing all the issues brought up in the PR review and Im not sure when I will so Im happy for you to take over from where that pr is at the moment if you want. I think from memory I have fixed the accidental indentation of code around it and removing the format calls. |
Thanks @tpine! I'll try to have a look this weekend, and will see how to go about it. |
In Stumpwm, users are able to have initialization code ran at startup so they can customize the WM's behavior. We need the same thing in Mahogany.
Requirements for a complete implementation:
Like with
defcommand
, not all of this needs to be implemented at once.The text was updated successfully, but these errors were encountered: