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.

Sign up for GitHub

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 root path on centos has no /usr/local/bin #345

Merged
merged 2 commits into from
Oct 19, 2021
Merged

The root path on centos has no /usr/local/bin #345

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d6771a5
The root path on centos has no /usr/local/bin
afbjorklund Oct 18, 2021
f87b176
Need path for containerd-rootless-setuptool.sh
afbjorklund Oct 18, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pkg/cidata/cidata.TEMPLATE.d/boot/25-guestagent-base.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,5 +38,5 @@ else
# Remove legacy systemd service
rm -f "/home/${LIMA_CIDATA_USER}.linux/.config/systemd/user/lima-guestagent.service"

sudo lima-guestagent install-systemd
sudo /usr/local/bin/lima-guestagent install-systemd
Copy link
Member

Choose a reason for hiding this comment

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

Can we rather modify $PATH globally?

containerd installer needs /usr/local/bin too

Copy link
Member Author

Choose a reason for hiding this comment

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

Not really sure why containerd doesn't use /usr, but maybe it is related to not using packages ?

Copy link
Member Author

Choose a reason for hiding this comment

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

For some reason it seems like the CentOS containers have it in the path, but not the CentOS VMs ? Weird.

Copy link
Member Author

Choose a reason for hiding this comment

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

It is actually sudo that is dropping it, if you log in as root then you have the /usr/local in there as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope, that was only /usr/local/sbin - there is still no /usr/local/bin in the PATH for root. Only /root/bin.

fi