Postinst overrides permissions on /var/lib/mysql-files

Hi Sigurd,
this is present up to the latest version of mysql.
So I'm adding tasks for them as well.

I remember having seen the discussion about ucf or something else to be used for that but it was low-gain/high-effort and therefore so far neglected. I can't find that old bug, I will update here if I do so.

In my (personal and humble) opinion this is a valid, but low prio issue.
Leaving it for Lars (Maintainer) to decide on it.

In general I think if one comes up with a working patch people are willing to review&test but I'd not expect dev's to jump onto this as it might cause as much trouble (e.g. fail after upgrade as it can#t start on the custom permission dir).

Thank you Christian! And sorry for the late follow-up.

I agree that this is low prio, but low hanging fruit can be tasty too:-)

I have made a simple patch that envelopes the chmod and chown inside the if that runs mkdir.

I can't see any scenario where this change will have negative effects.

kind regards,
-sigurd

The attachment "Diff for DEBIAN/postinst for mysql-5.*" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

This is basically how upstream packaging does it (with some additional restrictions: https://github.com/mysql/mysql-server/blob/5.7/packaging/deb-in/extra/mysql-helpers#L76); if the directory already exists it isn't touched.
There are some restrictions on what the permissions on the secure-file-priv location should be (the server will log a warning if it considers the directory insecure), but the proposed change will just make it more in line with the upstream packaging.

The patch needs an indentation fix, though :)

Thanks Lars!

And indeed it was that pesky indentation again;-) This new one should be better

-sig