Merge lp:~robru/unity-webapps-googlenews/packaging-fixups into lp:unity-webapps-googlenews

Proposed by Robert Bruce Park
Status: Merged
Approved by: Robert Bruce Park
Approved revision: 23
Merged at revision: 22
Proposed branch: lp:~robru/unity-webapps-googlenews/packaging-fixups
Merge into: lp:unity-webapps-googlenews
Diff against target: 72 lines (+10/-10)
4 files modified
GoogleNews.user.js (+2/-2)
debian/control (+7/-3)
debian/copyright (+0/-5)
manifest.json (+1/-0)
To merge this branch: bzr merge lp:~robru/unity-webapps-googlenews/packaging-fixups
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
WebApps Pending
Review via email: mp+169883@code.launchpad.net

Commit message

Minor packaging fixes.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'GoogleNews.user.js'
2--- GoogleNews.user.js 2013-05-16 05:15:08 +0000
3+++ GoogleNews.user.js 2013-06-17 18:57:52 +0000
4@@ -6,7 +6,7 @@
5 // ==/UserScript==
6
7 // This placeholder gets munged with real data at build time.
8-const WebappsGettextDict = JSON.parse(unescape(
9+var WebappsGettextDict = JSON.parse(unescape(
10 "%7B%22GETTEXT%22%3A%22PLACEHOLDER%22%7D"
11 ));
12
13@@ -17,7 +17,7 @@
14 var i, elements = document.getElementsByClassName('titletext');
15 for (i = 0; i < elements.length; i++) {
16 var title = elements[i].textContent;
17- res.push({ icon: null, author: _("Google News"), body: title })
18+ res.push({ icon: null, author: _("Google News"), body: title });
19 }
20 return res;
21 }
22
23=== modified file 'debian/control'
24--- debian/control 2013-05-15 07:18:26 +0000
25+++ debian/control 2013-06-17 18:57:52 +0000
26@@ -1,17 +1,21 @@
27 Source: unity-webapps-googlenews
28 Section: web
29-Priority: extra
30+Priority: optional
31 Standards-Version: 3.9.4
32 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
33 Build-Depends: debhelper (>=9),
34 unity-webapps-dev (>=2.4.16),
35 Homepage: https://news.google.com/
36+# If you aren't a member of ~webapps but need to upload packaging changes,
37+# just go ahead. ~webapps will notice and sync up the code again.
38+Vcs-Bzr: https://code.launchpad.net/~webapps/unity-webapps-googlenews/trunk
39
40 Package: unity-webapps-googlenews
41 Architecture: all
42 Section: web
43-Priority: extra
44-Depends: unity-webapps-common, xdg-utils, ${misc:Depends}
45+Depends: unity-webapps-common,
46+ xdg-utils,
47+ ${misc:Depends},
48 XB-Ubuntu-Webapps-Includes: https://news.google.com/*
49 XB-Ubuntu-Webapps-Name: GoogleNews
50 XB-Ubuntu-Webapps-Domain: google.com
51
52=== modified file 'debian/copyright'
53--- debian/copyright 2013-04-30 18:20:17 +0000
54+++ debian/copyright 2013-06-17 18:57:52 +0000
55@@ -22,11 +22,6 @@
56 The full text of the can be found in the file
57 '/usr/share/common-licenses/GPL-3'.
58
59-Files: debian/*
60-Copyright: 2012 The friendly pkgme.net robot
61-License: public-domain
62- This automatically generated code which is free from copyright restrictions.
63-
64
65 Files: */*.png */*.svg
66 Copyright: various
67
68=== added file 'manifest.json'
69--- manifest.json 1970-01-01 00:00:00 +0000
70+++ manifest.json 2013-06-17 18:57:52 +0000
71@@ -0,0 +1,1 @@
72+{"includes":["https://news.google.com/*","http://news.google.com/*"],"requires":["utils.js","google-common.js"],"name":"GoogleNews","scripts":["GoogleNews.user.js"],"maintainer":"Webapps Team <webapps@lists.launchpad.net>","manifest-version":"1.0","integration-version":"2.4","package-name":"GoogleNews","icons":{"128":"128/unity-webapps-google-news.png","48":"48/unity-webapps-google-news.png","52":"52/unity-webapps-google-news.png","64":"64/unity-webapps-google-news.png"},"domain":"google.com","homepage":"http://news.google.com","license":"GPL-3"}

Subscribers

People subscribed via source and target branches

to all changes: