[gluster-packaging] Fwd: Bug#873878: glusterfs-client: mount.glusterfs needs bash as /bin/sh

[Niels de Vos]

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