build(rust/driver/snowflake): include go source in package, add env override#2786
build(rust/driver/snowflake): include go source in package, add env override#2786mbrobbel wants to merge 2 commits intoapache:mainfrom
Conversation
lidavidm
left a comment
There was a problem hiding this comment.
Hmm, in lieu of the symlink, I don't mind the old approach, so long as it can be overridden by the release script
Is there a reason you don't want the symlink? If we don't have a symlink we must either:
I think the package should be usable without doing anything special i.e. we should package the required source for the |
|
It may not be related to ADBC but... In general, UNIX style symlink doesn't work on Windows. For example, MSYS2 can't extract |
As long as we package the crate for publishing on linux it will work fine on systems that don't support symlinks. The files will be copied into the package: |
|
Symlinks in source archive may have a problem when msys2/MINGW-packages#21803 is implemented. It's not related to Rust crates. We can solve it by resolving symlinks in release process like apache/arrow does. |
Closes #2783.
This adds a symlink to the go source to the Snowflake driver, which means cargo will include it in the package. This means the build script now works after packaging. This also adds support to override this with an env var (
ADBC_GO_SRC_DIR).