devserver: better handling of archive directory

devserver: better handling of archive directory

Fixes relative references to static directory in archive dir handling.

Originally, the archive dir was linked into static/archive. But
the location of static was hardcoded to be in ./, which is untrue
for chroot installed devserver. This also works around for odd uses
of archive directory to keep cros_au_test_harness working, before
it can be fixed - see inlined code comments.

BUG=5246
TEST=run cros_au_test_harness

Change-Id: Ia3eb04fdb9e85d94e2e9e65bd8b89ce5ca930bf8

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

[sosa, 2011-02-18 22:24:42]

nits

http://codereview.chromium.org/6469046/diff/1/devserver.py
File devserver.py (right):

http://codereview.chromium.org/6469046/diff/1/devserver.py#newcode167
devserver.py:167: # FIXME(zbehan) Legacy support:
TODO(zbehan):  Remove legacy support rather than FIXME

http://codereview.chromium.org/6469046/diff/1/devserver.py#newcode175
devserver.py:175: os.chdir(devserver_dir) # this surely exists
I'd prefer if you checked if that path was relative path than chdir

[sosa, 2011-02-18 23:16:44]

LGTM, thanks for the change!

[sosa, 2011-02-18 23:16:45]

LGTM, thanks for the change!

[zbehan, 2011-02-18 23:39:00]

Should've waited for harness to pass before uploading :/ Now it does work.

On Sat, Feb 19, 2011 at 12:35 AM, <zbehan@chromium.org> wrote:

> http://codereview.chromium.org/6469046/
>