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

cylc job script to use "bash -l" instead of sourcing profiles #2424

Closed
matthewrmshin opened this issue Sep 11, 2017 · 0 comments
Closed

cylc job script to use "bash -l" instead of sourcing profiles #2424

matthewrmshin opened this issue Sep 11, 2017 · 0 comments
Assignees
Milestone

Comments

@matthewrmshin
Copy link
Contributor

It is only legacy reason that we have our job scripts sourcing profiles instead of calling the login shell (#!/usr/bin/bash -l or #!/usr/bin/ksh -l).

From discussion with @hjoliver via email:

Our current use of /etc/profile and .profile is technically incorrect ... Fixing it could result in a change in behavior for users with a .bash_profile who configured stuff for cylc in .profile just because they know our job scripts source it. But we could warn about that in next release.

@matthewrmshin matthewrmshin added this to the soon milestone Sep 11, 2017
@matthewrmshin matthewrmshin self-assigned this Sep 11, 2017
@matthewrmshin matthewrmshin modified the milestones: soon, next release Sep 29, 2017
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

No branches or pull requests

1 participant