Re: [gentoo-user] profile-sync-daemon "bad substitution" at boot

[Mick]

On Thursday, 6 September 2018 09:22:35 BST Alexander Kapshuk wrote:
> On Thu, Sep 6, 2018 at 11:18 AM Alexander Kapshuk
>
> <alexander.kapshuk@×××××.com> wrote:
> > On Thu, Sep 6, 2018 at 10:48 AM Samuraiii <samurai.no.dojo@×××××.com>
wrote:
> > > On 5.9.2018 21:20, Mick wrote:
> > >
> > > Hi All,
> > >
> > > I just noticed a psd error at boot time:
> > >
> > > /usr/bin/profile-sync-daemon: line 325: ${#DIRArr[@]##*/}: bad
> > > substitution
> > >
> > > Any idea what's brought this about?
> > >
> > > Hi,
> > >
> > > my bet is that your /bin/sh != /bin/bash and
> > > /usr/bin/profile-sync-daemon starts with #!/bin/sh.
> > >
> > > The other way around /usr/bin/profile-sync-daemon contains bashishms.
> > >
> > > You can try (as a test) to change #!/bin/sh for #!/bin/bash...
> > >
> > > S
> >
> > The shebang in the master repository is '#!/bin/bash'.
> > https://raw.githubusercontent.com/graysky2/profile-sync-daemon/master/comm
> > on/profile-sync-daemon.in
> >
> > The error message is a shell parameter expansion syntax error, which
> > comprises two different expansion constructs referenced below.
> > ${#parameter}
> > String Length.
> >
> > ${parameter##[word]}
> > Remove Largest Prefix Pattern.
> >
> > See section 2.6.2 Parameter Expansion,
> > http://pubs.opengroup.org/onlinepubs/9699919799/nframe.html.
> >
> > The script in the master repository referenced above does not contain
> > the line in question. It has '${#DIRArr[@]/}' and '${DIRArr[@]##*/}'.
>
> This appears to have been reported as a bug in Debian [1]
> [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=839049
> And applied upstream [2].
> https://github.com/graysky2/profile-sync-daemon/pull/185/files/654f5359abfe7
> aab2e1616dd74c8ba0f76ce4ab4

Thanks Alexander, good catch!
-- 
Regards,
Mick