-
Notifications
You must be signed in to change notification settings - Fork 57
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
Fail to move when using non-EN locale #49
Comments
Maybe we can use |
strace available everywhere?
Might need to progressively enhance
…On Wed, 5 Mar 2025 at 10:51 pm, SNRainiar ***@***.***> wrote:
Maybe we can use strace mv $FROM $TO 2>&1 | grep 'EPERM\|EACCES' to check
whether move failed by operation not permitted or permission denied,
instead of just checking the output string.
—
Reply to this email directly, view it on GitHub
<#49 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAE2X47UYAEQLMVH5CKY7E32S3QN5AVCNFSM6AAAAABYL32QICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBQG4YTCMZQHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
[image: SamuNatsu]*SamuNatsu* left a comment (jpillora/installer#49)
<#49 (comment)>
Maybe we can use strace mv $FROM $TO 2>&1 | grep 'EPERM\|EACCES' to check
whether move failed by operation not permitted or permission denied,
instead of just checking the output string.
—
Reply to this email directly, view it on GitHub
<#49 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAE2X47UYAEQLMVH5CKY7E32S3QN5AVCNFSM6AAAAABYL32QICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBQG4YTCMZQHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Not really, for some other distributions it should be installed manually. Another solution is to use ‘ls‘ and username to check whether user is the owner of target folder. Naive but less dependencies. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Lines here only works on English locales:
installer/scripts/install.sh.tmpl
Line 170 in 746f746
If the user uses other locales (e.g. Chinese), there's no
Permission denied
string in the output, instead a权限不够
.So maybe we should try other methods to check whether move fails by permission denied.
The text was updated successfully, but these errors were encountered: