Skip to content

config/path: removed erroneous brackets #983

Merged
merged 1 commit into from Jul 17, 2012

4 participants

@adam111316

No description provided.

@sraue sraue merged commit 5a7d126 into OpenELEC:master Jul 17, 2012
@vpeter4

Did you check what happens with this change? Probably you didn't or it could be that my 2 systems behaves differently.
What looks like error doesn't mean it really is :-)

Ok, it seems my change works only in bash but not in dash. Seems we need better fix.

Whats about

if [ "$(echo $PKG_DIR |wc -w)" -gt "1" ]; then
It only give false positives if we have targets with spaces in...

EDIT: sorry, the backticks don't show up - so I used the meta "$(...)"

What about

if [ "${PKG_DIR}" != "$(echo $PKG_DIR | tr -d '\n')" ]; then
  echo "Error - multiple package folders:"
  echo "$PKG_DIR"
  exit 1
fi
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.