Skip to content

moved environment variables #3476

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

Merged
merged 1 commit into from
May 7, 2025
Merged

Conversation

kairveeehh
Copy link
Contributor

fixes #3459

```sh
export QEMU_SYSTEM_ARM=/usr/local/bin/qemu-system-arm
```

Copy link
Member

Choose a reason for hiding this comment

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

There are also riscv64 and s390x

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will add and push.

Copy link
Member

Choose a reason for hiding this comment

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

@AkihiroSuda AkihiroSuda added this to the v1.1.0 milestone Apr 30, 2025
@AkihiroSuda AkihiroSuda added the documentation Improvements or additions to documentation label Apr 30, 2025
@@ -107,3 +117,48 @@ This page documents the environment variables used in Lima.
```
- **Note**: It is an experimental setting and has no guarantees being ever promoted to stable. It may be removed
or changed at any stage of project development.

### `QEMU_SYSTEM_X86_64`
Copy link
Member

Choose a reason for hiding this comment

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

I know this PR is just moving stuff around, but I'm wondering if the documentation should somehow explain that the arch setting in lima.yaml determines which environment variable is being used, and that armv7l is a special case, that is mapped to just arm?

Copy link
Contributor Author

@kairveeehh kairveeehh Apr 30, 2025

Choose a reason for hiding this comment

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

yes , I would like to do that , just followed the description as of now but sure !
Should I add it as a note format below the variables or some other suggested way is to be followed?

export QEMU_SYSTEM_X86_64=/usr/local/bin/qemu-system-x86_64
```

### `QEMU_SYSTEM_AARCH64`
Copy link
Member

Choose a reason for hiding this comment

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

I think the variables should be listed in alphabetical order.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do the changes , thanks

@AkihiroSuda
Copy link
Member

Please squash the commits

@AkihiroSuda
Copy link
Member

ping @kairveeehh 🙏

@kairveeehh
Copy link
Contributor Author

ping @kairveeehh 🙏

On it by today.

Signed-off-by: Kairvee Vaswani <imt_2023037@iiitm.ac.in>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

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 fb5057f into lima-vm:master May 7, 2025
9 checks passed
@kairveeehh
Copy link
Contributor Author

Thanks

sorry for delaying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

website: consolidate dev/internals/#environment-variables into config/environment-variables
3 participants