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

cloe-launch: status indication upon first opening of the shell #219

Open
clonker opened this issue Feb 8, 2024 · 3 comments
Open

cloe-launch: status indication upon first opening of the shell #219

clonker opened this issue Feb 8, 2024 · 3 comments
Labels
c-launcher Related to component: cloe-launch

Comments

@clonker
Copy link

clonker commented Feb 8, 2024

when opening a cloe-launch shell for the very first time, it may take a prolonged period of time before anything happens. i have had several users report back to me that they thought the program hangs although it was just downloading conan dependencies. is there a possibility to add progress output?

@cassava cassava added the c-launcher Related to component: cloe-launch label Feb 23, 2024
@cassava
Copy link
Contributor

cassava commented Apr 29, 2024

The shell command was not intended to show any output unless an error occurs.
By default, it does not build any packages, however it may download them.
Ideally, the user will first use cloe-launch prepare and then follow up with cloe-launch shell.
However, how are users to know its unintended usage if we don't mention it anywhere.

For a great UX we need to take expected incorrect usage into account, so I agree that we need to improve this here. Since I don't know what conan is doing, adding a progress bar is difficult.

I'll look into providing transient Conan output via the rich Python library.

@clonker
Copy link
Author

clonker commented Apr 29, 2024

Worst case it may be enough to just periodically print some message ("hang in there, it may take a few minutes") after the first 10 seconds have passed - that way, there is feedback that the process hasn't stalled out but is indeed doing something in the background. :)

cassava added a commit that referenced this issue May 6, 2024
This is important to let the user know that something is happening,
if for example conan is doing something that takes a long time.

This commit resolves eclipse/cloe issue #219.
cassava added a commit that referenced this issue May 6, 2024
This is important to let the user know that something is happening,
if for example conan is doing something that takes a long time.

This commit resolves eclipse/cloe issue #219.
cassava added a commit that referenced this issue May 6, 2024
This is important to let the user know that something is happening,
if for example conan is doing something that takes a long time.

This commit resolves eclipse/cloe issue #219.
@cassava cassava added this to the 0.24.0 milestone May 6, 2024
cassava added a commit that referenced this issue May 6, 2024
This is important to let the user know that something is happening,
if for example conan is doing something that takes a long time.

This commit resolves eclipse/cloe issue #219.
@cassava cassava closed this as completed May 6, 2024
@cassava cassava modified the milestones: 0.24.0, 0.25.0 May 6, 2024
@cassava
Copy link
Contributor

cassava commented May 6, 2024

Commit had to be reverted because output ordering was not working.

@cassava cassava reopened this May 6, 2024
@cassava cassava removed this from the 0.25.0 milestone Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-launcher Related to component: cloe-launch
Projects
None yet
Development

No branches or pull requests

2 participants