Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

common/build.sh: line 34: syntax error near unexpected token `;;' #35

Closed
omron93 opened this issue Oct 9, 2017 · 6 comments
Closed

common/build.sh: line 34: syntax error near unexpected token `;;' #35

omron93 opened this issue Oct 9, 2017 · 6 comments

Comments

@omron93
Copy link
Contributor

omron93 commented Oct 9, 2017

Reported in sclorg/s2i-python-container#220. So moving there.

on mac osx 10.12.6
$make build TARGET=centos7 VERSIONS=2.7
VERSION="2.7" SKIP_SQUASH=1 UPDATE_BASE= OS=centos7 CLEAN_AFTER= DOCKER_BUILD_CONTEXT=. OPENSHIFT_NAMESPACES="3.3" common/build.sh
common/build.sh: line 34: syntax error near unexpected token `;;'
make[1]: *** [2.7] Error 2
@omron93 omron93 mentioned this issue Oct 9, 2017
Closed
@praiskup
Copy link
Contributor

praiskup commented Oct 9, 2017

@evilbinary hi, thanks for the report. Can you specify what version of bash is on your system?
Can you experiment a bit with the code in build.sh?

@praiskup
Copy link
Contributor

praiskup commented Oct 9, 2017

Resp. it looks like the >( ) construct isn't recognized by your bash; can you post the output from echo >(:) so we can see that shell-managed fifo is created?

@evilbinary
Copy link

[evil]$echo >(:)
/dev/fd/63
[evil]$bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin16)
Copyright (C) 2007 Free Software Foundation, Inc.

I found that $( case statment ) not support.

@praiskup
Copy link
Contributor

Ah, OK -> seems like older bashes had problems with echo $(case x in x) echo x ;; esac), parser probably takes the first closing ) bracket as end of '$()'.

How difficult is to install up2date bash (version 4+) on your box?

praiskup added a commit to praiskup/container-common-scripts that referenced this issue Oct 10, 2017
@praiskup
compat: avoid case statements in $()
17d2320 
This is because mac still ships with obsoleted /bin/bash v3.

Fixes: sclorg#35
@praiskup praiskup mentioned this issue Oct 10, 2017
Merged
praiskup added a commit to praiskup/container-common-scripts that referenced this issue Oct 10, 2017
@praiskup
compat: avoid case statements in $()
0305758 
This is because mac still ships with obsoleted /bin/bash v3.

Fixes: sclorg#35
praiskup added a commit to praiskup/container-common-scripts that referenced this issue Oct 31, 2017
@praiskup
compat: avoid case statements in $()
b547562 
This is because mac still ships with obsoleted /bin/bash v3.

Fixes: sclorg#35
pkubatrh pushed a commit that referenced this issue Nov 3, 2017
@praiskup @pkubatrh
compat: avoid case statements in $()
605b2ff 
This is because mac still ships with obsoleted /bin/bash v3.

Fixes: #35
@afnan9632
Copy link

Hey please help ne solve this error
./phish.sh: line 430: syntax error near unexpected token ;;' ./phish.sh: line 430: ;;'

@snowtta
Copy link

snowtta commented Oct 6, 2020

./phish sh: line 430:,...,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@praiskup @omron93 @evilbinary @afnan9632 @snowtta