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.

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

missing "\" at the end of "|" #4

Closed
haifengkao opened this Issue Nov 9, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@haifengkao

haifengkao commented Nov 9, 2017

xcodebuild -destination 'platform=iOS Simulator,name=iPhone 8' \
  -sdk iphonesimulator -project YourProject.xcodeproj \
  -scheme Chuck -configuration Debug \
  clean build \
  OTHER_SWIFT_FLAGS="-driver-time-compilation \
    -Xfrontend -debug-time-function-bodies \
    -Xfrontend -debug-time-compilation" |  <- should have "\" here?
tee profile.log
@fastred

This comment has been minimized.

Show comment
Hide comment
@fastred

fastred Nov 9, 2017

Owner

It shouldn't be needed after a pipe operator. I tested this under Bash and zsh. What shell do you use?

Owner

fastred commented Nov 9, 2017

It shouldn't be needed after a pipe operator. I tested this under Bash and zsh. What shell do you use?

@haifengkao

This comment has been minimized.

Show comment
Hide comment
@haifengkao

haifengkao Nov 9, 2017

without \, it will hang indefinitely
2017-11-09 5 49 11

my zsh:
zsh 5.3.1 (x86_64-apple-darwin17.0.0)

haifengkao commented Nov 9, 2017

without \, it will hang indefinitely
2017-11-09 5 49 11

my zsh:
zsh 5.3.1 (x86_64-apple-darwin17.0.0)

@haifengkao

This comment has been minimized.

Show comment
Hide comment
@haifengkao

haifengkao commented Nov 9, 2017

with \
2017-11-09 5 52 39

@fastred

This comment has been minimized.

Show comment
Hide comment
@fastred

fastred Nov 11, 2017

Owner

I'm somehow not able to reproduce this.

Does this not return the correct result for you too?

echo "1\n2" |
wc
Owner

fastred commented Nov 11, 2017

I'm somehow not able to reproduce this.

Does this not return the correct result for you too?

echo "1\n2" |
wc
@yakovlevmaksym

This comment has been minimized.

Show comment
Hide comment
@yakovlevmaksym

yakovlevmaksym Nov 12, 2017

I have the same issue. Should use '' after '|'.

Shell: fish.

yakovlevmaksym commented Nov 12, 2017

I have the same issue. Should use '' after '|'.

Shell: fish.

@fastred fastred closed this in 46a6241 Nov 13, 2017

fastred added a commit that referenced this issue Nov 13, 2017

Merge pull request #8 from fastred/missing-back-slashes
Fix #4: scripts not working on some shells
@fastred

This comment has been minimized.

Show comment
Hide comment
@fastred

fastred Nov 13, 2017

Owner

Thanks for reporting! Should be fixed now on master.

Owner

fastred commented Nov 13, 2017

Thanks for reporting! Should be fixed now on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment