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

[Bug] configure development script generates an 'sh' shell error due to an initialized variable (ac_cv_func_fseeko) #516

Closed
1 of 2 tasks
edespino opened this issue Jul 12, 2024 · 2 comments · Fixed by #543
Labels
type: Bug Something isn't working

Comments

@edespino
Copy link
Contributor

Cloudberry Database version

main (HEAD)

What happened

The following sh scripting error (unary operator expected) is seen when configure is run:

checking for __crc32cb, __crc32ch, __crc32cw, and __crc32cd with CFLAGS=-march=armv8-a+crc... no
checking which CRC-32C implementation to use... SSE 4.2
./configure: line 21391: test: =: unary operator expected
checking size of off_t... (cached) 8
checking for library containing sem_init... none required

What you think should happen instead

The missing variable ac_cv_func_fseeko should be defined and the error should not happen.

How to reproduce

Simply run configure script.

Operating System

Platform independent (reproduced on Rocky Linux 9.4)

Anything else

No response

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct

@edespino edespino added the type: Bug Something isn't working label Jul 12, 2024
Copy link

Hey, @edespino welcome!🎊 Thanks for taking the time to point this out.🙌

@gfphoenix78
Copy link
Contributor

Hi @edespino , the variable ac_cv_func_fseeko is not set by the macro AC_FUNC_FSEEKO. The pg upstream has fixed this issue. I opened a PR to align the code with pg upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants