Last Comment Bug 766358 - Update device storage paths for Gaia (again)
: Update device storage paths for Gaia (again)
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla16
Assigned To: Doug Turner (:dougt)
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2012-06-19 15:34 PDT by Doug Turner (:dougt)
Modified: 2012-06-21 04:05 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch v.1 (1.11 KB, patch)
2012-06-20 01:23 PDT, Doug Turner (:dougt)
fabrice: review+
Details | Diff | Splinter Review

Description Doug Turner (:dougt) 2012-06-19 15:34:14 PDT
Turn out that the picture directory might need to be /sdcard/DCIM/.
Comment 1 Hubert Figuiere [:hub] 2012-06-19 15:58:55 PDT
The DCF spec is at:

If we want to have the camera recognized as a camera on major OS (this includes Linux), then DCIM has to be at the root of that filesystem, and images stored in subdirectories following the right conventions as set in this spec.
This supports multiples vendor / device for the same storage (see 3rd party camera app)

Do be like these guys:
Comment 2 Hubert Figuiere [:hub] 2012-06-19 20:26:49 PDT
> Do be like these guys:

Here I meant "do not be like these guys".
Where the breakage is propagated to other vendors like Samsung.
Comment 3 Doug Turner (:dougt) 2012-06-19 21:14:33 PDT

it was like this before I was asked to change it to /sdcard/Pictures/  :(

We'll revert it back.
Comment 4 Doug Turner (:dougt) 2012-06-20 01:23:23 PDT
Created attachment 634811 [details] [diff] [review]
patch v.1
Comment 5 [:fabrice] Fabrice Desré 2012-06-20 11:49:38 PDT
Comment on attachment 634811 [details] [diff] [review]
patch v.1

Review of attachment 634811 [details] [diff] [review]:

I love the way you increase your commit log score ;)
Comment 6 Doug Turner (:dougt) 2012-06-20 12:50:57 PDT
i might be #19 now!
Comment 7 Ed Morley [:emorley] 2012-06-21 04:05:16 PDT

Note You need to log in before you can comment on or make changes to this bug.