junfenglx / googletest

Automatically exported from code.google.com/p/googletest
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Support Android tmp dir for CapturedStream. #407

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Android's tmp dir is /data/local/tmp

Original issue reported on code.google.com by zheng...@google.com on 16 Mar 2012 at 4:12

GoogleCodeExporter commented 9 years ago
jgm@, we use /data/local/tmp as tmp dir for Chromium Android. I'm not sure 
whether it's preferable than your recent change of /mnt/sdcard/.

Original comment by zheng...@google.com on 16 Mar 2012 at 4:26

GoogleCodeExporter commented 9 years ago
zhenghao@ - is the use of /data/local/tmp documented somewhere? /mnt/sdcard/ is 
a known location, while /data/local/tmp doesn't appear to be.

Original comment by vladlosev on 26 Apr 2012 at 5:56

GoogleCodeExporter commented 9 years ago
Well, it's not documented. I think users could make dir at /data/local/tmp 
without root, so I think it's a good place. /mnt/sdcard is also good to me. 
What if the device doesn't have a sdcard?

Original comment by zheng...@google.com on 27 Apr 2012 at 3:37

GoogleCodeExporter commented 9 years ago
Aren't devices that have no SD cards still supposed to have the /mnt/sdcard 
directory in the filesystem? At least Nexus S and Galaxy Nexus do.

Original comment by vladlosev on 27 Apr 2012 at 6:08

GoogleCodeExporter commented 9 years ago
OK, then /mnt/sdcard/ is fine for me. Looks like I cannot close the bug. Please 
close it if you can :) Thanks!

Original comment by zheng...@google.com on 9 May 2012 at 4:15

GoogleCodeExporter commented 9 years ago
Fixed in revision 625.

Original comment by vladlosev on 22 Sep 2012 at 2:57