generated from cockpit-project/starter-kit
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Paste is broken as administrator #435
Comments
Also make sure that pasting does not overwrite existing files. |
jelly
added a commit
to jelly/cockpit-navigator
that referenced
this issue
May 16, 2024
When copy pasting pass the `--archive` flag to `cp` so permissions are retained. This also allows us to enable superuser try as permissions are now retained. Related: cockpit-project#435
jelly
added a commit
to jelly/cockpit-navigator
that referenced
this issue
May 21, 2024
When copy pasting pass the `--archive` flag to `cp` so permissions are retained. This also allows us to enable superuser try as permissions are now retained. As we don't have a file conflict dialog yet, don't allow overriding existing files by passing `--interactive` and `n`. Related: cockpit-project#435
jelly
added a commit
to jelly/cockpit-navigator
that referenced
this issue
May 28, 2024
When copy pasting pass the `--archive` flag to `cp` so permissions are retained. This also allows us to enable superuser try as permissions are now retained. As we don't have a file conflict dialog yet, don't allow overriding existing files by passing `--interactive` and `n`. Related: cockpit-project#435
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is no
{superuser: "try"}
in spawnPaste so copying as root fails. Also as we use -R we need to make sure we retain the original permissions and test for this.The text was updated successfully, but these errors were encountered: