Closed
Bug 983508
Opened 11 years ago
Closed 11 years ago
[Build System] install-test-media get wrong "Pictures" path
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: GaryChen, Assigned: GaryChen)
Details
Attachments
(1 file)
$ make install-test-media
adb push test_media/DCIM /sdcard/DCIM
cannot stat 'test_media/DCIM': No such file or directory
make: *** [install-test-media] Error 1
Actually all pictures are under test_media/Pictures in gaia code base.
https://github.com/mozilla-b2g/gaia/tree/master/test_media/Pictures
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → gchen
Summary: [Build System] install-test-media wrong path → [Build System] install-test-media get wrong "Pictures" path
Assignee | ||
Comment 1•11 years ago
|
||
Hi Yuren,
please help to review this patch, thanks.
Attachment #8391005 -
Flags: review?(yurenju.mozilla)
Comment 2•11 years ago
|
||
Comment on attachment 8391005 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/17180
Gary, could you verify the content of output file to check what command are you executed?
Attachment #8391005 -
Flags: review?(yurenju.mozilla)
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8391005 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/17180
Hi Yuren,
I've addressed your comment, please review it again.
Thanks.
Attachment #8391005 -
Flags: review?(yurenju.mozilla)
Comment 4•11 years ago
|
||
Comment on attachment 8391005 [details] [review]
pull request: https://github.com/mozilla-b2g/gaia/pull/17180
thanks. fake-adb may not useful for this case but it would be useful if we have install-gaia integration test.
Attachment #8391005 -
Flags: review?(yurenju.mozilla) → review+
Assignee | ||
Comment 5•11 years ago
|
||
travis green:
https://travis-ci.org/mozilla-b2g/gaia/builds/20756761
land in gaia master:
https://github.com/mpizza/gaia/commit/3702cdccb382a99ef590a183a1ede6fea4f4e558
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•