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

[2.3.x] build(cmake): Fix CMake warning regarding extracted archive timestamps #10864

Merged
merged 1 commit into from
Sep 4, 2022

Conversation

Holzhaus
Copy link
Member

@Holzhaus Holzhaus commented Sep 3, 2022

Accidently opened #10855 against main instead of 2.3. I rebased, because this warning also happens occurs when building 2.3.

Fixes warnings like these for project that were added using the
`ExternalProject` module (i.e. libdjinterop and KeyFinder):

    CMake Warning (dev) at /usr/share/cmake/Modules/ExternalProject.cmake:3071 (message):
      The DOWNLOAD_EXTRACT_TIMESTAMP option was not given and policy CMP0135 is
      not set.  The policy's OLD behavior will be used.  When using a URL
      download, the timestamps of extracted files should preferably be that of
      the time of extraction, otherwise code that depends on the extracted
      contents might not be rebuilt if the URL changes.  The OLD behavior
      preserves the timestamps from the archive instead, but this is usually not
      what you want.  Update your project to the NEW behavior or specify the
      DOWNLOAD_EXTRACT_TIMESTAMP option with a value of true to avoid this
      robustness issue.
    Call Stack (most recent call first):
      /usr/share/cmake/Modules/ExternalProject.cmake:4167 (_ep_add_download_command)
      CMakeLists.txt:1949 (ExternalProject_Add)
    This warning is for project developers.  Use -Wno-dev to suppress it.
@Holzhaus Holzhaus added the build label Sep 3, 2022
Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

@daschuer daschuer merged commit 87d6906 into mixxxdj:2.3 Sep 4, 2022
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.

2 participants