aixPb: Put /usr/bin in front of PATH#3716
Conversation
sxa
left a comment
There was a problem hiding this comment.
Why is /etc in there? I can't think of anything we need in the PATH that would be in there
|
Not sure. I'll remove it |
sxa
left a comment
There was a problem hiding this comment.
We should also bear in mind that while this will be ok for new machines it likely won't change the existing machines which have PATH=/opt/freeware/bin at the start as most machines currently won't match this regex, so if we don't put another case in here for that we'll need to look at patching the existing machines up (as described in #2288 (comment)). Also noting that this regex to update the config will trigger on every playbook execution since it sets the path to something that has PATH=/usr/bin at the start. Something like @aixtools' proposal in #2288 (review) may be preferable, but I'm also ok with a one-off manual fixup.
holding off merging until we raise any appropriate work items here for consistency. |
|
I've manually updated the PATH on the current machines to start with |
To escape the rebase hell that is #2288, ive simply created a new pr :)