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)

Version 3
defect

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)

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.

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)
Flags: needinfo?(james)
Priority: -- → P3
Assignee: nobody → bob
Status: NEW → ASSIGNED
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
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: