Skip to content

[vcpkg baseline][dpp] Add DISABLE_PARALLEL_CONFIGURE#26687

Merged
dan-shaw merged 1 commit intomicrosoft:masterfrom
Cheney-W:Dev/Cheney/dpp
Sep 6, 2022
Merged

[vcpkg baseline][dpp] Add DISABLE_PARALLEL_CONFIGURE#26687
dan-shaw merged 1 commit intomicrosoft:masterfrom
Cheney-W:Dev/Cheney/dpp

Conversation

@Cheney-W
Copy link
Copy Markdown
Contributor

@Cheney-W Cheney-W commented Sep 5, 2022

Describe the pull request

  • What does your PR fix?

    In VCPKG CI testing, dpp failed with following error:
  CMake Error at CMakeLists.txt:54 (configure_file):
  No such file or directory

This issue is because the upstream's CMakeLists.txt file contains following codes:
configure_file("${CMAKE_CURRENT_SOURCE_DIR}/src/dpp/dpp.rc.in" "${CMAKE_CURRENT_SOURCE_DIR}/src/dpp/dpp.rc" NEWLINE_STYLE WIN32)

The file operation here may cause the same file to be operated at the same time during parallel config, resulting in the above error. This issue could be fixed by adding DISABLE_PARALLEL_CONFIGURE in the config step.

@Cheney-W Cheney-W added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Sep 5, 2022
@JackBoosY JackBoosY marked this pull request as ready for review September 5, 2022 08:07
@JackBoosY
Copy link
Copy Markdown
Contributor

@JavierMatosD Please merge this PR first.

@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Sep 5, 2022
@dan-shaw dan-shaw merged commit bdb28dc into microsoft:master Sep 6, 2022
@Cheney-W Cheney-W deleted the Dev/Cheney/dpp branch October 9, 2022 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants