Two bashisms in ReleaseCppUTest.sh

[cpputest]

Two bashisms in ReleaseCppUTest.sh

[basvodde]

Uhm.. .sorry for the slowness this time @rhertzog. It is the opposite as the previous request.

I checked in this and I can fix it, but perhaps I ought to just change the shebang to bash instead of sh. Is there any good reason for not doing that?

[rhertzog]

On Wed, 19 Dec 2012, Bas Vodde wrote:

I checked in this and I can fix it, but perhaps I ought to just change
the shebang to bash instead of sh. Is there any good reason for not
doing that?

Changing the shebang is OK too, of course. It just means that bash
is a new dependency of cpputest as opposed to just having a POSIX
conformant /bin/sh.

Cheers,

Raphaël Hertzog ◈ Debian Developer

Get the Debian Administrator's Handbook:
→ http://debian-handbook.info/get/

[basvodde]

Yeah, main reason is that the other scripts in the script folder already seem to have a bash shebang. So therefore it wouldn't make any difference. Correct?

[rhertzog]

On Wed, 19 Dec 2012, Bas Vodde wrote:

Yeah, main reason is that the other scripts in the script folder already
seem to have a bash shebang. So therefore it wouldn't make any
difference. Correct?

Right.

Cheers,

Raphaël Hertzog ◈ Debian Developer

Get the Debian Administrator's Handbook:
→ http://debian-handbook.info/get/

[basvodde]

Oki, changed it to bash and close this issue.