Opened 16 years ago

Closed 16 years ago

Last modified 5 years ago

#8838 closed patch

Solaris /bin/sh doesn't like $()

Reported by: SF/lblume Owned by: jvprat
Priority: normal Component: Port: Solaris
Version: Keywords:
Cc: Game:

Description

A new way to display option has been introduced in the ./configure script, that uses the shell $() syntax for execution. However, Solaris' /bin/sh is old and crusty, and does not support that, which leads to the following result when --help is used:

--$(echo disable-scumm don't build the SCUMM engine --$(echo disable-scumm_ exclude the v7 & v8 games in SCUMM engine --$(echo disable-he | s exclude the HE71+ games in SCUMM engine --$(echo disable-agi | don't build the AGI engine

I attached an obvious patch to fix that, should work on any shell.

Ticket imported from: #1901742. Ticket imported from: patches/943.

Attachments (1)

configure_solaris_shell.patch​ (433 bytes ) - added by SF/lblume 16 years ago.
Small change to fix a cosmetic issue with Solaris /bin/sh

Download all attachments as: .zip

Change History (6)

by SF/lblume, 16 years ago

Attachment: configure_solaris_shell.patch​ added

Small change to fix a cosmetic issue with Solaris /bin/sh

comment:1 by fingolfin, 16 years ago

Owner: set to jvprat

comment:2 by jvprat, 16 years ago

Keywords: script removed
Status: new → closed

comment:3 by jvprat, 16 years ago

Commited as is. Thanks for the patch!

comment:4 by digitall, 5 years ago

Component: → Ports

comment:5 by digitall, 5 years ago

Component: Ports → Port: Solaris
Note: See TracTickets for help on using tickets.