Merge lp:~compiz-team/compiz/compiz.fix_1184755 into lp:compiz/0.9.10

Proposed by Sam Spilsbury
Status: Superseded
Proposed branch: lp:~compiz-team/compiz/compiz.fix_1184755
Merge into: lp:compiz/0.9.10
Diff against target: 79 lines (+31/-13)
2 files modified
cmake/CompizGSettings.cmake (+10/-1)
compizconfig/ccsm/setup.py (+21/-12)
To merge this branch: bzr merge lp:~compiz-team/compiz/compiz.fix_1184755
Reviewer Review Type Date Requested Status
Compiz Maintainers Pending
Review via email: mp+161308@code.launchpad.net

This proposal has been superseded by a proposal from 2013-04-28.

Commit message

Fix some files not installing:

 1. gsettings schemas never get added to the install_manifest list
    because we use install (CODE) so we need to add a custom
    uninstall target for those, and also need to recompile on uninstall
    too
 2. Run the gtk-update-icon-cache stage during install for ccsm and
    also add icon-theme.cache to the uninstall target

(LP: #1168475)

Description of the change

Fix some files not installing:

 1. gsettings schemas never get added to the install_manifest list
    because we use install (CODE) so we need to add a custom
    uninstall target for those, and also need to recompile on uninstall
    too
 2. Run the gtk-update-icon-cache stage during install for ccsm and
    also add icon-theme.cache to the uninstall target

(LP: #1168475)

To post a comment you must log in.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmake/CompizGSettings.cmake'
2--- cmake/CompizGSettings.cmake 2013-02-20 10:53:57 +0000
3+++ cmake/CompizGSettings.cmake 2013-04-28 04:51:28 +0000
4@@ -55,6 +55,9 @@
5 execute_process (COMMAND cmake -DSCHEMADIR_USER=${_schemadir_user} -DSCHEMADIR_ROOT=${GSETTINGS_GLOBAL_INSTALL_DIR} -P ${COMPIZ_CMAKE_MODULE_PATH}/recompile_gsettings_schemas_in_dir_user_env.cmake)
6 ")
7
8+ # We must also recompile on uninstall too
9+ compiz_add_code_to_uninstall_target ("cmake -DSCHEMADIR_USER=${_schemadir_user} -DSCHEMADIR_ROOT=${GSETTINGS_GLOBAL_INSTALL_DIR} -P ${COMPIZ_CMAKE_MODULE_PATH}/recompile_gsettings_schemas_in_dir_user_env.cmake" ${CMAKE_CURRENT_BINARY_DIR})
10+
11 endif (GSETTINGS_GLOBAL_INSTALL_DIR_SET)
12
13 endfunction (compiz_add_install_recompile_gsettings_schemas)
14@@ -76,9 +79,15 @@
15
16 # Install schema file
17 install (CODE "
18- execute_process (COMMAND cmake -DFILE=${_src} -DINSTALLDIR_USER=${_dst} -DINSTALLDIR_ROOT=${GSETTINGS_GLOBAL_INSTALL_DIR} -P ${COMPIZ_CMAKE_MODULE_PATH}/copy_file_install_user_env.cmake)
19+ execute_process (COMMAND cmake -DFILE=${_src}
20+ -DINSTALLDIR_USER=${_dst}
21+ -DINSTALLDIR_ROOT=${GSETTINGS_GLOBAL_INSTALL_DIR}
22+ -P ${COMPIZ_CMAKE_MODULE_PATH}/copy_file_install_user_env.cmake)
23 ")
24
25+ get_filename_component (_src_file_basename ${_src} NAME)
26+ compiz_add_code_to_uninstall_target ("cmake -E remove -f ${_dst}/${_src_file_basename}" ${CMAKE_CURRENT_BINARY_DIR})
27+
28 get_property (COMPIZ_INHIBIT_ADD_INSTALL_RECOMPILE_RULE_SET
29 GLOBAL
30 PROPERTY COMPIZ_INHIBIT_ADD_INSTALL_RECOMPILE_RULE
31
32=== modified file 'compizconfig/ccsm/setup.py'
33--- compizconfig/ccsm/setup.py 2012-10-16 13:53:20 +0000
34+++ compizconfig/ccsm/setup.py 2013-04-28 04:51:28 +0000
35@@ -36,7 +36,28 @@
36
37 def run (self):
38 _install.run (self)
39+
40+ icon_prefix = '%s/share/ccsm/icons/hicolor' % prefix
41+ icon_theme_cache = '%s/icon-theme.cache' % icon_prefix
42+
43+ print ("icon_prefix ", icon_prefix)
44+ print ("icon_theme_cache ", icon_theme_cache)
45+
46 outputs = self.get_outputs ()
47+
48+ # Update icon cache
49+ gtk_update_icon_cache = '''gtk-update-icon-cache -f -t \
50+%s''' % icon_prefix
51+ root_specified = len (filter (lambda s: s.startswith ("--root"),
52+ sys.argv)) > 0
53+ if not root_specified:
54+ print "Updating Gtk icon cache."
55+ os.system (gtk_update_icon_cache)
56+ outputs.append (icon_theme_cache)
57+ else:
58+ print '''*** Icon cache not updated. After install, run this:
59+*** %s''' % gtk_update_icon_cache
60+
61 length = 0
62 if self.root:
63 length += len (self.root)
64@@ -216,15 +237,3 @@
65 )
66
67 os.remove ("ccm/Constants.py")
68-
69-if sys.argv[1] == "install":
70- gtk_update_icon_cache = '''gtk-update-icon-cache -f -t \
71-%s/share/ccsm/icons/hicolor''' % prefix
72- root_specified = len (filter (lambda s: s.startswith ("--root"),
73- sys.argv)) > 0
74- if not root_specified:
75- print "Updating Gtk icon cache."
76- os.system (gtk_update_icon_cache)
77- else:
78- print '''*** Icon cache not updated. After install, run this:
79-*** %s''' % gtk_update_icon_cache

Subscribers

People subscribed via source and target branches

to all changes: