Basic Debian packaging for Linux Me2Me host

Basic Debian packaging for Linux Me2Me host

This facilitates building a Debian package, currently for Ubuntu 10.04.  This is based on work by benley@ and jamiewalch@ (http://codereview.chromium.org/10800099/)

TODO:
* Allow building packages for other versions of Ubuntu (maybe detect the current build platform and use that)
* Integrate better with gyp, instead of hard-coding out/Release and manually supplying version_full to the script.
* Add man-page for me2me_virtual_host
* Use separate staging area for the debian/ build directory, instead of polluting the source tree.


BUG=131245
TEST=


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

[Jamie, 2012-07-25 23:40:40]

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/build-deb.sh (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/build-deb.sh:8:
version_full="${version_full:-1.2.3.4}"
Probably better to fail if version_full isn't set. It's easy enough to set it on
the command-line for testing.

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/build-deb.sh:17: "Debian package"
What is this string used for?

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/compat (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/compat:1: 7
What does this mean?

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/control (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/control:15: Depends: python (>= 2.6), xvfb,
${shlibs:Depends}, ${misc:Depends}, ${python:Depends}
Aren't we going to install a RANDR-aware version of Xvbf as part of the package?
At the point, presumably this dependency will go away, correct?

[Lambros, 2012-07-26 21:19:04]

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/build-deb.sh (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/build-deb.sh:8:
version_full="${version_full:-1.2.3.4}"
On 2012/07/25 23:40:40, Jamie wrote:
> Probably better to fail if version_full isn't set. It's easy enough to set it
on
> the command-line for testing.

Done.

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/build-deb.sh:17: "Debian package"
On 2012/07/25 23:40:40, Jamie wrote:
> What is this string used for?
It's the text of the changelog entry.  I've changed it back to "New Debian
package" so that's more obvious.

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/compat (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/compat:1: 7
On 2012/07/25 23:40:40, Jamie wrote:
> What does this mean?
It refers to the debhelper version supported by the layout and content of the
debian/ scripts.  It's a bit of a mysterious art-form, but 7 is the recommended
value in Lucid. See 'man debhelper' and scroll down to 'Debhelper compatibility
levels'.

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/control (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/control:15: Depends: python (>= 2.6), xvfb,
${shlibs:Depends}, ${misc:Depends}, ${python:Depends}
On 2012/07/25 23:40:40, Jamie wrote:
> Aren't we going to install a RANDR-aware version of Xvbf as part of the
package?
> At the point, presumably this dependency will go away, correct?
Good point - we don't need xvfb as a dependency if we're bundling a patched
Xvfb.  Done.

[Jamie, 2012-07-26 21:50:31]

lgtm with a few nits.

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/compat (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/compat:1: 7
On 2012/07/26 21:19:04, Lambros wrote:
> On 2012/07/25 23:40:40, Jamie wrote:
> > What does this mean?
> It refers to the debhelper version supported by the layout and content of the
> debian/ scripts.  It's a bit of a mysterious art-form, but 7 is the
recommended
> value in Lucid. See 'man debhelper' and scroll down to 'Debhelper
compatibility
> levels'.

I don't suppose this file supports comments? If so, then a comment to that
effect and a copyright notice wouldn't go amiss.

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
File remoting/host/installer/linux/Makefile (right):

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:1: SRC = ../../../..
Please add a copyright header.

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:5: BIN = $(DESTDIR)/usr/bin
I assume that DESTDIR is set by the enclosing Makefile?

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:17: install $(RELEASE)/Xvfb-randr -m 0755
$(BIN)
Unless we're planning on building this from source, I doubt we'll pull it from
RELEASE. Probably fine for now, but maybe add a TODO to decide what to do about
bundling Xvfb?

[Lambros, 2012-07-26 23:42:38]

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
File remoting/host/installer/linux/debian/compat (right):

http://codereview.chromium.org/10821028/diff/1/remoting/host/installer/linux/...
remoting/host/installer/linux/debian/compat:1: 7
On 2012/07/26 21:50:31, Jamie wrote:
> On 2012/07/26 21:19:04, Lambros wrote:
> > On 2012/07/25 23:40:40, Jamie wrote:
> > > What does this mean?
> > It refers to the debhelper version supported by the layout and content of
the
> > debian/ scripts.  It's a bit of a mysterious art-form, but 7 is the
> recommended
> > value in Lucid. See 'man debhelper' and scroll down to 'Debhelper
> compatibility
> > levels'.
> 
> I don't suppose this file supports comments? If so, then a comment to that
> effect and a copyright notice wouldn't go amiss.
Nope, I checked.  I plan to generate the debian/ directory structure somewhere
else, in which case it makes sense to generate the debian/compat file.

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
File remoting/host/installer/linux/Makefile (right):

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:1: SRC = ../../../..
On 2012/07/26 21:50:31, Jamie wrote:
> Please add a copyright header.

Done.

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:5: BIN = $(DESTDIR)/usr/bin
On 2012/07/26 21:50:31, Jamie wrote:
> I assume that DESTDIR is set by the enclosing Makefile?

Added comment.

http://codereview.chromium.org/10821028/diff/6001/remoting/host/installer/lin...
remoting/host/installer/linux/Makefile:17: install $(RELEASE)/Xvfb-randr -m 0755
$(BIN)
On 2012/07/26 21:50:31, Jamie wrote:
> Unless we're planning on building this from source, I doubt we'll pull it from
> RELEASE. Probably fine for now, but maybe add a TODO to decide what to do
about
> bundling Xvfb?

Done.

[commit-bot: I haz the power, 2012-07-26 23:42:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/lambroslambrou@chromium.org/10821028/9001

[commit-bot: I haz the power, 2012-07-27 02:22:33]

Change committed as 148701