Merge lp:~percona-dev/percona-server/bugfix735423 into lp:~percona-dev/percona-server/5.1.55

Proposed by Aleksandr Kuzminsky
Status: Merged
Approved by: Ignacio Nin
Approved revision: no longer in the source branch.
Merge reported by: Ignacio Nin
Merged at revision: not available
Proposed branch: lp:~percona-dev/percona-server/bugfix735423
Merge into: lp:~percona-dev/percona-server/5.1.55
Diff against target: 28 lines (+7/-4)
1 file modified
build/debian/percona-server-server-5.1.postinst (+7/-4)
To merge this branch: bzr merge lp:~percona-dev/percona-server/bugfix735423
Reviewer Review Type Date Requested Status
Ignacio Nin (community) Approve
Percona developers Pending
Review via email: mp+53416@code.launchpad.net

Description of the change

Fixed Bug 735423: Change files owner to mysql after mysql_install_db is run by percona-server-server-5.1.postinst

To post a comment you must log in.
Revision history for this message
Vadim Tkachenko (vadim-tk) wrote :

Merge should go into /5.1.56

I am asking Nacho for review

Revision history for this message
Ignacio Nin (ignacio-nin) wrote :

Updated the commit with an additional fix, merged into 5.1.56

Revision history for this message
Ignacio Nin (ignacio-nin) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'build/debian/percona-server-server-5.1.postinst'
2--- build/debian/percona-server-server-5.1.postinst 2010-12-20 20:03:55 +0000
3+++ build/debian/percona-server-server-5.1.postinst 2011-03-17 20:31:44 +0000
4@@ -115,10 +115,10 @@
5 # The mysql_datadir must not be writable by the mysql user under any
6 # circumstances as it contains scripts that are executed by root.
7 set +e
8- chown -R 0:0 $mysql_datadir
9- chown -R mysql $mysql_statedir
10- chown -R mysql $mysql_rundir
11- chown -R mysql:adm $mysql_newlogdir; chmod 2750 $mysql_newlogdir;
12+ chown -RL 0:0 $mysql_datadir
13+ chown -RL mysql $mysql_statedir
14+ chown -RL mysql $mysql_rundir
15+ chown -RL mysql:adm $mysql_newlogdir; chmod 2750 $mysql_newlogdir;
16 for i in log err; do
17 touch $mysql_logdir/mysql.$i
18 chown mysql:adm $mysql_logdir/mysql.$i
19@@ -139,6 +139,9 @@
20 # Debian: can safely run on upgrades with existing databases
21 set +e
22 /bin/bash /usr/bin/mysql_install_db --rpm 2>&1 | $ERR_LOGGER
23+ chown -RL mysql $mysql_statedir
24+ chown -RL mysql $mysql_rundir
25+ chown -RL mysql:adm $mysql_newlogdir; chmod 2750 $mysql_newlogdir;
26 if [ "$?" != "0" ]; then
27 echo "ATTENTION: An error has occured. More info is in the syslog!"
28 fi

Subscribers

People subscribed via source and target branches