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

Version test in configure/configure uses non-BRE regex, triggering failure to match #112

Open
Gumnos opened this issue Jan 2, 2025 · 0 comments · May be fixed by #113
Open

Version test in configure/configure uses non-BRE regex, triggering failure to match #112

Gumnos opened this issue Jan 2, 2025 · 0 comments · May be fixed by #113

Comments

@Gumnos
Copy link

Gumnos commented Jan 2, 2025

The configure file tests the version using non-BRE (Basic Regular Expression) \+ tokens, causing it not to match with OpenBSD's grep. Extracting the relevant log output during the cargo build execution:

configure:8265: error: 5.3.0-1-ge13ca993e8ccb9ba9847cc330696e02839f328f7 does not match <major>.<minor>.<bugfix>-<nrev>-g<gid> or VERSION                                                                                          

The grep on line 8259 needs to use actual BRE (just using the * and doubling up the previous atom to require one or more):

echo "${with_version}" | grep '^[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*-[0-9][0-9]*-g[0-9a-f][0-9a-f]*$'

or (possibly less-portably) you could use -E for Extended Regular Expressions and just + (not \+) such as

echo "${with_version}" | grep -E '^[0-9]+\.[0-9]+\.[0-9]+-[0-9]+-g[0-9a-f]+$'
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 a pull request may close this issue.

1 participant