Thanks to visit codestin.com
Credit goes to github.com

Skip to content

deeplab/local_test.sh: Use 'bash' instead of 'sh'#8534

Merged
YknZhu merged 1 commit into
tensorflow:masterfrom
ruslo:pr.bash
Jul 16, 2020
Merged

deeplab/local_test.sh: Use 'bash' instead of 'sh'#8534
YknZhu merged 1 commit into
tensorflow:masterfrom
ruslo:pr.bash

Conversation

@ruslo

@ruslo ruslo commented May 19, 2020

Copy link
Copy Markdown
Contributor

Description

Use 'bash' instead of 'sh' in deeplab/local_test.sh:

  • Make it consistent with '/bin/bash' shebang
  • Fix "Bad substitution" error in BASH_SOURCE[0]

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Tests

[research/deeplab]> bash local_test.sh

Test Configuration:

  • Tensorflow 1.15.2
  • Ubuntu 18.04.4 LTS
  • CUDA 10.1

Checklist

- Make it consistent with '/bin/bash' shebang
- Fix "Bad substitution" error in BASH_SOURCE[0]
@ruslo

ruslo commented Jul 16, 2020

Copy link
Copy Markdown
Contributor Author

@aquariusjay Let me know if there are any problems with this one

@YknZhu YknZhu merged commit 5f9bfd8 into tensorflow:master Jul 16, 2020
@YknZhu

YknZhu commented Jul 16, 2020

Copy link
Copy Markdown
Contributor

Thanks Ruslan!

@ruslo ruslo deleted the pr.bash branch July 16, 2020 17:23
kmindspark pushed a commit to kmindspark/models that referenced this pull request Jul 19, 2020
- Make it consistent with '/bin/bash' shebang
- Fix "Bad substitution" error in BASH_SOURCE[0]
kmindspark pushed a commit to kmindspark/models that referenced this pull request Jul 23, 2020
- Make it consistent with '/bin/bash' shebang
- Fix "Bad substitution" error in BASH_SOURCE[0]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants