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.

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

Circular dependency on bash within 'scripts/install-build-tools.sh' #287

Open
1 task done
rockett-m opened this issue Jul 19, 2024 · 0 comments
Open
1 task done
Assignees
Labels
feedback/logistics A suggested change in the project's governance or model of collaboration

Comments

@rockett-m
Copy link
Contributor

Abstract

We can't rely on bash as the shebang if we install bash within the script:scripts/install-build-tools.sh

Proposal: switch the shebang to #!/bin/sh or #!/usr/bin/env sh to eliminate circular dependency on bash. This may trigger several other errors as well. Could also have a mini script to install bash first so that scripts/install-build-tools.sh can remain as is.

Issue stemmed from a discussion with @maurermi about this

Description

Changing the shebang to #!/bin/sh triggered issues right away, for example:

| 0.091 scripts/install-build-tools.sh: 13: !=: not found
| 0.091 scripts/install-build-tools.sh: 19: Syntax error: "(" unexpected
| ------
| Dockerfile:25
| --------------------
|   23 |     WORKDIR /opt/tx-processor
|   24 |     
|   25 | >>> RUN scripts/install-build-tools.sh
|   26 |     RUN scripts/setup-dependencies.sh
|   27 |     

So we should consider keeping the same shell requirement and a script to get bash installed first or changing it to sh and fixing all the issues that arise.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@rockett-m rockett-m added the feedback/logistics A suggested change in the project's governance or model of collaboration label Jul 19, 2024
@rockett-m rockett-m changed the title install-build-tools.sh uses #!/bin/bash as shebang but installs bash in the script [sh dep] Circular dependency on bash within 'scripts/install-build-tools.sh' Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback/logistics A suggested change in the project's governance or model of collaboration
Projects
None yet
Development

No branches or pull requests

2 participants