Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • CMake CMake
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,929
    • Issues 3,929
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 21
    • Merge requests 21
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • External wiki
    • External wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • CMake
  • CMakeCMake
  • Merge requests
  • !1072

File copy/install permissions code modifying timestamps

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Robert Dailey requested to merge rcdailey/cmake:master into master Jul 20, 2017
  • Overview 5
  • Commits 1
  • Pipelines 2
  • Changes 1

When invoking file(COPY) or file(INSTALL), the flow is:

Copy File -> Modify Timestamps -> Set Permissions

Now the flow is:

Copy File -> Set Permissions -> Modify Timestamps

Setting permissions after modifying the timestamps causes the timestamps to be modified again, resulting in the timestamps being set to the current clock time instead of the timestamp of the source file of the copy.

Additionally, none of the above 3 operations are performed if the two file timestamps (source & destination) are the same.

Fixes #17087 (closed)

Topic-rename: update-permissions-before-updating-timestamps

Edited Jul 21, 2017 by Ben Boeckel
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master