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

Improve build #44

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Improve build #44

wants to merge 6 commits into from

Conversation

NathanBnm
Copy link
Contributor

I improved a little the way your app is build with meson.

I split the meson.build files within the different folders for more clarity. I also simplified the post_install.py script and added the .editorconfig with elementary recommended parameters.

Finally I moved the gresources to a better location in icons/symbolic.

This PR is ready for review

@ryonakano
Copy link
Contributor

The CI complains about the permission error of the postinst script:

meson.build:42:6: ERROR:  Script or command 'meson/post_install.py' not found or not executable

I guess you need to chmod the permission of meson/post_install.py.

@NathanBnm
Copy link
Contributor Author

You're absolutely right, it happens sometimes. I'm going to fix this during the day !

@NathanBnm NathanBnm requested a review from Mohelm97 January 18, 2020 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants