Closed
Bug 1584966
Opened 5 years ago
Closed 4 years ago
[mozdevice] push() and push_dir() do not set the specified file permissions
Categories
(Testing :: Mozbase Rust, defect, P3)
Tracking
(firefox83 fixed)
RESOLVED
FIXED
83 Branch
Tracking | Status | |
---|---|---|
firefox83 | --- | fixed |
People
(Reporter: whimboo, Assigned: bc)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
As noticed by Nick while implementing the code for bug 1577850 both the push()
and push_dir()
commands do not correctly set the file permissions as specified by the mode
argument. Instead a shell command for chmod
is necessary to do so.
This is most likely a bug in the Android Bridge API, which we could workaround by calling chmod
for each pushed file.
I will have a look at this when refactoring the code for mozdevice on bug 1578424.
Comment 1•5 years ago
|
||
The priority flag is not set for this bug.
:jgraham, could you have a look please?
For more information, please visit auto_nag documentation.
Flags: needinfo?(james)
Updated•5 years ago
|
Flags: needinfo?(james)
Priority: -- → P3
Assignee | ||
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → bob
Status: NEW → ASSIGNED
Updated•4 years ago
|
Attachment #9175836 -
Attachment description: Bug 1584966 - Device::push must use octal values for mode. → Bug 1584966 - [rust mozdevice] Pre-create directories when pushing to a device.
Pushed by bclary@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a7d4f8360a70 [rust mozdevice] Pre-create directories when pushing to a device. r=webdriver-reviewers,whimboo,jgraham
Comment 4•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox83:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•