Skip to content

Adjust build.rs to not overwrite temp_wrapper.h when using same build version#25

Closed
ION28 wants to merge 1 commit into
chdb-io:mainfrom
ION28:main
Closed

Adjust build.rs to not overwrite temp_wrapper.h when using same build version#25
ION28 wants to merge 1 commit into
chdb-io:mainfrom
ION28:main

Conversation

@ION28

@ION28 ION28 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Closes #24

@CLAassistant

CLAassistant commented Apr 9, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@ION28

ION28 commented Apr 10, 2026

Copy link
Copy Markdown
Contributor Author

Unfortunately, I have decided not to sign the CLA due to Provision 11. You are welcome to take the code fix provided in my issue or fork if you would like to integrate it); however, until the repository is migrated to the ClickHouse Individual Contributor License Agreement v1.0, I will not be signing it.

@wudidapaopao

wudidapaopao commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Hi @ION28, thank you for the detailed report in #24 and for putting together this PR.

I understand your concerns regarding the CLA. We'll look into updating the CLA terms to something more standard and contributor-friendly.
In the meantime, I've submitted a separate PR with the fix to unblock this.

@ION28 ION28 closed this Apr 10, 2026
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.

Constant recompilation of chdb-rust caused by temp_wrapper.h timestamp updates

3 participants