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

Add passing bash version to read_vars_file #729

Merged
merged 2 commits into from
Nov 18, 2024

Conversation

BolunThompson
Copy link
Contributor

@BolunThompson BolunThompson commented Nov 12, 2024

Use the bash version to determine how to parse the vars file as discussed in binpash/sh-expand#7. Should be merged at the same time as those changes (and will fail any tests until then).

@angelhof
Copy link
Member

angelhof commented Nov 13, 2024

Thanks for this change! We can make it ready for review since the sh-expand library v0.1.7 is deployed to PyPI! Could you change the bash version to be read only once here (https://github.com/binpash/pash/blob/future/pa.sh)? This script runs in bash so it should be able to determine the version and save it in a variable. This is better than hiding the call to bash deep in the compiler.

@BolunThompson BolunThompson marked this pull request as ready for review November 18, 2024 16:53
Copy link

OS =
CPU =
Ram =
Hash = 6eb1e15
Kernel=
||
|-|-|-|-|-|-|-|-|-|

Copy link

OS:ubuntu-20.04
Mon Nov 18 16:56:28 UTC 2024
intro: 2/2 tests passed.
interface: 41/41 tests passed.
compiler: 54/54 tests passed.

@angelhof
Copy link
Member

Thanks @BolunThompson ! Could you sign the commits for the DCO to pass? Then I will merge the PR :)

Copy link

OS =
CPU =
Ram =
Hash = 4426066
Kernel=
||
|-|-|-|-|-|-|-|-|-|

Copy link

OS:ubuntu-20.04
Mon Nov 18 17:58:36 UTC 2024
intro: 2/2 tests passed.
interface: 41/41 tests passed.
compiler: 54/54 tests passed.

@angelhof angelhof merged commit d5a3b13 into binpash:future Nov 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants