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. Opinions welcome :) Thanks, Niels > > > > -------- Weitergeleitete Nachricht -------- > Betreff: Bug#873878: glusterfs-client: mount.glusterfs needs bash as > /bin/sh > Weitersenden-Datum: Thu, 31 Aug 2017 20:21:01 +0000 > Weitersenden-Von: Michael Lundkvist <brels.deb...@solske.net> > Weitersenden-An: debian-bugs-d...@lists.debian.org > Weitersenden-CC: Patrick Matthäi <pmatth...@debian.org> > Datum: Thu, 31 Aug 2017 21:46:18 +0200 > Von: Michael Lundkvist <brels.deb...@solske.net> > Antwort an: Michael Lundkvist <brels.deb...@solske.net>, > 873...@bugs.debian.org > An: Debian Bug Tracking System <sub...@bugs.debian.org> > > > > Package: glusterfs-client > Version: 3.12.0-1 > Severity: serious > Tags: upstream > Justification: Policy 10.4 > > Version 3.12 of Glusterfs adds code in /sbin/mount.glusterfs that depends on > bash. > > With dash as /bin/sh, I get the following error message when trying to mount > a glusterfs volume: > > /sbin/mount.glusterfs: 667: /sbin/mount.glusterfs: Bad substitution > > Line 667 is: > 667 [ ${volume_str:0:1} = '/' ] && { > > Modifying mount.glusterfs to use /bin/bash makes it possible to mount again. > > /Micke > > > -- System Information: > Debian Release: buster/sid > APT prefers unstable > APT policy: (500, 'unstable') > Architecture: amd64 (x86_64) > > Kernel: Linux 4.12.0-1-amd64 (SMP w/4 CPU cores) > Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), > LANGUAGE=en_US:en (charmap=UTF-8) > Shell: /bin/sh linked to /bin/dash > Init: systemd (via /run/systemd/system) > > Versions of packages glusterfs-client depends on: > ii fuse 2.9.7-1 > ii glusterfs-common 3.12.0-1 > ii libc6 2.24-17 > ii libssl1.1 1.1.0f-5 > ii python 2.7.13-2 > > glusterfs-client recommends no packages. > > glusterfs-client suggests no packages. > > -- no debconf information > > _______________________________________________ > packaging mailing list > packaging@gluster.org > http://lists.gluster.org/mailman/listinfo/packaging _______________________________________________ packaging mailing list packaging@gluster.org http://lists.gluster.org/mailman/listinfo/packaging