Create the latest_download dir if it doesn't exist.

Create the latest_download dir if it doesn't exist.

I started to use /tmp, but it doesn't exist both inside and outside of the chroot. Using a consistent name like this has the advantage that the files will be cached and not redownloaded every time. Well, except on the build servers that wipe the related directories between builds.

Change-Id: I58b42d9f1bd18cc4eb44043ed2436d4ffa482767

BUG=chromium-os:10012
TEST=

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=e55c89a

[sosa, 2010-12-06 19:42:15]

LGTM but can you move it?

http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py
File bin/cros_au_test_harness.py (right):

http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py#new...
bin/cros_au_test_harness.py:53: os.makedirs(self.download_folder)
Can you switch to using self.crosutils?

[dgarrett, 2010-12-06 19:51:28]

http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py
File bin/cros_au_test_harness.py (right):

http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py#new...
bin/cros_au_test_harness.py:53: os.makedirs(self.download_folder)
On 2010/12/06 19:42:15, sosa wrote:
> Can you switch to using self.crosutils?

Do you mean put the downloaded files directly into crosutils, or put
latest_download in crosutils?

Current latest_download is created in crosutilsbin for a desktop developer
build.

[sosa, 2010-12-06 22:48:27]

Suggested moving it to be consistent with the builders workflow.

On Mon, Dec 6, 2010 at 11:51 AM,  <dgarrett@chromium.org> wrote:
>
> http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py
> File bin/cros_au_test_harness.py (right):
>
>
http://codereview.chromium.org/5597004/diff/1/bin/cros_au_test_harness.py#new...
> bin/cros_au_test_harness.py:53: os.makedirs(self.download_folder)
> On 2010/12/06 19:42:15, sosa wrote:
>>
>> Can you switch to using self.crosutils?
>
> Do you mean put the downloaded files directly into crosutils, or put
> latest_download in crosutils?
>
> Current latest_download is created in crosutilsbin for a desktop
> developer build.
>
> http://codereview.chromium.org/5597004/
>

[dgarrett, 2010-12-06 23:27:22]

Just to be certain, is this what you mean?

[sosa, 2010-12-06 23:57:52]

Yes, LGTM

On Mon, Dec 6, 2010 at 3:27 PM,  <dgarrett@chromium.org> wrote:
> Just to be certain, is this what you mean?
>
> http://codereview.chromium.org/5597004/
>