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

[Clipper2] Update from v1.0.4 to v1.2.2 #7093

Merged
merged 8 commits into from
Jul 30, 2023

Conversation

ferrolho
Copy link
Contributor

No description provided.

This was referenced Jul 22, 2023
@giordano
Copy link
Member

Sounds like the shared library isn't built?

@ferrolho
Copy link
Contributor Author

I tried running the BinaryBuilder.jl wizard locally, but I am getting the following error at the cmake stage:

sandbox:${WORKSPACE}/srcdir/Clipper2/CPP/build # cmake -DCMAKE_INSTALL_PREFIX=$prefix -DCMAKE_TOOLCHAIN_FILE=${CMAKE_TARGET_TOOLCHAIN} -DCMAKE_BUILD_TYPE=Release -DCLIPPER2_TESTS=OFF -DCLIPPER2_EXAMPLES=OFF -DCLIPPER2_UTILS=OFF ..
Re-run cmake no build system arguments
CMake Error at /usr/share/cmake/Modules/Compiler/._ADSP-DetermineCompiler.cmake:1:
  Parse error.  Expected a command name, got bad character with text "".
Call Stack (most recent call first):
  /usr/share/cmake/Modules/CMakeCompilerIdDetection.cmake:6 (include)
  /usr/share/cmake/Modules/CMakeCompilerIdDetection.cmake:33 (_readFile)
  /usr/share/cmake/Modules/CMakeDetermineCompilerId.cmake:282 (compiler_id_detection)
  /usr/share/cmake/Modules/CMakeDetermineCompilerId.cmake:301 (CMAKE_DETERMINE_COMPILER_ID_WRITE)
  /usr/share/cmake/Modules/CMakeDetermineCompilerId.cmake:6 (CMAKE_DETERMINE_COMPILER_ID_BUILD)
  /usr/share/cmake/Modules/CMakeDetermineCompilerId.cmake:59 (__determine_compiler_id_test)
  /usr/share/cmake/Modules/CMakeDetermineCCompiler.cmake:123 (CMAKE_DETERMINE_COMPILER_ID)
  CMakeLists.txt:2 (project)


-- Configuring incomplete, errors occurred!
See also "/workspace/srcdir/Clipper2/CPP/build/CMakeFiles/CMakeOutput.log".

Also, I manually had to create the patch file locally within the Docker container. I wanted to add the bundled folder as a DirectorySource during the wizard setup, but I couldn't find a way to do it.

@ferrolho ferrolho force-pushed the hf/clipper2-v1.2.2 branch from bc9edb9 to a1cc625 Compare July 30, 2023 14:28
@giordano giordano enabled auto-merge (squash) July 30, 2023 22:36
@giordano giordano merged commit e219018 into JuliaPackaging:master Jul 30, 2023
@ferrolho ferrolho deleted the hf/clipper2-v1.2.2 branch July 30, 2023 22:53
@ferrolho
Copy link
Contributor Author

For completeness, the earlier issues were due to a missing SHARED keyword on an add_library instruction and also a stupid mistake from my part in which I was creating the patch file from a git diff w.r.t. the master branch of Clipper2 instead of w.r.t. the latest tagged version.

Special thanks to @giordano for the help during the very well known JuliaCon @ Boston Logan hackathon.

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.

3 participants