Skip to content
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

Move to meson #1224

Open
apprehensions opened this issue Nov 2, 2023 · 4 comments · May be fixed by #1226
Open

Move to meson #1224

apprehensions opened this issue Nov 2, 2023 · 4 comments · May be fixed by #1226
Labels

Comments

@apprehensions
Copy link
Contributor

Hi @fwsmit I just to hear your thoughts for moving Dunst to a better build system such as Meson. The current makefile is using alot of GNUisms and isn't that portable and readable for that matter.

I would like to try to give this a shot.

@apprehensions
Copy link
Contributor Author

This can also fix alot of consistencies, as this project is featureful and aims for both wayland and x11.

@fwsmit
Copy link
Member

fwsmit commented Nov 2, 2023

Yeah, go for it!

This can also fix alot of consistencies, as this project is featureful and aims for both wayland and x11.

What inconsistencies have you noticed with the current setup?

@apprehensions
Copy link
Contributor Author

Unfortunately, i am unable to have the current test framework moved to Meson (valgrind is a maybe). This is mostly because of how it is designed (run this program called 'test').

I am unsure what to do.

@apprehensions
Copy link
Contributor Author

For the time being i will immediately give up due to the complexity of the tests and makefile as a whole:

I had underestimated how big this project is, which is why i initially thought this would be quite easy, as i had figured dunst is this simple notification daemon program.

Could you try atleast making it smaller?

@fwsmit fwsmit linked a pull request Jan 1, 2024 that will close this issue
@fwsmit fwsmit added the Feature label Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants