Skip to content

build: Install bash-completion relative to datadir#2014

Merged
superm1 merged 1 commit intofwupd:masterfrom
jtojnar:bash-completion-2.10
Apr 16, 2020
Merged

build: Install bash-completion relative to datadir#2014
superm1 merged 1 commit intofwupd:masterfrom
jtojnar:bash-completion-2.10

Conversation

@jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Apr 16, 2020

Type of pull request:

Since bash-completion 2.9, it was no longer possible to override
the completionsdir through prefix. 1 In 2.10, the overridability
was re-estabilished but this time through datadir variable. 2

This should not really matter except for developers installing the project
into a custom prefix or distros using per-package prefixes like NixOS.

Tested on NixOS with bash-completion 2.9 with the offending patch 1 reverted and with 2.10.

Since bash-completion 2.9, it was no longer possible to override
the completionsdir through prefix. [1] In 2.10, the overridability
was re-estabilished but this time through datadir variable. [2]

This should not really matter except for developers installing the project
into a custom prefix or distros using per-package prefixes like NixOS.

[1]: scop/bash-completion@81ba2c7
[2]: scop/bash-completion#344
@jtojnar jtojnar mentioned this pull request Apr 16, 2020
10 tasks
@superm1 superm1 merged commit 0f03501 into fwupd:master Apr 16, 2020
@jtojnar jtojnar deleted the bash-completion-2.10 branch April 16, 2020 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants