[Chromoting] Include pkg-dmg script into installer archive.

[Chromoting] Include pkg-dmg script into installer archive.

A separate cl will modify the do_signing.sh script to use this
script instead of calling hdiutil directly.

This cl is a follow-up to 10197005 to address comments.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134842

[alexeypa (please no reviews), 2012-05-01 20:00:41]

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
File remoting/host/installer/build-installer-archive.py (right):

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:77: Any matching |src_root|
prefix is stripped from |src_file| before
nit: |src_roots|

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:87: If no |src_roots| match,
then the |src_file| path is used directly.
Can this be turned into a sanity check? I.e. if no |src_roots| matched then
|src_file| is assumed to be wrong, the script exits with an error and the build
breaks letting us know that there is a problem with GYP authoring.

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:219: if len(sys.argv) < 3:
This check does not seem to be correct any more.

[garykac, 2012-05-01 21:45:28]

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
File remoting/host/installer/build-installer-archive.py (right):

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:77: Any matching |src_root|
prefix is stripped from |src_file| before
On 2012/05/01 20:00:42, alexeypa wrote:
> nit: |src_roots|

Done.

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:87: If no |src_roots| match,
then the |src_file| path is used directly.
On 2012/05/01 20:00:42, alexeypa wrote:
> Can this be turned into a sanity check? I.e. if no |src_roots| matched then
> |src_file| is assumed to be wrong, the script exits with an error and the
build
> breaks letting us know that there is a problem with GYP authoring.

Done.

I needed to update the gen_files copy to no longer use remapSrcFile, so that's
even better.

http://codereview.chromium.org/10261033/diff/1/remoting/host/installer/build-...
remoting/host/installer/build-installer-archive.py:219: if len(sys.argv) < 3:
On 2012/05/01 20:00:42, alexeypa wrote:
> This check does not seem to be correct any more.

Done.

[alexeypa (please no reviews), 2012-05-01 21:50:22]

lgtm

[commit-bot: I haz the power, 2012-05-01 23:17:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/garykac@chromium.org/10261033/6001

[commit-bot: I haz the power, 2012-05-02 01:08:36]

Change committed as 134842