[: 57: =: unexpected operator

[riseuplabs]

On starting etherpad-lite I see this:

npm WARN [email protected] package.json: bugs['web'] should probably be bugs['url']
npm WARN [email protected] package.json: bugs['web'] should probably be bugs['url']
[email protected] ./node_modules/async
[email protected] ./node_modules/gzip
[email protected] ./node_modules/log4js
[email protected] ./node_modules/formidable
[email protected] ./node_modules/uglify-js
[email protected] ./node_modules/clean-css
└── [email protected]
[email protected] ./node_modules/ueberDB
├── [email protected]
├── [email protected]
└── [email protected]
[email protected] ./node_modules/express
├── [email protected]
├── [email protected]
├── [email protected]
└── [email protected]
[email protected] ./node_modules/socket.io
├── [email protected]
├── [email protected]
└── [email protected]
[email protected] ./node_modules/joose
├── [email protected]
├── [email protected] ([email protected])
└── [email protected]
Ensure jQuery is downloaded and up to date...
[: 57: =: unexpected operator
Clear minfified cache...
ensure custom css/js files are created...
start...

[Pita]

I know that I've seen that too. But I can't reproduce it anymore

[Pita]

I think this only happens if you upgrade from an older jquery version

[riseuplabs]

Its true I was using 1.6 and went to 1.7, but this looks like a shell script test problem. I would be able to figure it out if I understood what shell script was being run here.

[Pita]

Its installDeps.sh

[riseuplabs]

Ok, its because this line:

VERSION=$(cat static/js/jquery.min.js | head -n 1 | grep -o "v[0-9].[0-9]");

VERSION is not getting set to anything, so the test that follows:

if [ ${VERSION#v} = $NEEDED_VERSION ]; then

fails because $VERSION is empty.

Now, why that test fails...

[Pita]

yeah, but how to resolve this?

[riseuplabs]

Ok, the test fails with jquery 1.7 because head -1 produces:

/*

if that head -1 was replaced with head -3, I sent a pull request

[Pita]

I merged your fix. Thx