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

change remove_path dependency to MainControl.quiet #670

Closed

Conversation

francinelapid
Copy link

@francinelapid francinelapid commented Sep 26, 2023

Hello,

We're seeing an issue with version 8.7.32 where we're unable to module load because of a remove_path line in the module file.

Lmod has detected the following error: Unable to load module because of error
when evaluating modulefile:
     ....lua:
...Lmod-8.7.32/src/MainControl.lua:339: attempt to
call a nil value (method 'trim')
     Please check the modulefile and especially if there is a line number
specified in the above message 
While processing the following module(s):

It seems to be caused by remove_path inheriting from MainControl.setenv_env. Because of this, a table is being passed as the first setenv_env parameter when remove_path is called.

@francinelapid francinelapid changed the title change remove_path dependency to change remove_path dependency to MainControl.quiet Sep 26, 2023
@francinelapid francinelapid changed the title change remove_path dependency to MainControl.quiet change remove_path dependency to MainControl.quiet Sep 26, 2023
@rtmclay
Copy link
Member

rtmclay commented Dec 4, 2023

This pull request has been merged into the testing branch. Please try the Lmod testing branch to see if it works for you.

@rtmclay
Copy link
Member

rtmclay commented Jan 8, 2024

O.K to close this PR?

@Paul-Ferrell
Copy link

Looks like this is in as of .33, so we're all good here. Thanks!

@rtmclay rtmclay closed this Feb 2, 2024
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