Skip to content

Include vendored nanoarrow source in Hex package#159

Merged
josevalim merged 1 commit intolivebook-dev:mainfrom
trevenrawr:nanoarrow-src-in-hex-pkg
Apr 24, 2026
Merged

Include vendored nanoarrow source in Hex package#159
josevalim merged 1 commit intolivebook-dev:mainfrom
trevenrawr:nanoarrow-src-in-hex-pkg

Conversation

@trevenrawr
Copy link
Copy Markdown
Contributor

Howdy, ADBC folks!

I have a bit of a special setup where I'm running on Alpine and need to build ADBC from source. I just upgraded to 0.11.0 and my build threw an error because the nanoarrow source doesn't seem to be included in the Hex package like the source for apache_arrow_adbc is: https://github.com/livebook-dev/adbc/blob/main/mix.exs#L114

I'm working around the issue with a git checkout and copying the vendored source over to my deps/adbc dir during my image build, but figured I should propose this change here in case nanoarrow is around to stay.

Thanks for all the awesome work!

Comment thread mix.exs
name: "adbc",
files:
~w(3rd_party/apache-arrow-adbc c_src lib mix.exs README* LICENSE* CMakeLists.txt Makefile Makefile.win checksum.exs),
~w(3rd_party/apache-arrow-adbc 3rd_party/nanoarrow c_src lib mix.exs README* LICENSE* CMakeLists.txt Makefile Makefile.win checksum.exs),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another option, perhaps:

Suggested change
~w(3rd_party/apache-arrow-adbc 3rd_party/nanoarrow c_src lib mix.exs README* LICENSE* CMakeLists.txt Makefile Makefile.win checksum.exs),
~w(3rd_party c_src lib mix.exs README* LICENSE* CMakeLists.txt Makefile Makefile.win checksum.exs),

@josevalim josevalim merged commit 0597f68 into livebook-dev:main Apr 24, 2026
3 checks passed
@josevalim
Copy link
Copy Markdown
Contributor

💚 💙 💜 💛 ❤️

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