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

The scripts alter bashrc breaking everything else. #10

Open
rkouznetsov opened this issue Nov 4, 2024 · 2 comments
Open

The scripts alter bashrc breaking everything else. #10

rkouznetsov opened this issue Nov 4, 2024 · 2 comments
Assignees

Comments

@rkouznetsov
Copy link

setting LDFLAGS and LD_LIBRARY flags from .bashrc is a very bad idea.
So is setting path to outdated and trimmed versions of nco and netcdf from WRF packages.

If you do so, please put a big red note into your README.md, that everything but WRF will probably stop working for the user after running the script.

Few of our users destroyed their environment with the scripts. Yes, i know that , it is not a good idea to run random scripts without reading them. Some learn it hard way, some do not learn and do it repeatedly....

Thank you!

@bakamotokatas
Copy link
Owner

Hi @rkouznetsov,

Thank you for your feedback. I’m sorry to hear that you encountered issues. I will think solutions for the problem. This is actually the first time in 6 years that someone has reported this problem.

My main goal with this script was to help beginners, which is why some parts are written with them in mind. Although the code installs and uses the system netcdf, it does not install nco.

Thanks.

@bakamotokatas bakamotokatas self-assigned this Nov 27, 2024
@rkouznetsov
Copy link
Author

Thank you and sorry for accusing the scripts to install netcdf and nco. Indeed, in our case it was some other version of the installer. Yet, setting user-specific environment (if you really need it) should be done from .bash_profile rather than .bashrc , and application-specific settings are better done from wrapper scripts for the application.

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

2 participants