[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2018-02-20 Thread Robie Basak
** Changed in: mysql-5.7 (Ubuntu) Status: Confirmed => Triaged -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1734129 Title: Postinst overrides permissions on /var/lib/mysql-files To manage

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2018-01-09 Thread Sigurd Urdahl
Thanks Lars! And indeed it was that pesky indentation again;-) This new one should be better -sig ** Patch added: "improved intendation" https://bugs.launchpad.net/ubuntu/+source/mysql-5.5/+bug/1734129/+attachment/5033439/+files/postinst.diff -- You received this bug notification because

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2018-01-09 Thread Lars Tangvald
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

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2017-12-12 Thread Ubuntu Foundations Team Bug Bot
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

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2017-12-12 Thread Sigurd Urdahl
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.

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2017-11-24 Thread ChristianEhrhardt
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

[Bug 1734129] Re: Postinst overrides permissions on /var/lib/mysql-files

2017-11-24 Thread ChristianEhrhardt
In general I think if one comes up with a working patch people are willing to review 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). -- You received this bug notification because you are a