Re: [PATCH] bashisms prevent build in doc/Makefile
arielCo <[email protected]>
| Newsgroups | gmane.comp.db.tds.freetds |
|---|---|
| Message-ID | <CAMvU69K9BM=4LzRYG77tion=KtEqar0ijXr7EKdGQ284XXTMxg@mail.gmail.com> |
Actually that's the only Makefile.am that defines SHELL. The rest use the value of $SHELL from the environment, so any other bash-isms would be concealed since most users run bash or ksh. Ariel Cornejo +58·412·8083546 On Thu, Jun 20, 2013 at 5:23 PM, Frediano Ziglio <[email protected]> wrote: > Applied! > > Thanks. > > I would ask why this file set shell. Don't really know, change was 9f107033 > where a doc/Makefile was converted to doc/Makefile.am > > > 2013/6/20 arielCo <[email protected]> > >> There are a few bash-isms in doc/Makefile.am that prevent build (which >> specifies SHELL = /bin/sh). Things like " NAME=$$(echo $@ | sed >> 's/\..*$$//') ". >> >> The attached patch fixes them (based on the tarball from >> http://gitorious.org/freetds/freetds/trees/master). Maybe all >> Makefiles should use /bin/sh? >> >> Ariel Cornejo >> +58·412·8083546 >> >> _______________________________________________ >> FreeTDS mailing list >> [email protected] >> http://lists.ibiblio.org/mailman/listinfo/freetds >> >> > _______________________________________________ > FreeTDS mailing list > [email protected] > http://lists.ibiblio.org/mailman/listinfo/freetds