Skip to content

use ${datadir} for default bash/zsh completion dirs - #424

Merged
N-R-K merged 3 commits into
resurrecting-open-source-projects:masterfrom
N-R-K:use-datadir
Mar 5, 2026
Merged

N-R-K merged 3 commits into
resurrecting-open-source-projects:masterfrom
N-R-K:use-datadir

Conversation

@N-R-K

@N-R-K N-R-K commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator

this respects the prefix and fixes make distcheck.

this respects the prefix and fixes `make distcheck`.
@N-R-K

N-R-K commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator Author

Interesting that the CI is still failing, it's working fine for me locally.

@daltomi

daltomi commented Mar 5, 2026 •

Copy link
Copy Markdown
Collaborator

I see some errors; in some cases it can't find tcc and in others it can't find tar:

debian-testing-tcc

E: Package 'tcc' has no installation candidate

alpine-latest-gcc
alpine-latest-clang
alpine-edge-clang

checking how to create a ustar tar archive... none

Some work but "cheat", that is, they don't run make distcheck, they just compile:

alpine-latest-bare-build

c99 -o src/scrot src/*.c $(pkg-config --cflags --libs ./deps.pc)

@N-R-K

N-R-K commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator Author

debian-testing-tcc

Seems like tcc was removed from debian testing: https://tracker.debian.org/pkg/tcc/news/

I've disabled it for now.

checking how to create a ustar tar archive... none

This is interesting... I've added explicit tar installation, let's see if that works or not.

@N-R-K

N-R-K commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator Author

Seems to be working. Aside from freebsd-clang which isn't starting for whatever reason.

the configure script now installs these files by default and
supports setting custom installation directories as well.
@N-R-K
N-R-K merged commit 4f6f69a into resurrecting-open-source-projects:master Mar 5, 2026
0 of 15 checks passed
@N-R-K
N-R-K deleted the use-datadir branch March 5, 2026 17:23
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