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.

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

"Please ensure Windows is powered on" wrong help text #213

Closed
reenberg opened this issue Aug 14, 2024 · 0 comments · Fixed by #232
Closed

"Please ensure Windows is powered on" wrong help text #213

reenberg opened this issue Aug 14, 2024 · 0 comments · Fixed by #232
Labels
bug Something isn't working legacy Using legacy winapps

Comments

@reenberg
Copy link

I mistakenly used the compose.yml file as described by the dokur/windows project, and thus didn't have the correct container name set to WinApps, which makes the install script fail with the following message:

ERROR: CONTAINER NOT RUNNING.
Windows is not running.
--------------------------------------------------------------------------------
Please ensure Windows is powered on:
podman-compose --file ~/.config/winapps/winapps.conf start
--------------------------------------------------------------------------------

Clearly it is wrongly using the winapps.conf filename in the compose command. also if the container is not running, chances are it should be created with up -d instead of just a start command, at least as long as 'stoped' is not returned by the container status a few lines above:

CONTAINER_STATE=$("$WAFLAVOR" ps --all --filter name="WinApps" --format '{{.Status}}')
@oskardotglobal oskardotglobal added bug Something isn't working legacy Using legacy winapps labels Aug 18, 2024
@oskardotglobal oskardotglobal linked a pull request Aug 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working legacy Using legacy winapps
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants