-
-
Notifications
You must be signed in to change notification settings - Fork 113
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
Cleanup build system #191
Cleanup build system #191
Conversation
Test building now... |
Yes this will be prepended to every install() command (cmake) |
The CMAKE_INSTALL_PREFIX is not being used as you describe above...
And I ended up with some files in With those build options here is the full file list file list
Note that besides the problem with the base paths, there is also a lot of cruft in here that should NOT be installed at all. Notably stuff that will conflict with other system packages like everything in |
Can you remove the slash in front of share? I think it will otherwise be interpreted as absolute path. |
Either the change you made or removing the leading slash did change where most of the resources went and they are in |
what is the path of "$pkgdir"? |
The This is quite typical of distro packaging and the typical usage of things like DESTDIR vars for make/ninja/etc.
Here is one I am using for testing:
It should be compiled using |
That PKGBUILD will be a little hard to test build without the expected desktop file and other resources next to it. Here is a fork of the AUR package I'm working with to test this: https://github.com/alerque/gittyup-git |
But you can rename the desktop file from the repository or? |
Don't worry about the desktop file, I'll look into using the one from the repo. I don't think it existed when this was written up. I'll massage the details later, just the build & install steps need to do roughly the right thing. Right now with everything going to bare |
Another thing to be on the lookout for: in the current |
This one was taken over by @exactly-one-kas with #314 which was merged. The translation topic shall be done in another one. All of this changes are already in the current master. |
fixes #167