Skip to content

Don't just source lima.env; value might include spaces#98

Merged
AkihiroSuda merged 1 commit into
lima-vm:mainfrom
rancher-sandbox:spaces
Feb 11, 2023
Merged

Don't just source lima.env; value might include spaces#98
AkihiroSuda merged 1 commit into
lima-vm:mainfrom
rancher-sandbox:spaces

Conversation

@jandubois

Copy link
Copy Markdown
Member

Instead copy the code from Lima's boot.sh.

This is required to fix lima-vm/lima#933

Instead copy the code from Lima's boot.sh.

This is required to fix lima-vm/lima#933

Signed-off-by: Jan Dubois <jan.dubois@suse.com>
@jandubois jandubois requested a review from a team February 11, 2023 04:58
jandubois added a commit to jandubois/sshocker that referenced this pull request Feb 11, 2023
This is required because the names are sent as a single string over ssh.

This (together with lima-vm/alpine-lima#98)
will fix lima-vm/lima#933.

Signed-off-by: Jan Dubois <jan.dubois@suse.com>

@AkihiroSuda AkihiroSuda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit 3fc0281 into lima-vm:main Feb 11, 2023
@jandubois jandubois deleted the spaces branch February 11, 2023 19:09
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.

Mountpoints cannot include whitespace

2 participants