Skip to content

bootstrap: sanity check install prefix and sysconfdir #113580

Closed

Description

In the followup discussion on #110906, we discovered that their original problem was because they had chosen a path under their home directory for [install] prefix, but they had not changed the default sysconfdir = "/etc". So most things installed fine, but they didn't have permission to write /etc/bash_completion.d/cargo.

Bootstrap could sanity check this when starting an install, seeing that prefix is writable by the current user but the absolute sysconfdir is not. (A relative sysconfdir goes under prefix, which should be fine.) This should also take DESTDIR into account as a further prefix on all installation paths.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

C-enhancementCategory: An issue proposing an enhancement or a PR with one.Category: An issue proposing an enhancement or a PR with one.T-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions