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

"source" not available in all systems #4260

Closed
rljacob opened this issue May 25, 2022 · 4 comments · Fixed by #4263
Closed

"source" not available in all systems #4260

rljacob opened this issue May 25, 2022 · 4 comments · Fixed by #4263
Assignees
Labels

Comments

@rljacob
Copy link
Member

rljacob commented May 25, 2022

env_mach_specific.py has this line:
source_cmd = "source {} && ".format(init_path)

Turns out "source" is not available in Ubuntu. Most systems link "/bin/sh" to bash but Ubuntu links it to "dash" which is faster and does not define the "source" command.

Solution is to use "." which is POSIX-standard command for executing commands from a file.
source_cmd = ". {} && ".format(init_path)

Thanks to @dqwu who found this and the solution.

@rljacob
Copy link
Member Author

rljacob commented May 25, 2022

Grepping the CCS for "source" is fun but I don't think there's anywhere else this is used.

@jasonb5
Copy link
Collaborator

jasonb5 commented Jun 2, 2022

@rljacob I'll make this update and double check it's not needed anywhere else.

@rljacob
Copy link
Member Author

rljacob commented Jun 2, 2022

I think I saw @sarich push a branch with some fixes.

@rljacob
Copy link
Member Author

rljacob commented Jun 2, 2022

Nevermind, that was part of some other development. Go ahead @jasonb5

jedwards4b added a commit that referenced this issue Jun 30, 2022
Fixes source command


Changes source command from source to . for sh shell, leaves
source for csh.

Test suite: scripts_regression_tests.py
Test baseline: n/a
Test namelist changes: n/a
Test status: n/a

Fixes #4260
User interface changes?: N
Update gh-pages html (Y/N)?: N
billsacks added a commit to ESCOMP/CTSM that referenced this issue Jul 18, 2022
This PR includes a number of minor fixes and updates associated with FATES.
A duplicate filter for all soils that was introduced during the the initial
implementation of FATES-SP mode is removed.  A small set of FATES history
variable names have been updated necessitating an update to the testmod
user namelists.  The ccs_config and cime externals have been updated to
include the FATES development workstation, lobata.  The cime tag update
is necessary to include a fix enabling support for module initilization
on certain linux distributions (ESMCI/cime#4260).
Finally, a minor fix was included to the initialization of the fates_levcdam
dimension that was introduced with the last fates api update.

Fixes #1794

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

Successfully merging a pull request may close this issue.

2 participants