On Fri, Sep 01, 2017 at 03:54:05PM +0200, Michael Lundkvist wrote: > On 09/01/2017 01:25 PM, Patrick Matthäi wrote: > > > > Am 01.09.2017 um 11:40 schrieb Niels de Vos: > > > On Fri, Sep 01, 2017 at 09:36:16AM +0200, Patrick Matthäi wrote: > > > > Hi, > > > > > > > > how should it be fixed for glusterfs now? Better shell code without > > > > bashishm or do you want /bin/bash as shebang? > > > Do you have a preference? I do not know how much work is it is to > > > rewrite the mount.glusterfs script to remove all the Bashisms. At least > > > in the Debian builds you may want to patch it to /bin/bash for the time > > > being. > > > > I would prefer a patch, so that it works without bash. Luckily the bug > > reporter just wrote a patch :) > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=873878#10 > > > > To be very clear, I've only tested this by manually modifying > /sbin/mount.glusterfs in the same way. I've not even tried building it. > > Also, this code seems related to the new sub-directory mounts that came in > 3.12 and I'm not using that. > > But at least it takes the package from unusable to fully functional for me. > :)
Thanks! Kaleb filed https://bugzilla.redhat.com/show_bug.cgi?id=1487830 for this and posted the patch for review. Once it has been merged it will be backported to 3.12 as well (through Bug 1488119). Niels _______________________________________________ packaging mailing list packaging@gluster.org http://lists.gluster.org/mailman/listinfo/packaging
