rpms/selenium-remote-control/EL-5 selenium-safari-cleanup.patch, NONE, 1.1 selenium-remote-control.spec, 1.1, 1.2

[Lubomir Rintel]

Author: lkundrak

Update of /cvs/pkgs/rpms/selenium-remote-control/EL-5
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv17291/EL-5

Modified Files:
	selenium-remote-control.spec 
Added Files:
	selenium-safari-cleanup.patch 
Log Message:
Clean up after Safari properly

selenium-safari-cleanup.patch:
 SafariCustomProfileLauncher.java |    3 +++
 1 file changed, 3 insertions(+)

--- NEW FILE selenium-safari-cleanup.patch ---
From: Lubomir Rintel (Good Data) <lubo.rintel at gooddata.com>
Subject: [PATCH] Clean up after Safari terminates

Safari Launcher did nor remove the profile directories it created, causing
disk space shortages to occur. This tiny change addresses the problem.

Index: server-coreless/src/main/java/org/openqa/selenium/server/browserlaunchers/SafariCustomProfileLauncher.java
===================================================================
--- server-coreless/src/main/java/org/openqa/selenium/server/browserlaunchers/SafariCustomProfileLauncher.java	(revision 7778)
+++ server-coreless/src/main/java/org/openqa/selenium/server/browserlaunchers/SafariCustomProfileLauncher.java	(working copy)
@@ -136,6 +136,9 @@
             originalCookieFile = new File(originalCookieFilePath);
             LauncherUtils.copySingleFile(backedUpCookieFile, originalCookieFile);
         }
+
+        LOGGER.info("Removing profile directory...");
+        LauncherUtils.deleteTryTryAgain(customProfileDir, 6);
     }
 
     protected void ensureCleanSession() {


Index: selenium-remote-control.spec
===================================================================
RCS file: /cvs/pkgs/rpms/selenium-remote-control/EL-5/selenium-remote-control.spec,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -p -r1.1 -r1.2
--- selenium-remote-control.spec	4 Jan 2010 21:40:12 -0000	1.1
+++ selenium-remote-control.spec	5 Jan 2010 13:10:48 -0000	1.2
@@ -1,6 +1,6 @@
 Name:           selenium-remote-control
 Version:        1.0.2
-Release:        0.2.20091211svn%{?dist}
+Release:        0.3.20091211svn%{?dist}
 Summary:        Tool for remotely controlling a web browser to test a web site
 
 Group:          Development/Tools
@@ -13,6 +13,8 @@ URL:            http://seleniumhq.org/pr
 Source0:        selenium-remote-control.tar.gz
 Source1:        selenium-server.script
 Patch0:         server-coreless-1.0.1-tests.patch
+# http://jira.openqa.org/browse/SRC-802
+Patch1:         http://jira.openqa.org/secure/attachment/12212/selenium-safari-cleanup.patch
 BuildRoot:      %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
 
 BuildRequires:  ant-trax
@@ -54,6 +56,8 @@ proxy for web requests from them.
 %prep
 %setup -q -n %{name}
 %patch0 -p0 -b .test
+cd server-coreless
+%patch1 -p1 -b .safari-cleanup
 
 
 %build
@@ -111,6 +115,9 @@ rm -rf $RPM_BUILD_ROOT
 
 
 %changelog
+* Tue Jan 05 2010 Lubomir Rintel (GoodData) <lubo.rintel at gooddata.com> - 1.0.2-0.3.20091211svn
+- Fix Safari cleanup
+
 * Fri Dec 11 2009 Lubomir Rintel (GoodData) <lubo.rintel at gooddata.com> - 1.0.2-0.2.20091211svn
 - Fux up dependencies
 - Add launcher script