From 32ba95a539a77d1dd2b94d19e83024c40d9c10c7 Mon Sep 17 00:00:00 2001
From: "P. F. Chimento" <philip.chimento@gmail.com>
Date: Tue, 21 Jun 2011 17:48:31 +0200
Subject: [PATCH] Remove conditional compilation of recent files

...and split chimara.menus into menus files for browser and player
windows. The conditional compilation can be removed because the bug on
OS X for which we introduced it doesn't exist anymore.
---
 configure.ac                              | 13 -------------
 player/Makefile.am                        |  2 +-
 player/browser.c                          |  4 ++--
 player/browser.menus                      | 17 +++++++++++++++++
 player/player.c                           |  4 ++--
 player/{chimara.menus.in => player.menus} | 14 --------------
 6 files changed, 22 insertions(+), 32 deletions(-)
 create mode 100644 player/browser.menus
 rename player/{chimara.menus.in => player.menus} (70%)

diff --git a/configure.ac b/configure.ac
index 9e73d11..1ae23b3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -85,18 +85,6 @@ AC_ARG_ENABLE([iliad],
 	[enable_iliad=no])
 AM_CONDITIONAL(TARGET_ILIAD, $TEST "x$enable_iliad" = xyes)
 
-### BUILD WITHOUT RECENT FILES MANAGER #########################################
-# (to work around a bug on OS X)
-AC_ARG_ENABLE([recent],
-	[AS_HELP_STRING([--disable-recent],
-		[Omit recent files menu (to work around a bug on OS X])],
-	[],
-	[enable_recent=yes])
-AS_IF([$TEST "x$enable_recent" = "xyes"],
-	[OPEN_RECENT_MENU_ITEM="<menuitem action=\"recent\"/>"],
-	[OPEN_RECENT_MENU_ITEM="<!--  <menuitem action=\"recent\"/>-->"])
-AC_SUBST(OPEN_RECENT_MENU_ITEM)
-
 ### RPM CONFIGURATION ##########################################################
 # --enable-rpm requires rpm and rpmbuild
 AC_PATH_PROG([RPMBUILD], [rpmbuild], [notfound])
@@ -182,7 +170,6 @@ interpreters/glulxe/Makefile
 interpreters/git/Makefile
 tests/Makefile
 player/Makefile
-player/chimara.menus
 docs/Makefile
 docs/reference/Makefile
 docs/reference/version.xml
diff --git a/player/Makefile.am b/player/Makefile.am
index dd0d0cb..a8343b6 100644
--- a/player/Makefile.am
+++ b/player/Makefile.am
@@ -13,7 +13,7 @@ bin_PROGRAMS = chimara_iliad
 
 else
 
-dist_pkgdata_DATA = chimara.ui chimara.menus style.css
+dist_pkgdata_DATA = chimara.ui browser.menus player.menus style.css
 bin_PROGRAMS = chimara
 
 chimara_SOURCES = main.c \
diff --git a/player/browser.c b/player/browser.c
index e4b3e74..4646eef 100644
--- a/player/browser.c
+++ b/player/browser.c
@@ -50,11 +50,11 @@ chimara_browser_init(ChimaraBrowser *self)
 		NULL);
 
 	GtkUIManager *uimanager = gtk_ui_manager_new();
-	if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_DATA_DIR "/chimara.menus", &error) ) {
+	if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_DATA_DIR "/browser.menus", &error) ) {
 #ifdef DEBUG
 		g_error_free(error);
 		error = NULL;
-		if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_SRC_DIR "/chimara.menus", &error) ) {
+		if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_SRC_DIR "/browser.menus", &error) ) {
 #endif /* DEBUG */
 			error_dialog(NULL, error, "Error while building interface: ");
 			return;
diff --git a/player/browser.menus b/player/browser.menus
new file mode 100644
index 0000000..a1151c9
--- /dev/null
+++ b/player/browser.menus
@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<ui>
+  <menubar name="browser_menu">
+    <menu action="game">
+      <menuitem action="open"/>
+	  <menuitem action="recent"/>
+      <separator/>
+      <menuitem action="quit_chimara"/>
+    </menu>
+    <menu action="edit">
+      <menuitem action="preferences"/>
+    </menu>
+    <menu action="help">
+      <menuitem action="about"/>
+    </menu>
+  </menubar>
+</ui>
diff --git a/player/player.c b/player/player.c
index 0d8d629..d8b094f 100644
--- a/player/player.c
+++ b/player/player.c
@@ -132,11 +132,11 @@ chimara_player_init(ChimaraPlayer *self)
 	//g_settings_bind(state_settings, "show-toolbar-default", toolbar_action, "active", G_SETTINGS_BIND_SET);
 		
 	GtkUIManager *uimanager = gtk_ui_manager_new();
-	if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_DATA_DIR "/chimara.menus", &error) ) {
+	if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_DATA_DIR "/player.menus", &error) ) {
 #ifdef DEBUG
 		g_error_free(error);
 		error = NULL;
-		if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_SRC_DIR "/chimara.menus", &error) ) {
+		if( !gtk_ui_manager_add_ui_from_file(uimanager, PACKAGE_SRC_DIR "/player.menus", &error) ) {
 #endif /* DEBUG */
 			error_dialog(NULL, error, "Error while building interface: ");
 			return;
diff --git a/player/chimara.menus.in b/player/player.menus
similarity index 70%
rename from player/chimara.menus.in
rename to player/player.menus
index a140a2e..3d80345 100644
--- a/player/chimara.menus.in
+++ b/player/player.menus
@@ -1,19 +1,5 @@
 <?xml version="1.0"?>
 <ui>
-  <menubar name="browser_menu">
-    <menu action="game">
-      <menuitem action="open"/>
-	  @OPEN_RECENT_MENU_ITEM@
-      <separator/>
-      <menuitem action="quit_chimara"/>
-    </menu>
-    <menu action="edit">
-      <menuitem action="preferences"/>
-    </menu>
-    <menu action="help">
-      <menuitem action="about"/>
-    </menu>
-  </menubar>
   <menubar name="player_menu">
     <menu action="game">
       <menuitem action="stop"/>
-- 
2.30.2