Merge lp:~ken-vandine/unity-webapps-launchpad/lp1369208 into lp:unity-webapps-launchpad

Proposed by Ken VanDine on 2015-09-29
Status: Merged
Approved by: Alexandre Abreu on 2015-11-19
Approved revision: 56
Merged at revision: 55
Proposed branch: lp:~ken-vandine/unity-webapps-launchpad/lp1369208
Merge into: lp:unity-webapps-launchpad
Diff against target: 46 lines (+0/-35)
2 files modified
debian/unity-webapps-launchpad.migrations (+0/-1)
scripts/update-desktop-file-unity-webapps-launchpad.py (+0/-34)
To merge this branch: bzr merge lp:~ken-vandine/unity-webapps-launchpad/lp1369208
Reviewer Review Type Date Requested Status
Alexandre Abreu (community) 2015-09-29 Approve on 2015-11-19
Review via email: mp+272785@code.launchpad.net

Commit message

When using gi.repository you must not import static modules like gobject, patch thanks to Puspendu Banerjee (LP: #1369208)

Description of the change

When using gi.repository you must not import static modules like gobject, patch thanks to Puspendu Banerjee (LP: #1369208)

To post a comment you must log in.
55. By Ken VanDine on 2015-09-29

When using gi.repository you must not import static modules like gobject, patch thanks to Puspendu Banerjee (LP: #1369208)

56. By Ken VanDine on 2015-09-29

dropped migration script, no longer needed

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== removed file 'debian/unity-webapps-launchpad.migrations'
2--- debian/unity-webapps-launchpad.migrations 2013-09-24 16:18:15 +0000
3+++ debian/unity-webapps-launchpad.migrations 1970-01-01 00:00:00 +0000
4@@ -1,1 +0,0 @@
5-scripts/update-desktop-file-unity-webapps-launchpad.py
6
7=== removed directory 'scripts'
8=== removed file 'scripts/update-desktop-file-unity-webapps-launchpad.py'
9--- scripts/update-desktop-file-unity-webapps-launchpad.py 2013-09-24 16:18:15 +0000
10+++ scripts/update-desktop-file-unity-webapps-launchpad.py 1970-01-01 00:00:00 +0000
11@@ -1,34 +0,0 @@
12-#!/usr/bin/python
13-from gi.repository import Gio
14-import os
15-import glib
16-import glob
17-import re
18-
19-APP_ID = 'Launchpadlaunchpadnet'
20-
21-def get_local_applications_path():
22- return os.path.join(glib.get_user_data_dir(), 'applications')
23-
24-def update_desktop_file_startupwmclass():
25- local_applications_path = get_local_applications_path()
26- if not os.path.exists(local_applications_path) or not os.path.isdir(local_applications_path):
27- return
28-
29- desktop_filename = os.path.join(local_applications_path, '{0}.desktop'.format(APP_ID))
30- if not os.path.exists(desktop_filename) or not os.path.isfile(desktop_filename):
31- return
32-
33- try:
34- desktop_file_content = open(desktop_filename).read()
35- start_idx = desktop_file_content.find('[Desktop Entry]')
36- if start_idx != -1 and desktop_file_content.find('StartupWMClass') == -1:
37- start_idx += len('[Desktop Entry]')
38- updated_desktop_file_content = desktop_file_content[:start_idx] + '\nStartupWMClass={0}'.format(APP_ID) + desktop_file_content[start_idx:]
39- open(desktop_filename, "w+").write(updated_desktop_file_content)
40- except Exception, e:
41- print 'Error while upgrading the desktop file: ', str(e)
42-
43-if __name__ == "__main__":
44- update_desktop_file_startupwmclass()
45-
46\ No newline at end of file

Subscribers

People subscribed via source and target branches

to all changes: