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
arrow-cpp: init at 0.8.0 #37196
arrow-cpp: init at 0.8.0 #37196
Conversation
@GrahamcOfBorg build arrow-cpp |
Success on x86_64-linux (full log) Attempted: arrow-cpp Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: arrow-cpp Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
version = "0.8.0"; | ||
|
||
src = fetchurl { | ||
url = "http://apache.mesi.com.ar/arrow/arrow-${version}/apache-arrow-${version}.tar.gz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the best mirror?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed to use mirror://
# detected by gcc7 | ||
NIX_CFLAGS_COMPILE = [ "-Wno-error=implicit-fallthrough" ]; | ||
preConfigure = '' | ||
substituteInPlace CMakeLists.txt --replace "-Werror" "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment why we need to disable errors completely?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could add
-Wno-error=zero-as-null-pointer-constant
and -Wno-error=shadow
to use with modern clang, but what is the point? Compilers will get more sophisticated diagnostics, there will be more warnings and it's not like we actually going to fix any of the code. So we should just let developers compile with -Werror
Hey, thanks for pulling this out! |
Success on x86_64-darwin (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
description = "A cross-language development platform for in-memory data"; | ||
homepage = https://arrow.apache.org/; | ||
license = stdenv.lib.licenses.asl20; | ||
platforms = stdenv.lib.platforms.unix; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you going to maintain this expression?
Success on x86_64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
@GrahamcOfBorg build python2Packages.pyarrow python35Packages.pyarrow python36Packages.pyarrow |
Failure on x86_64-linux (full log) Attempted: python2Packages.pyarrow The following builds were skipped because they don't evaluate on x86_64-linux: python35Packages.pyarrow, python36Packages.pyarrow Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: python2Packages.pyarrow The following builds were skipped because they don't evaluate on x86_64-darwin: python35Packages.pyarrow, python36Packages.pyarrow Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: arrow-cpp, gflags, rapidjson Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: arrow-cpp, gflags, parquet-cpp, rapidjson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, gflags, parquet-cpp, rapidjson Partial log (click to expand)
|
I can't reproduce darwin failure for pyarrow, it builds just fine for me on mac. This seems to be good to merge. |
Failure on aarch64-linux (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Disables -Werror Alternatively one could use NIX_CFLAGS_COMPILE = [ "-Wno-error=implicit-fallthrough" "-Wno-error=zero-as-null-pointer-constant" "-Wno-error=shadow" ];
cfb804b
to
6b855cc
Compare
Also fix RPATH issues in pyarrow
Success on x86_64-linux (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
@GrahamcOfBorg build python2Packages.pyarrow |
Failure on x86_64-linux (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
@GrahamcOfBorg build python2Packages.pyarrow |
Failure on x86_64-darwin (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: arrow-cpp, parquet-cpp, rapidjson Partial log (click to expand)
|
moved to #38440 |
Failure on aarch64-linux (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python2Packages.pyarrow Partial log (click to expand)
|
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)