-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Make the terminal page support opening with a different path #21021
Comments
This shouldn't disrupt/stop the current terminal if it is currently running something. My first idea how to do "busy?" detection: cockpit-project/cockpit-files#464 (comment) |
My recommendation: to unblock your work (for a few days), replace
with something like
and use cockpit.file() to read the pid. This is an utter hack which we can't land, but I think there's enough unrelated work to be done here in the meantime. In the next days I'll see to teach the stream channel about exposing its pid. |
Add the spawned PID to the channel's "ready" message. This will be useful for cockpit-project#21021
@ashley-cui see #21132 for adding a proper API to get the pid. |
Add the spawned PID to the channel's "ready" message. This will be useful for cockpit-project#21021
Add the spawned PID to the channel's "ready" message. This will be useful for #21021
Add the spawned PID to the channel's "ready" message. This will be useful for cockpit-project#21021
@martinpitt so this is now fixed right? :) |
Err yes, forgot to close it with the PR. |
Page: terminal
Cockpit-files has a feature request to open the terminal page in the current directory of files.
Currently the terminal page will default open the users home directory with a fallback to
/
. We want it to open a provided path for example via uri params, for example:https://cockpit.org/system/terminal#/?path=/home
The text was updated successfully, but these errors were encountered: