Skip to content
This repository has been archived by the owner. It is now read-only.

overmount bei mxonastick vorhanden #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

overmount bei mxonastick vorhanden #1

wants to merge 1 commit into from

Conversation

david
Copy link

@david david commented Sep 12, 2016

problem war $V{DEFAULT_MOUNT_PREFIX}/ == /amd/SHORTHOST

problem war $V{DEFAULT_MOUNT_PREFIX}/ == /amd/SHORTHOST
@donald
Copy link
Contributor

donald commented Feb 16, 2017

if ($mp->{mountpoint} eq "/amd/$hostname/0") { is ugly but
if ($mp->{mountpoint} eq expand_variables("$V{DEFAULT_MOUNT_PREFIX}/0")) { is worse and
if ($mp->{mountpoint} eq expand_variables(expand_variables('DEFAULT_MOUNT_PREFIX/0'))) { even more so.
I'd vote to remove this config variable stuff from completely and put the strings into the script everywhere. Much more understandable. And the script uses hardcoded configuration anyway (''X','C','M' ..)

@donald
Copy link
Contributor

donald commented Feb 4, 2023

@david: Reminder: We already identified the reason

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants