devserver: start using payload scripts bundled with devserver

devserver: start using payload scripts bundled with devserver

This follows CL's which move payload generators to devutils, and
removes the dependency on these scripts in src/scripts.

Following this, the autogeneration of payloads by the devserver
installed in chroot should work.

BUG=chromium-os:5246
TEST=cros_au_test_harness

Change-Id: Id3be5e140e93ee1813849f6ede6bbac53c60dde4

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

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

cpl nits, ow lgtm

http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
File host/cros_generate_stateful_update_payload (right):

http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
host/cros_generate_stateful_update_payload:30: # FIXME: This is only used for
mount_gpt_image.sh. That script also
TODO format plz

http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
host/cros_generate_stateful_update_payload:32: crosutils_dir = "%s/src/scripts"
% os.environ["CROS_WORKON_SRCROOT"]
don't mix "" with '' .. all non doc-strings here use '' so use '' rather than ""

[zbehan, 2011-02-18 22:55:29]

On Fri, Feb 18, 2011 at 11:29 PM, <sosa@chromium.org> wrote:

> cpl nits, ow lgtm
>
>
>
>
http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
> File host/cros_generate_stateful_update_payload (right):
>
>
>
http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
> host/cros_generate_stateful_update_payload:30: # FIXME: This is only
>
> used for mount_gpt_image.sh. That script also
> TODO format plz
>

Will fix.

http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
> host/cros_generate_stateful_update_payload:32: crosutils_dir =
>
> "%s/src/scripts" % os.environ["CROS_WORKON_SRCROOT"]
> don't mix "" with '' .. all non doc-strings here use '' so use '' rather
> than ""


Not all. I followed how the modified code was written. I can of course
change it.

[sosa, 2011-02-18 23:06:43]

On Fri, Feb 18, 2011 at 2:55 PM, Zdenek Behan <zbehan@chromium.org> wrote:
>
>
> On Fri, Feb 18, 2011 at 11:29 PM, <sosa@chromium.org> wrote:
>>
>> cpl nits, ow lgtm
>>
>>
>>
>>
http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
>> File host/cros_generate_stateful_update_payload (right):
>>
>>
>>
http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
>> host/cros_generate_stateful_update_payload:30: # FIXME: This is only
>> used for mount_gpt_image.sh. That script also
>> TODO format plz
>
> Will fix.
>>
>>
>>
http://codereview.chromium.org/6542009/diff/1/host/cros_generate_stateful_upd...
>> host/cros_generate_stateful_update_payload:32: crosutils_dir =
>> "%s/src/scripts" % os.environ["CROS_WORKON_SRCROOT"]
>> don't mix "" with '' .. all non doc-strings here use '' so use '' rather
>> than ""
>
> Not all. I followed how the modified code was written. I can of course
> change it.

Yes please.

[sosa, 2011-02-18 23:17:20]

LGTM