Add archiving of src tarball and debian packages

Add archiving of src tarball and debian packages

R=sgjesse@google.com

Committed: https://code.google.com/p/dart/source/detail?r=34155

[ricow1, 2014-03-17 15:20:24]

https://codereview.chromium.org/198723006/diff/1/tools/bots/src-tarball.py
File tools/bots/src-tarball.py (right):

https://codereview.chromium.org/198723006/diff/1/tools/bots/src-tarball.py#ne...
tools/bots/src-tarball.py:61: version = utils.GetVersion()
this block is just moved out of the with

[Søren Gjesse, 2014-03-17 18:07:34]

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py
File tools/bots/bot_utils.py (right):

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py#...
tools/bots/bot_utils.py:139: return self._variant_directory('linux_packages',
revision)
ubuntu_precise_packages? For the Debian chroot builds the .deb files will be
different and should be archived in e.g. debian_wheezy_packages.

[ricow1, 2014-03-17 18:24:47]

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py
File tools/bots/bot_utils.py (right):

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py#...
tools/bots/bot_utils.py:139: return self._variant_directory('linux_packages',
revision)
On 2014/03/17 18:07:35, Søren Gjesse wrote:
> ubuntu_precise_packages? For the Debian chroot builds the .deb files will be
> different and should be archived in e.g. debian_wheezy_packages.

We need to pass that into the script then, i.e., we should name the bots
accordingly. Do we want to actually produce the packages for ubuntu? I thought
that was just for now until we had the chroot

[Søren Gjesse, 2014-03-18 11:43:41]

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py
File tools/bots/bot_utils.py (right):

https://codereview.chromium.org/198723006/diff/20001/tools/bots/bot_utils.py#...
tools/bots/bot_utils.py:139: return self._variant_directory('linux_packages',
revision)
On 2014/03/17 18:24:47, ricow1 wrote:
> On 2014/03/17 18:07:35, Søren Gjesse wrote:
> > ubuntu_precise_packages? For the Debian chroot builds the .deb files will be
> > different and should be archived in e.g. debian_wheezy_packages.
> 
> We need to pass that into the script then, i.e., we should name the bots
> accordingly. Do we want to actually produce the packages for ubuntu? I thought
> that was just for now until we had the chroot

We should be able to produce packages for both Ubuntu and Debian, and have them
in different buckets. How about:

linux_packages/ubuntu_precise
linux_packages/debian_wheezy

[ricow1, 2014-03-18 14:27:42]

sounds good to me, I changed it now to get a linux system from the name of
builder.

I will send another buildbot cl to update the name of the existing builders

[Søren Gjesse, 2014-03-18 14:55:28]

lgtm

[ricow1, 2014-03-20 07:41:36]

Committed patchset #3 manually as r34155 (presubmit successful).