Sutou Kouhei

Results 417 comments of Sutou Kouhei

英語で説明できる気がしないので日本語ですみません。。。 ユーザーとしての意見だと、提案してもらった方法は自分で各ホスト毎のsaltを生成する必要があって面倒だなぁと感じます。 (環境変数を使うと他のプラグインが`spawn`したりしたときに別プロセスにも情報が渡ってしまいやすくなるので、この方向で実現(設定ファイルの中にRubyのコードを書いて実現)するときは、私は`File.read("/path/to/salt-file")`みたいにすると思います。) 1開発者の意見だと、多少ユーザーが不便になったとしても使用や実装をシンプルに保ちたいというのは理解できるので、このプロジェクトがそういう方針であれば、このpull requestは取り下げます。 という感じです!

I tried https://github.com/microsoft/vcpkg/issues/22552#issuecomment-1093273619 (that is based on Apache Arrow 7.0.0). The following dependencies are missed from link command line: * `vcpkg_installed/x64-linux/debug/lib/libthriftd.a` * `vcpkg_installed/x64-linux/debug/lib/liblz4d.a` * `vcpkg_installed/x64-linux/debug/lib/libbrotli{enc,dec,common}-static.a` It seems that `thrift::thrift`, `lz4::lz4`...

The following patch for Apache Arrow may fix the `vcpkg_installed/x64-linux/debug/lib/libthriftd.a` (`PARQUET_REQUIRE_ENCRYPTION`) case: ```diff diff --git a/cpp/src/parquet/CMakeLists.txt b/cpp/src/parquet/CMakeLists.txt index 542fa5bc08..88f270b6b1 100644 --- a/cpp/src/parquet/CMakeLists.txt +++ b/cpp/src/parquet/CMakeLists.txt @@ -221,6 +221,8 @@ if(NOT PARQUET_MINIMAL_DEPENDENCY)...

@JackBoosY I've created it: https://github.com/apache/arrow/pull/13863

@assignUser Sorry. My patch had a typo: `PARQUET_STATIC_INTERFACE_INSTALL_LIBS` should have been `PARQUET_STATIC_INSTALL_INTERFACE_LIBS`. This may not be related to `PARQUET_REQUIRE_ENCRYPTION`. I can reproduce the missing `libthrift.a` with my local build without...

Rebased. I hope that all CI jobs are green...

The AppVeyor job is green but some Travis CI jobs are still failed. But the same Travis CI jobs on master are failed: * master: https://app.travis-ci.com/github/apache/thrift/builds/255364984 * This pull request:...

I don't like this approach. I want to build archive in CI instead of building archive on local. And I just want to download archive from CI and upload it...

https://github.com/groonga/groonga/commit/c63a7c23563c6e44aca2f60fda09520de95b2ea4 build and upload source archive but it doesn't support uploading to packages.groonga.org yet.