[GitHub] xhochy closed pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON

2018-06-19 Thread GitBox
xhochy closed pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON URL: https://github.com/apache/arrow-dist/pull/26 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] yahonda opened a new pull request #29: ARROW-2768: Support Ubuntu 18.04

2018-06-29 Thread GitBox
yahonda opened a new pull request #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29 This pull request provides Ubuntu 18.04 support. This is an automated message from the Apache Git

[GitHub] kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-07-02 Thread GitBox
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401697847 I've uploaded packages for Ubuntu 18.04 to packages.red-data-tools.org. Keep up the good work!

[GitHub] kou closed pull request #29: ARROW-2768: Support Ubuntu 18.04

2018-07-01 Thread GitBox
kou closed pull request #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-07-01 Thread GitBox
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401673189 It works well. I've merged. This is an automated message from the Apache Git Service. To

[GitHub] yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-07-01 Thread GitBox
yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401674535 Thanks for the follow-up and merging. This is an automated message from the Apache Git

[GitHub] kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-07-01 Thread GitBox
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401662209 Umm. Parallel build may cause build failure. This is an automated message from the Apache

[GitHub] kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-07-01 Thread GitBox
kou commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401663340 I've disabled parallel build. If it works well, I'll merge this. This is an automated

[GitHub] kou closed pull request #27: [deb] Stop to use -Wdate-time

2018-06-20 Thread GitBox
kou closed pull request #27: [deb] Stop to use -Wdate-time URL: https://github.com/apache/arrow-dist/pull/27 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from

[GitHub] kou commented on issue #27: [deb] Stop to use -Wdate-time

2018-06-20 Thread GitBox
kou commented on issue #27: [deb] Stop to use -Wdate-time URL: https://github.com/apache/arrow-dist/pull/27#issuecomment-398978251 The build failure isn't related. This is an automated message from the Apache Git Service. To

[GitHub] kou opened a new pull request #27: [deb] Stop to use -Wdate-time

2018-06-19 Thread GitBox
kou opened a new pull request #27: [deb] Stop to use -Wdate-time URL: https://github.com/apache/arrow-dist/pull/27 Because FlatBuffers uses `__DATE__` and `__TIME__` and it also uses `-Werror` flag. See also "timeless" value description of "reproducible" in

[GitHub] kou opened a new pull request #28: Support specifying downloaded file name

2018-06-26 Thread GitBox
kou opened a new pull request #28: Support specifying downloaded file name URL: https://github.com/apache/arrow-dist/pull/28 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] kou closed pull request #28: Support specifying downloaded file name

2018-06-26 Thread GitBox
kou closed pull request #28: Support specifying downloaded file name URL: https://github.com/apache/arrow-dist/pull/28 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] kou commented on issue #28: Support specifying downloaded file name

2018-06-26 Thread GitBox
kou commented on issue #28: Support specifying downloaded file name URL: https://github.com/apache/arrow-dist/pull/28#issuecomment-400542712 The build failure of manylinux1 is unrelated. This is an automated message from the

[GitHub] yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04

2018-06-29 Thread GitBox
yahonda commented on issue #29: ARROW-2768: Support Ubuntu 18.04 URL: https://github.com/apache/arrow-dist/pull/29#issuecomment-401519060 cc @kou who suggested to open a pull request and a JIRA ticket. This is an automated

[GitHub] kou closed pull request #22: Drop Ubuntu 17.04 support

2018-01-25 Thread GitBox
kou closed pull request #22: Drop Ubuntu 17.04 support URL: https://github.com/apache/arrow-dist/pull/22 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] kou commented on issue #22: Drop Ubuntu 17.04 support

2018-01-25 Thread GitBox
kou commented on issue #22: Drop Ubuntu 17.04 support URL: https://github.com/apache/arrow-dist/pull/22#issuecomment-360671739 Travis CI is green again. Merged. This is an automated message from the Apache Git Service. To

[GitHub] kou opened a new pull request #22: Drop Ubuntu 17.04 support

2018-01-25 Thread GitBox
kou opened a new pull request #22: Drop Ubuntu 17.04 support URL: https://github.com/apache/arrow-dist/pull/22 Because it's EOL-ed. This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] kou opened a new pull request #21: Fix .typelib path for Ubuntu trusty

2018-01-25 Thread GitBox
kou opened a new pull request #21: Fix .typelib path for Ubuntu trusty URL: https://github.com/apache/arrow-dist/pull/21 Reported by Mike Sam. Thanks!!! This is an automated message from the Apache Git Service. To respond to

[GitHub] kou closed pull request #21: Fix .typelib path for Ubuntu trusty

2018-01-25 Thread GitBox
kou closed pull request #21: Fix .typelib path for Ubuntu trusty URL: https://github.com/apache/arrow-dist/pull/21 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] kou commented on issue #21: Fix .typelib path for Ubuntu trusty

2018-01-25 Thread GitBox
kou commented on issue #21: Fix .typelib path for Ubuntu trusty URL: https://github.com/apache/arrow-dist/pull/21#issuecomment-360459875 +1 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] kou opened a new pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON

2018-06-18 Thread GitBox
kou opened a new pull request #26: [RPM] Remove duplicated -DARROW_ORC=ON URL: https://github.com/apache/arrow-dist/pull/26 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] xhochy closed pull request #25: Fix OSX RPATHs for Boost

2018-04-05 Thread GitBox
xhochy closed pull request #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] xhochy commented on issue #25: Fix OSX RPATHs for Boost

2018-04-04 Thread GitBox
xhochy commented on issue #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-378660695 This is now ready to be merged and uploaded to PyPI. This is an automated message from the

[GitHub] xhochy commented on issue #25: Fix OSX RPATHs for Boost

2018-04-04 Thread GitBox
xhochy commented on issue #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-378660872 I will do the upload and verify beforehand that the wheels really work. This is an automated

[GitHub] wesm commented on issue #25: Fix OSX RPATHs for Boost

2018-03-31 Thread GitBox
wesm commented on issue #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-377731322 Is this ready to go? This is an automated message from the Apache Git Service. To respond to the

[GitHub] xhochy commented on issue #25: Fix OSX RPATHs for Boost

2018-04-01 Thread GitBox
xhochy commented on issue #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-33355 I still need to incorporate @cpcloud comments and we also will need to set `SETUPTOOLS_SCM_PRETEND_VERSION=0.9.0.post1` as we cannot reupload `0.9.0` again to

[GitHub] cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost

2018-03-29 Thread GitBox
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132589 ## File path: python-macos/config.sh ## @@ -65,7 +75,23 @@ function build_wheel { install > /dev/null 2>&1 popd

[GitHub] xhochy commented on issue #25: Fix OSX RPATHs for Boost

2018-03-29 Thread GitBox
xhochy commented on issue #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#issuecomment-377303277 OSX wheels are working again but we probably need to override the version to 0.9.0.post1 to upload it to PyPI.

[GitHub] cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost

2018-03-29 Thread GitBox
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132935 ## File path: python-macos/config.sh ## @@ -153,9 +187,19 @@ function build_wheel { --bundle-arrow-cpp

[GitHub] cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost

2018-03-29 Thread GitBox
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178132846 ## File path: python-macos/config.sh ## @@ -153,9 +187,19 @@ function build_wheel { --bundle-arrow-cpp

[GitHub] cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost

2018-03-29 Thread GitBox
cpcloud commented on a change in pull request #25: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25#discussion_r178151267 ## File path: python-macos/config.sh ## @@ -65,7 +75,23 @@ function build_wheel { install > /dev/null 2>&1 popd

[GitHub] wesm closed pull request #23: Build wheels for 0.9.0

2018-03-24 Thread GitBox
wesm closed pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-25 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375968445 We don't have any automation for this yet. We should add a step for this in the Linux container. If I have time, I can try to add it today.

[GitHub] kou commented on issue #23: Build wheels for 0.9.0

2018-03-25 Thread GitBox
kou commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375968543 I see. Thank! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] xhochy opened a new pull request #25: WIP: Fix OSX RPATHs for Boost

2018-03-28 Thread GitBox
xhochy opened a new pull request #25: WIP: Fix OSX RPATHs for Boost URL: https://github.com/apache/arrow-dist/pull/25 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-21 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375068142 @xhochy @kou I need some help with this. There's a ton of stuff here that I'm unfamiliar with especially the details of yum and apt-get packages. I don't have

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375291195 Taking a look at this now This is an automated message from the Apache Git Service. To respond to

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176430763 ## File path: cpp-linux/yum/arrow.spec.in ## @@ -220,6 +225,13 @@ Documentation for Apache Arrow GLib. %endif %changelog

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375323642 @xhochy looks like upgrading bison might broken thrift generation on osx: ```

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375323642 @xhochy looks like upgrading bison might broken thrift generation: ```

[GitHub] xhochy commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176458928 ## File path: python-macos/config.sh ## @@ -21,37 +21,66 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176430349 ## File path: cpp-linux/debian/rules ## @@ -23,8 +23,11 @@ override_dh_auto_configure:

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176435008 ## File path: scripts/python-wheels-windows.bat ## @@ -80,23 +79,25 @@ pushd parquet-cpp\build cmake -G "%GENERATOR%" ^

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203 ## File path: python-macos/config.sh ## @@ -21,37 +21,66 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176449003 ## File path: cpp-linux/yum/arrow.spec.in ## @@ -60,13 +60,17 @@ mkdir cpp/build cd cpp/build %cmake3 .. \

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176431502 ## File path: scripts/python-wheels-windows.bat ## @@ -80,23 +79,25 @@ pushd parquet-cpp\build cmake -G "%GENERATOR%" ^

[GitHub] xhochy commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176431729 ## File path: scripts/python-wheels-windows.bat ## @@ -80,23 +79,25 @@ pushd parquet-cpp\build cmake -G "%GENERATOR%" ^

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203 ## File path: python-macos/config.sh ## @@ -21,37 +21,66 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176446260 ## File path: python-macos/config.sh ## @@ -21,37 +21,66 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] kou commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
kou commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176446024 ## File path: cpp-linux/yum/arrow.spec.in ## @@ -60,13 +60,17 @@ mkdir cpp/build cd cpp/build %cmake3 .. \

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375325408 I think it also needs a newer `flex` version on OSX, too. This is an automated message from the

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176441255 ## File path: cpp-linux/debian/rules ## @@ -23,8 +23,11 @@ override_dh_auto_configure:

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176440203 ## File path: python-macos/config.sh ## @@ -21,37 +21,66 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375358552 @xhochy Is there any problem with installing thrift from homebrew? This is an automated message

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375357003 Why the heck do we keep getting random 404s? This is an automated message from the Apache Git

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360715 > Then I'm also able pass in the BISON and FLEX executable via CMake variables. Does that actually work? `find_package(FLEX)` explicitly sets this variable

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360715 > Then I'm also able pass in the BISON and FLEX executable via CMake variables. Does that actually work? `find_package(FLEX)` explicitly sets this variable

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375377611 > > Then I'm also able pass in the BISON and FLEX executable via CMake variables. > Does that actually work? find_package(FLEX) explicitly sets this

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375360027 @cpcloud We actually want Thrift to use the same namespaced Boost. I would change our build scripts to explicitly build thrift. Then I'm also able pass in the

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375382433 `parquet-cpp`'s `thrift_ep` does not pick up the correct Boost version: https://issues.apache.org/jira/browse/PARQUET-1252 thus I have no moved the thrift build

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176430659 ## File path: cpp-linux/yum/arrow.spec.in ## @@ -60,13 +60,17 @@ mkdir cpp/build cd cpp/build %cmake3 .. \

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176430659 ## File path: cpp-linux/yum/arrow.spec.in ## @@ -60,13 +60,17 @@ mkdir cpp/build cd cpp/build %cmake3 .. \

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375324912 Actually, it's finding the wrong version. This is an automated message from the Apache Git

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176527524 ## File path: python-macos/config.sh ## @@ -21,37 +21,94 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375405785 @xhochy is there any way we can silence all of the `bcp`/`bjam` output? It will make for one fewer indirection when debugging travis.

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375408993 @cpcloud no idea if they have a be-non-verbose option but I'm ok with piping their stdout to `/dev/null` as they seem to make their work in a reliable way.

[GitHub] xhochy commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176509215 ## File path: appveyor.yml ## @@ -29,11 +29,10 @@ environment: PYTHON: "3.6" ARCH: "64" MSVC_DEFAULT_OPTIONS:

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176514837 ## File path: appveyor.yml ## @@ -29,11 +29,10 @@ environment: PYTHON: "3.6" ARCH: "64" MSVC_DEFAULT_OPTIONS:

[GitHub] xhochy commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176527820 ## File path: python-macos/config.sh ## @@ -21,37 +21,94 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176529648 ## File path: python-macos/config.sh ## @@ -21,37 +21,94 @@ set -e function build_wheel { echo `pwd` +export

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375416775 working on the windows failure now This is an automated message from the Apache Git Service. To

[GitHub] xhochy commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
xhochy commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375433610 I think we don't need to run delocate over the wheel, we should bundle all libs already in the wheel. We could skip that and I can have a try tomorrow to see if

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375430607 Oh man, we are getting so close on the OS X builds! https://travis-ci.org/apache/arrow-dist/jobs/357034487 Looks like the wheel delocator cannot find

[GitHub] wesm commented on issue #23: Build wheels for 0.9.0

2018-03-21 Thread GitBox
wesm commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-374970680 LGTM, but do we want to bundle the required Boost shared libraries with the Windows and macOS wheels?

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375488961 @xhochy It looks like the wheels include the libs we need: https://travis-ci.org/cpcloud/arrow-dist/jobs/357120681 Still have to figure out why

[GitHub] cpcloud commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
cpcloud commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176859450 ## File path: scripts/python-wheels-windows.bat ## @@ -40,7 +37,7 @@ pushd %ARROW_SRC% @rem fix up symlinks git config

[GitHub] wesm commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
wesm commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176854345 ## File path: scripts/python-wheels-windows.bat ## @@ -40,7 +37,7 @@ pushd %ARROW_SRC% @rem fix up symlinks git config

[GitHub] wesm commented on a change in pull request #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
wesm commented on a change in pull request #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#discussion_r176854385 ## File path: scripts/python-wheels-windows.bat ## @@ -54,24 +51,26 @@ mkdir %ARROW_SRC%\cpp\build pushd %ARROW_SRC%\cpp\build

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375668074 I think we may somehow have some additional whitespace in the `Boost_*_LIBRARY` variables on windows. I'm going to try and build a boost-bundled pyarrow today

[GitHub] wesm commented on issue #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
wesm commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375830903 You can see why this is broken right here: https://github.com/apache/arrow/blob/master/python/setup.py#L196 These should not be coupled. I will fix in master

[GitHub] wesm commented on issue #23: Build wheels for 0.9.0

2018-03-23 Thread GitBox
wesm commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375831209 See https://issues.apache.org/jira/browse/ARROW-2349 This is an automated message from the Apache

[GitHub] wesm commented on issue #23: Build wheels for 0.9.0

2018-03-21 Thread GitBox
wesm commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-374986924 Conceivably there would be the possibility of ABI conflicts, but that would be true if we were statically linking also. We could go the namespaced Boost route like

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-21 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-374983807 @wesm Might be a good idea. How will that interact with existing boost installations? This is an

[GitHub] kou opened a new pull request #24: Package .deb and .rpm for 0.9.0

2018-03-22 Thread GitBox
kou opened a new pull request #24: Package .deb and .rpm for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/24 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] kou commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
kou commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375198224 I created pull request for .deb and .rpm as #24. I'll merge #24 after I confirmed that it works on Travis CI. Can you rebase and remove cpp-linux staffs after

[GitHub] kou commented on issue #24: Package .deb and .rpm for 0.9.0

2018-03-22 Thread GitBox
kou commented on issue #24: Package .deb and .rpm for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/24#issuecomment-375204453 I've confirmed that it works on Travis CI. This is an automated message from the Apache Git

[GitHub] kou commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
kou commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375204488 Done. This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] kou closed pull request #24: Package .deb and .rpm for 0.9.0

2018-03-22 Thread GitBox
kou closed pull request #24: Package .deb and .rpm for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/24 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] cpcloud commented on issue #23: Build wheels for 0.9.0

2018-03-22 Thread GitBox
cpcloud commented on issue #23: Build wheels for 0.9.0 URL: https://github.com/apache/arrow-dist/pull/23#issuecomment-375488961 @xhochy It looks like the wheels include the libs we need: https://travis-ci.org/cpcloud/arrow-dist/jobs/357120681 Still have to figure out why

[GitHub] yahonda opened a new pull request #30: ARROW-3600: Support Ubuntu 18.10

2018-10-23 Thread GitBox
yahonda opened a new pull request #30: ARROW-3600: Support Ubuntu 18.10 URL: https://github.com/apache/arrow-dist/pull/30 This pull request provides Ubuntu 18.10 support. This is an automated message from the Apache Git

[GitHub] kou closed pull request #30: ARROW-3600: Support Ubuntu 18.10

2018-10-23 Thread GitBox
kou closed pull request #30: ARROW-3600: Support Ubuntu 18.10 URL: https://github.com/apache/arrow-dist/pull/30 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] kou commented on issue #30: ARROW-3600: Support Ubuntu 18.10

2018-10-23 Thread GitBox
kou commented on issue #30: ARROW-3600: Support Ubuntu 18.10 URL: https://github.com/apache/arrow-dist/pull/30#issuecomment-432491625 Thanks! But we've moved build scripts to https://github.com/apache/arrow/tree/master/dev/tasks/linux-packages . Can you work on apache/arrow?

[GitHub] yahonda commented on issue #30: ARROW-3600: Support Ubuntu 18.10

2018-10-23 Thread GitBox
yahonda commented on issue #30: ARROW-3600: Support Ubuntu 18.10 URL: https://github.com/apache/arrow-dist/pull/30#issuecomment-432492475 Sure. Thanks for the information. This is an automated message from the Apache Git

[GitHub] andygrove opened a new pull request #1: ARROW-4444: [Testing] Add test files that originated in DataFusion

2019-01-31 Thread GitBox
andygrove opened a new pull request #1: ARROW-: [Testing] Add test files that originated in DataFusion URL: https://github.com/apache/arrow-testing/pull/1 This is an automated message from the Apache Git Service. To

[GitHub] xhochy commented on a change in pull request #1: ARROW-4444: [Testing] Add test files that originated in DataFusion

2019-01-31 Thread GitBox
xhochy commented on a change in pull request #1: ARROW-: [Testing] Add test files that originated in DataFusion URL: https://github.com/apache/arrow-testing/pull/1#discussion_r252946771 ## File path: data/csv/uk_cities.csv ## @@ -0,0 +1,37 @@ +"Elgin, Scotland, the

[GitHub] andygrove commented on a change in pull request #1: ARROW-4444: [Testing] Add test files that originated in DataFusion

2019-02-01 Thread GitBox
andygrove commented on a change in pull request #1: ARROW-: [Testing] Add test files that originated in DataFusion URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253068821 ## File path: data/csv/uk_cities.csv ## @@ -0,0 +1,37 @@ +"Elgin, Scotland, the

[GitHub] xhochy commented on a change in pull request #1: ARROW-4444: [Testing] Add test files that originated in DataFusion

2019-02-01 Thread GitBox
xhochy commented on a change in pull request #1: ARROW-: [Testing] Add test files that originated in DataFusion URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253072622 ## File path: data/csv/uk_cities.csv ## @@ -0,0 +1,37 @@ +"Elgin, Scotland, the

[GitHub] andygrove commented on a change in pull request #1: ARROW-4444: [Testing] Add test files that originated in DataFusion

2019-02-01 Thread GitBox
andygrove commented on a change in pull request #1: ARROW-: [Testing] Add test files that originated in DataFusion URL: https://github.com/apache/arrow-testing/pull/1#discussion_r253242388 ## File path: data/csv/uk_cities.csv ## @@ -0,0 +1,37 @@ +"Elgin, Scotland, the

[GitHub] andygrove commented on issue #1: ARROW-4444: [Testing] Add CSV test file for use by DataFusion unit tests and examples

2019-02-02 Thread GitBox
andygrove commented on issue #1: ARROW-: [Testing] Add CSV test file for use by DataFusion unit tests and examples URL: https://github.com/apache/arrow-testing/pull/1#issuecomment-459979050 @xhochy I deleted all the files and started again. There is now a single generated file. I have

  1   2   3   >