Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

installation in specified prefix doesn't work, tries to create '/usr/local/Modules' #348

Closed
boegel opened this issue Jul 6, 2020 · 2 comments
Labels
Milestone

Comments

@boegel
Copy link

boegel commented Jul 6, 2020

Describe the bug

Installing in a specified prefix fails, because "make install" tries to create /usr/local/Modules:

$ ./configure --prefix /tmp/example
$ make install
...
mkdir -p /usr/local/Modules/libexec
mkdir: cannot create directory ‘/usr/local/Modules’: Permission denied
make: *** [install] Error 1

This triggers are Permission denied because I'm using a non-admin user account.

To Reproduce

Steps to reproduce the behavior:

$ tar xfvz modules-4.5.1.tar.gz
$ cd modules-4.5.1
$ ./configure --prefix /tmp/example
$ make install
...

Expected behavior

Installation to specified prefix works.

Error and debugging information

$ make install
mkdir -p /usr/local/Modules/libexec
mkdir: cannot create directory ‘/usr/local/Modules’: Permission denied
make: *** [install] Error 1
...

Modules version and configuration

Modules 4.5.1, 4.5.0, 4.4.1

Additional context

(none)

@boegel boegel added the bug label Jul 6, 2020
@boegel
Copy link
Author

boegel commented Jul 6, 2020

I've danced around this issue by replacing all occurences of /usr/local/Modules with my preferred prefix in Makefile.inc after running configure.

@xdelaruelle
Copy link
Member

Thanks for your detailed report @boegel.

--prefix value should be given in the --arg=value form. So here there are 2 issues:

  • an error should be raised if an unknown argument is used, ./configure cannot silently ignore it
  • --arg value form seems commonly used so it should also be supported

@xdelaruelle xdelaruelle added this to the 4.6 milestone Jul 7, 2020
xdelaruelle added a commit that referenced this issue Jul 7, 2020
Enable the definition of the installation directory argument of the
./configure script with the `--arg val` syntax form in addition to the
`--arg=val` syntax.

Fixes #348
@xdelaruelle xdelaruelle modified the milestones: 4.6, 4.5.2 Jul 29, 2020
xdelaruelle added a commit to xdelaruelle/modules that referenced this issue Jul 29, 2020
xdelaruelle added a commit to xdelaruelle/modules that referenced this issue Jul 29, 2020
Enable the definition of the installation directory argument of the
./configure script with the `--arg val` syntax form in addition to the
`--arg=val` syntax.

Fixes cea-hpc#348
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants