Skip to content

Updated SETTING_PATH.md for changes to affect all users #176

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

brandonp42
Copy link

This is just a minor update to the troubleshooting/SETTING_PATH.md file. On recent operating system releases the /QOpenSys/etc/profile should not be modified directly - local customizations should be in profile.local.

Also added instructions to adjust the system environment variable PASE_PATH as an alternative so that programs called directly via QP2SHELL will be affected.

Thanks!
Brandon

@brandonp42
Copy link
Author

FYI in my testing when I SSH to the IBM i, it doesn't seem to use the PASE_PATH system variable and you still want/need to adjust via the /QOpenSys/etc/profile.local or user profile files. I'd be interested to know if other people see the same behaviour or not.

Copy link
Member

@kadler kadler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Just a couple suggestions to improve wording and prevent a warning about untyped code blocks.

Co-authored-by: Korinne Adler <[email protected]>
Signed-off-by: Brandon Peterson <[email protected]>
@brandonp42 brandonp42 force-pushed the update-troubleshooting-path branch from c1485c7 to 33a603e Compare January 29, 2025 16:10
@brandonp42
Copy link
Author

Thank you for the suggestions @kadler!

@kadler
Copy link
Member

kadler commented Jan 29, 2025

We may want to add a note that profile.local doesn't work unless you are on certain releases and PTF levels, but I don't have that information available at the moment.

@brandonp42
Copy link
Author

I'll update the instructions more this weekend to explain older vs newer releases. I don't know what the cutoff is either but I can give instructions on what to look for to decide what to do.

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.

2 participants