Fix --payload for optional stateful.tgz.

Fix --payload for optional stateful.tgz.

Change-Id: I0af77b0ef83b016150aecdf6947182df8bad042d

BUG=chromium-os:9492
TEST=Ran manually

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

[sosa, 2010-11-23 00:49:41]

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode541
autoupdate.py:541: if os.path.dirname(src_path) !=
os.path.abspath(static_image_dir):
The comment here is somewhat misleading.  Can you change it to match the diff
logic

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode545
autoupdate.py:545: if os.path.exists(src_stateful):
I'm not really sure this should be optional.  Maybe at least print a warning
that it can't find it.

[dgarrett, 2010-11-23 01:30:27]

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode541
autoupdate.py:541: if os.path.dirname(src_path) !=
os.path.abspath(static_image_dir):
On 2010/11/23 00:49:42, sosa wrote:
> The comment here is somewhat misleading.  Can you change it to match the diff
> logic

Done.

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode545
autoupdate.py:545: if os.path.exists(src_stateful):
It's normal for it to be missing for some workflows. For example, in the
document "How to build and push an update locally".

On 2010/11/23 00:49:42, sosa wrote:
> I'm not really sure this should be optional.  Maybe at least print a warning
> that it can't find it.

[sosa, 2010-11-23 01:51:10]

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode545
autoupdate.py:545: if os.path.exists(src_stateful):
It should always be there for an image that is a Developer or Test image.  It's
not necessary for non-dev/test images but these are rare except as release
images.

On 2010/11/23 01:30:27, dgarrett wrote:
> It's normal for it to be missing for some workflows. For example, in the
> document "How to build and push an update locally".
> 
> On 2010/11/23 00:49:42, sosa wrote:
> > I'm not really sure this should be optional.  Maybe at least print a warning
> > that it can't find it.
>

[dgarrett, 2010-11-23 02:16:40]

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py
File autoupdate.py (right):

http://codereview.chromium.org/5281002/diff/2001/autoupdate.py#newcode545
autoupdate.py:545: if os.path.exists(src_stateful):
On 2010/11/23 01:51:11, sosa wrote:
> It should always be there for an image that is a Developer or Test image. 
It's
> not necessary for non-dev/test images but these are rare except as release
> images.
> 
> On 2010/11/23 01:30:27, dgarrett wrote:
> > It's normal for it to be missing for some workflows. For example, in the
> > document "How to build and push an update locally".
> > 
> > On 2010/11/23 00:49:42, sosa wrote:
> > > I'm not really sure this should be optional.  Maybe at least print a
warning
> > > that it can't find it.
> > 
> 

Done.