AlloSphere-Research-Group / AlloSystem

AlloSystem is a cross-platform suite of C++ components for building interactive multimedia tools and applications.
http://mat.ucsb.edu/allosphere/software.php
BSD 3-Clause "New" or "Revised" License
65 stars 22 forks source link

Assimp3 install and POSIX standard compatibility #32

Closed DrewRWx closed 10 years ago

DrewRWx commented 10 years ago

(Note: Removing bashisms makes the scripts POSIX compliant which "ensures" compatibility with anything a platform symlinks /bin/sh to. )

I started fixing the assimp2 uninstall and assimp3 install steps and ended up removing all bashisms from allocore's install dependencies every shell script.

Most importantly, I modified run.sh and debug.sh .

A few points:

  1. The fixes to the mingw stanzas need to be tested by someone with that platform.
  2. I need to go over the other install dependencies scripts... and all "sh" scripts for that matter.
  3. We ensure that assimp3 is used because compilation completes, but is wrong if there is a assimp2 and assimp3 mismatch between headers and libraries.

Canonical's guide for porting from bash to dash was useful.

DrewRWx commented 10 years ago

@LancePutnam - I added the cleanup code back in.

@mantaraya36 - The last commit modifies run.sh and debug.sh for POSIX compliance based on the recommendation of the checkbashisms package.

DrewRWx commented 10 years ago

Hooked up a POSIX shell linter to Sublimetext and fixed remaining compatibility issues.