Skip to content

remove '/' /var/cache/packages#6

Open
slackjeff wants to merge 1 commit intorworkman:masterfrom
slackjeff:master
Open

remove '/' /var/cache/packages#6
slackjeff wants to merge 1 commit intorworkman:masterfrom
slackjeff:master

Conversation

@slackjeff
Copy link
Copy Markdown

I'm trying to fix this! this is our first step to remove '/' //var/cache/packages (laughs)

@rworkman
Copy link
Copy Markdown
Owner

Here's the trouble: are we absolutely certain that this will not cause a problem when e.g. ROOT=/somewhere (i.e. not /) because someone is running slackpkg to operate on a chroot environment?

@opty77
Copy link
Copy Markdown

opty77 commented Dec 30, 2023

What about unsetting ROOT if / at the very beginning and then using ${ROOT}${ROOT:+/} instead of ${ROOT}/?

Not perfect because of subsequent ${TEMP}, though.

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.

3 participants