Re: Remove unneeded bash dependency

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Daniel Camolês <bigatojj(at)gmail(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Remove unneeded bash dependency
Date: 2016-03-07 13:52:14
Message-ID: CA+OCxowmSPs4JPPocD9rLRhzE2MSaFwixhiPc2seTPnW2VpDiw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Mon, Mar 7, 2016 at 1:40 PM, Daniel Camolês <bigatojj(at)gmail(dot)com> wrote:
> Hello guys,
>
> I tried to compile
> https://ftp.postgresql.org/pub/pgadmin3/release/v1.22.1/src/pgadmin3-1.22.1.tar.gz
> on OpenBSD 5.8 and it failed because the shell script
> pgadmin/ver_svn.sh shebang pointed to /bin/bash, which I don't have.
> Looking into the script, it doesn't use any bash specific idiom. I
> made a symbolic link /bin/bash pointing to /bin/sh and the build
> process completed successfully. I grepped the rest of the source for
> /bin/bash and the other places where bash is used are:
>
> bootstrap
> pkg/mac/create-dmg.sh
> pkg/mac/debug-bundle.sh
>
> I didn't try to use the bootstrap script because I didn't clone from a
> git repository. The mac references I have no way to test because I
> don't have a mac.
>
> Thus, for the sake of portability, I suggest that the
> pgadmin/ver_svn.sh file be modified to use /bin/sh instead, since it
> uses only simple sh structures.

Thanks - I've push a fix for this to the master and 1.22 branches.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2016-03-07 18:50:33 Re: PATCH: Schema/Catalog Node [pgAdmin4]
Previous Message Dave Page 2016-03-07 13:51:46 pgAdmin III commit: Use /bin/sh rather than /bin/bash for portability.