Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(855)

Issue 10800099: Initial prototype of Me2Me Debian package (Closed)

Created:
8 years, 5 months ago by Jamie
Modified:
8 years, 4 months ago
Reviewers:
Lambros
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Initial prototype of Me2Me Debian package Note that this has a few shortcomings: * The Depends: list is hard-coded. We should be using dpkg-shlibdeps and dh_gencontrol to generate it automatically, but I couldn't make it work. * It's not integrated with gyp. However, it is gyp-ready, insofar as it will use the version_full environment variable, if it is set. * The "lintian" tool gives a number of errors about missing man pages and changelog and also about the fact the our python scripts should not have a .py extension (this might be in conflict with the Chrome style guide). BUG=131245 TEST=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -0 lines) Patch
A remoting/host/installer/linux/build-deb.sh View 1 chunk +51 lines, -0 lines 0 comments Download
A remoting/host/installer/linux/control View 1 chunk +16 lines, -0 lines 0 comments Download
A remoting/host/installer/linux/copyright View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Jamie
8 years, 5 months ago (2012-07-24 17:16:00 UTC) #1
This is probably not committable as-is, but in conjunction with the work Ben
sent us, you might be able to come up with something fairly quickly that
addresses the concerns I've outlined.

Powered by Google App Engine
This is Rietveld 408576698