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

Installation of completions doesn't use DISTDIR #335

Closed
alerque opened this issue Dec 19, 2023 · 8 comments · Fixed by #336
Closed

Installation of completions doesn't use DISTDIR #335

alerque opened this issue Dec 19, 2023 · 8 comments · Fixed by #336

Comments

@alerque
Copy link
Contributor

alerque commented Dec 19, 2023

Having completions installed in nice, and the autoconf setup for configuring the locations seems fine. However the actual installation is broken for distro packagers:

cd $(bashcompletiondir) && \

This assumes you are installing directly to the host system and that these directories existing. Most distro package builds use DESTDIR= or similar mechanisms to install to an isolated location, then package up those files. This scripting ignores DESTDIR entirely.

@alerque
Copy link
Contributor Author

alerque commented Dec 19, 2023

...this issue partially misdiagnoses the issue because the directories do get created somewhere first, but these ln -s actions are still failing for some reason. I'm looking for the actual reason why.

@alerque alerque changed the title Installation makes incorrect assuption about directories existing Installation of completions doesn't use DISTDIR Dec 19, 2023
@alerque
Copy link
Contributor Author

alerque commented Dec 19, 2023

Updated issue with the actual problem.

@alerque
Copy link
Contributor Author

alerque commented Dec 19, 2023

Since this is going to affect all the other distros out of there and annoy send every packager on a goose chase to find either the problem or this issue, it might be worth doing a quick patch release before many of them get around to the bump to make downstream lives easier.

@andreasstieger
Copy link
Contributor

Meh, openSUSE is watching. Packagers normally know to check recently reported/closed issues and git changes.

@genivia-inc
Copy link
Member

I understand. No worries. The v4.4.1 patch update will be released very soon today.

genivia-inc added a commit that referenced this issue Dec 19, 2023
fix installation of completions #335
@barsnick
Copy link
Contributor

Since this is going to affect all the other distros out of there and annoy send every packager on a goose chase to find either the problem or this issue, it might be worth doing a quick patch release before many of them get around to the bump to make downstream lives easier.

Don't worry. I package ugrep myself for Fedora et.al. (because Fedora has fallen asleep), and figured out this problem and the fix within five minutes. You beat me to a pull request.

BTW, advertizing: My package updates to latest, packages completions (separately), packages ugrep-indexer, ... :-)

@carlwgeorge
Copy link
Contributor

@barsnick Package maintainers have day jobs and personal lives, so from time to time a package may lag behind upstream. This doesn't mean that they have "fallen asleep", and it's not nice to claim that's the case. When you notice a package in this state, it's more helpful to send a pull request to update the spec file than it is to advertise your own separate repo. Even better, consider becoming a package maintainer yourself to help with this work more directly.

@genivia-inc
Copy link
Member

@carlwgeorge Well said. Package maintainers are busy with their own lives and jobs. Same for many open source developers. Kudos to those who volunteer their time to help! I sincerely hope that package managers receive some acknowledgements and perhaps compliments for their work. I can imagine it can be stressful at times, especially when urgency is requested for critical patches etc.

I try not to release in the weekends or on Fridays. But time management for one person doesn't always align with someone else's time management. I usually have time on Fridays and weekends to get things done that aren't part of my regular job that pays my bills.

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 a pull request may close this issue.

5 participants