[PATCH util-modular 2/4] release.sh: move the bit to extract the section into a function

Peter Hutterer peter.hutterer at who-t.net
Thu May 29 15:24:47 PDT 2014


On Thu, May 29, 2014 at 05:18:32PM -0400, Gaetan Nadon wrote:
> On 14-05-29 12:51 AM, Peter Hutterer wrote:
> > No functional changes intended
> >
> > Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> > ---
> >  release.sh | 128 +++++++++++++++++++++++++++++++++++--------------------------
> >  1 file changed, 74 insertions(+), 54 deletions(-)
> >
> > diff --git a/release.sh b/release.sh
> > index abfcc29..a05b0c9 100755
> > --- a/release.sh
> > +++ b/release.sh
> > @@ -195,6 +195,63 @@ process_modules() {
> >  #------------------------------------------------------------------------------
> >  #			Function: process_module
> >  #------------------------------------------------------------------------------
> > +# Code 'return 0' on success
> > +# Code 'return 1' on error
> > +# Sets global variable $section
> > +get_section() {
> > +    local module_url
> > +    local full_module_url
> Local variables are not supported by plain Bourne Shell
> 
> If there is a consensus to reduce the script compatibility, the top
> comment should be changed to set the new lowest common denominator:
> 
>     # Note on portability:
>     # This script is intended to run on any platform supported by X.Org.
>     # Basically, it should be able to run in a Bourne shell.
> 
> This is the same requirement for build.sh for which I heard people still
> needing.

I was wondering about that, but at least bash --posix was happy with it and
I'm struggling how to test pure Bourne shell compatibility. I'd give a
big +1 for requiring bash for these scripts.

This isn't some super-portable script that needs to run on everything, it's
a script that is manually run by a maintainer. And I rather pity the
maintainer that can't install a proper shell.

Cheers,
   Peter


More information about the xorg-devel mailing list