From af1c56df54bad2cac6a95fbf6a9b336c84db490e Mon Sep 17 00:00:00 2001 From: Ben Konrath Date: Apr 27 2007 08:48:33 +0000 Subject: - Fix problem in launcher-addplatformtotildeeclipse.patch. - Resolves: #238109. --- diff --git a/eclipse-launcher-addplatformtotildeeclipse.patch b/eclipse-launcher-addplatformtotildeeclipse.patch index ec1eccc..ce701ef 100644 --- a/eclipse-launcher-addplatformtotildeeclipse.patch +++ b/eclipse-launcher-addplatformtotildeeclipse.patch @@ -6,8 +6,8 @@ RCS file: /cvsroot/eclipse/platform-launcher/library/gtk/eclipseGtk.c,v retrieving revision 1.27 diff -u -r1.27 eclipseGtk.c --- library/gtk/eclipseGtk.c 27 Mar 2006 18:25:42 -0000 1.27 -+++ library/gtk/eclipseGtk.c 28 Nov 2006 19:50:03 -0000 -@@ -335,3 +335,73 @@ ++++ library/gtk/eclipseGtk.c 27 Apr 2007 08:39:06 -0000 +@@ -335,3 +335,88 @@ gtk_main_quit(); return FALSE; } @@ -20,22 +20,37 @@ diff -u -r1.27 eclipseGtk.c + gsize old_length; + gchar *old_contents, *new_contents; + GError *error = NULL; -+ const gchar *site_element = "\n\n\n\n\0"; ++ const gchar *site_element = "\n\n\n\n\0"; + + platform_xml = g_strconcat(g_get_home_dir(), "/.eclipse/org.eclipse.platform_3.2.0/configuration/org.eclipse.update/platform.xml", NULL); -+ touched = g_strconcat(g_get_home_dir(), "/.eclipse/.homedirmodified-fedora", NULL); ++ touched = g_strconcat(g_get_home_dir(), "/.eclipse/.dont-need-to-modify-homedir-fedora", NULL); + -+ /* If platform.xml doesn't exist, Eclipse has yet to be started -+ * so don't worry about doing anything. Also, guard against doing -+ * this more than once with a dot file. */ -+ if (!g_file_test(platform_xml, G_FILE_TEST_EXISTS) || \ -+ g_file_test(touched, G_FILE_TEST_EXISTS)) { ++ ++ if (!g_file_test(platform_xml, G_FILE_TEST_EXISTS)) ++ { ++ /* If platform.xml doesn't exist, Eclipse has yet to be started. ++ * We don't have worry about doing anything now and in the future ++ * so add the appropriate file to ~/.eclipse. */ ++ if (g_file_set_contents(touched, "\0", -1, &error) == FALSE) ++ { ++ g_print("Error touching ~/.eclipse/.dont-need-to-modify-homedir-fedora."); ++ g_print(g_strconcat(error->message, "\n\0", NULL)); ++ g_free(error); ++ } + g_free(platform_xml); + g_free(touched); + return; ++ ++ } else if (g_file_test(touched, G_FILE_TEST_EXISTS)) { ++ /* touched exists, we don't need to do anything */ ++ g_free(platform_xml); ++ g_free(touched); ++ return; + } -+ + ++ ++ /* At this point platform_xml exists and touched does not exist */ ++ + if (g_file_get_contents (platform_xml, + &old_contents, &old_length, &error) == FALSE) + { @@ -68,7 +83,7 @@ diff -u -r1.27 eclipseGtk.c + + if (g_file_set_contents(touched, "\0", -1, &error) == FALSE) + { -+ g_print("Error touching ~/.eclipse/.homedirmodified-fedora.\ ++ g_print("Error touching ~/.eclipse/.dont-need-to-modify-homedir-fedora.\ + You should remove ~/.eclipse before restarting Eclipse:\0"); + g_print(g_strconcat(error->message, "\n\0", NULL)); + g_free(error); @@ -87,7 +102,7 @@ RCS file: /cvsroot/eclipse/platform-launcher/library/eclipse.c,v retrieving revision 1.71 diff -u -r1.71 eclipse.c --- library/eclipse.c 25 Apr 2006 14:31:50 -0000 1.71 -+++ library/eclipse.c 28 Nov 2006 19:50:03 -0000 ++++ library/eclipse.c 27 Apr 2007 08:39:05 -0000 @@ -512,6 +512,8 @@ /* Get the command to start the Java VM. */ vmCommandArgs = getVMCommand( argc, argv ); diff --git a/eclipse.spec b/eclipse.spec index c2b8706..87555ab 100644 --- a/eclipse.spec +++ b/eclipse.spec @@ -19,7 +19,7 @@ Epoch: 1 Summary: An open, extensible IDE Name: eclipse Version: %{eclipse_majmin}.%{eclipse_micro} -Release: 9%{?dist} +Release: 10%{?dist} License: Eclipse Public License Group: Text Editors/Integrated Development Environments (IDE) URL: http://www.eclipse.org/ @@ -1718,6 +1718,10 @@ fi %{_libdir}/%{name}/plugins/org.eclipse.sdk_* %changelog +* Fri Apr 27 2007 Ben Konrath 3.2.2-10 +- Fix problem in launcher-addplatformtotildeeclipse.patch. +- Resolves: #238109. + * Fri Apr 27 2007 Ben Konrath 3.2.2-9 - Remove BR eclipse-pde. - Related: #236895