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

Makefile adjustments for packaging #117

Open
lmartinez-mirror opened this issue Sep 3, 2021 · 1 comment
Open

Makefile adjustments for packaging #117

lmartinez-mirror opened this issue Sep 3, 2021 · 1 comment

Comments

@lmartinez-mirror
Copy link

tl;dr: see #116

Hi, just wanted to suggest some changes for the Makefile to be more packager-friendly:

  • The Makefile installs LICENSE to a hard-coded directory despite the prefixes. I suggest removing the install for it altogether and let packagers handle it themselves. I understand you did this because you maintain cpufetch-git on the AUR, but this should ideally be done in the PKGBUILD, not in the Makefile.
  • LDFLAGS should be appended to compilation. Arch has guidelines for compiling and likes their binaries hardened with full RELRO; other packagers for other distros may have linker flags of their own that they may need to add.
  • Maybe install the man pages without compression? At least on Arch, any packages that install man pages automatically gzip them.
@Dr-Noob
Copy link
Owner

Dr-Noob commented Sep 3, 2021

Hi, thank you for your comments.

I've changed the Makefile so many times because packaging, and it is becoming really painful. Regarding your changes:

  • Understandable, I will change it. When I read how to create a package I thought that the best approach was to include everything inside the install target of the Makefile, but now I agree that it is not since not there are packages for other distros, not only Arch.
  • Understandable again but not very clear in the documentation. Here talks about LDFLAGS, but does not explicitly say that one should allow passing LDFLAGS. Maybe this is explained on another page, but I didn't find it. I will change it too.
  • Do you mean to allow non-compressed man pages in other package management systems? Does it make sense? If I should change the entire install target in the Makefile maybe it's a better idea to leave it as it is for Arch and expect other package managers to do their own install process instead of using make install.

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

No branches or pull requests

2 participants