From b56e05d502f722094a723dd81a6b9d19bc97367a Mon Sep 17 00:00:00 2001
From: remi <fedora@famillecollet.com>
Date: Mon, 17 Sep 2012 16:34:56 +0200
Subject: [PATCH] remove session.save_path from php.ini

---
 php.conf | 6 ++++++
 php.ini  | 6 +++++-
 php.spec | 6 +++++-
 3 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/php.conf b/php.conf
index 0013860..bf19554 100644
--- a/php.conf
+++ b/php.conf
@@ -15,3 +15,9 @@ DirectoryIndex index.php
 # files as PHP source code:
 #
 #AddType application/x-httpd-php-source .phps
+
+#
+# Apache specific PHP configuration options
+# those can be override in each configured vhost
+#
+php_value session.save_path "/var/lib/php/session"
diff --git a/php.ini b/php.ini
index 400ea9d..30fff27 100644
--- a/php.ini
+++ b/php.ini
@@ -1350,7 +1350,11 @@ session.save_handler = files
 ; where MODE is the octal representation of the mode. Note that this
 ; does not overwrite the process's umask.
 ; http://php.net/session.save-path
-session.save_path = "/var/lib/php/session"
+
+; RPM note : session directory must be owned by process owner
+; for mod_php, see /etc/httpd/conf.d/php.conf
+; for php-fpm, see /etc/php-fpm.d/*conf
+;session.save_path = "/tmp"
 
 ; Whether to use cookies.
 ; http://php.net/session.use-cookies
diff --git a/php.spec b/php.spec
index 85f60d9..0764654 100644
--- a/php.spec
+++ b/php.spec
@@ -52,7 +52,7 @@
 Summary: PHP scripting language for creating dynamic web sites
 Name: php
 Version: 5.4.7
-Release: 1%{?dist}
+Release: 2%{?dist}
 License: PHP
 Group: Development/Languages
 URL: http://www.php.net/
@@ -1279,6 +1279,10 @@ fi
 
 
 %changelog
+* Mon Sep 17 2012 Remi Collet <remi@fedoraproject.org> 5.4.7-2
+- remove session.save_path from php.ini
+  move it to apache and php-fpm configuration files
+
 * Fri Sep 14 2012 Remi Collet <remi@fedoraproject.org> 5.4.7-1
 - update to 5.4.7
   http://www.php.net/releases/5_4_7.php