Skip to content
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

Revert "Use ccache in build-in-docker by default" #227

Merged
merged 2 commits into from
May 7, 2022

Conversation

gerashegalov
Copy link
Collaborator

Reverts #225 because it crossed with conflicting #199

@gerashegalov gerashegalov requested review from revans2 and jlowe May 6, 2022 21:54
@jlowe jlowe added the build label May 6, 2022
@gerashegalov
Copy link
Collaborator Author

build

2 similar comments
@jlowe
Copy link
Member

jlowe commented May 6, 2022

build

@gerashegalov
Copy link
Collaborator Author

build

@pxLi
Copy link
Collaborator

pxLi commented May 7, 2022

@gerashegalov looks like you forgot to approve duo push in time~

also, both pre-merge and nightly just started failing

[2022-05-06T22:50:48.563Z] [INFO]      [exec] -- Looking for _M_ARM64

[2022-05-06T22:50:48.824Z] [INFO]      [exec] -- Looking for _M_ARM64 - not found

[2022-05-06T22:50:48.824Z] [INFO]      [exec] -- Looking for __SIZEOF_INT128__

[2022-05-06T22:50:49.086Z] [INFO]      [exec] -- Looking for __SIZEOF_INT128__ - found

[2022-05-06T22:50:49.086Z] [INFO]      [exec] CMake Error at /usr/local/cmake-3.20.5-linux-x86_64/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:230 (message):

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   Could NOT find Boost (missing: Boost_INCLUDE_DIR system filesystem)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   (Required is at least version "1.58")

[2022-05-06T22:50:49.086Z] [INFO]      [exec] Call Stack (most recent call first):

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   /usr/local/cmake-3.20.5-linux-x86_64/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   /usr/local/cmake-3.20.5-linux-x86_64/share/cmake-3.20/Modules/FindBoost.cmake:2345 (find_package_handle_standard_args)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   build/_deps/arrow-src/cpp/cmake_modules/FindBoostAlt.cmake:48 (find_package)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   build/_deps/arrow-src/cpp/cmake_modules/ThirdpartyToolchain.cmake:241 (find_package)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   build/_deps/arrow-src/cpp/cmake_modules/ThirdpartyToolchain.cmake:956 (resolve_dependency)

[2022-05-06T22:50:49.086Z] [INFO]      [exec]   build/_deps/arrow-src/cpp/CMakeLists.txt:546 (include)

image required to be updated after #199. I will re-generate image (not reach cron of internal image build pipeline yet) and re-trigger builds

@pxLi
Copy link
Collaborator

pxLi commented May 7, 2022

build

@pxLi pxLi merged commit ce99c9f into branch-22.06 May 7, 2022
Signed-off-by: Gera Shegalov <[email protected]>
@hyperbolic2346 hyperbolic2346 deleted the revert-225-ccache-in-docker branch August 5, 2022 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants