Skip to content
Snippets Groups Projects

file: add SIZE option

Merged dublet requested to merge dublet/cmake:file-size into master
All threads resolved!

This commit adds the SIZE option to file(). It returns the file size of the given path if it exists and produces an error if not.

Merge request reports

Pipeline #124693 failed

Pipeline failed for 12b471e8 on dublet:file-size

Merged by Kitware RobotKitware Robot 6 years ago (Nov 28, 2018 2:05pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • dublet added 1 commit

    added 1 commit

    Compare with previous version

  • dublet added 1 commit

    added 1 commit

    Compare with previous version

  • Craig Scott resolved all discussions

    resolved all discussions

  • Do: stage

  • Brad King
  • Brad King
  • Do: unstage

  • added workflow:wip label and removed workflow:nightly-testing label

  • dublet added 1 commit

    added 1 commit

    Compare with previous version

  • Errors:

    • commit 03db5af3 is not allowed because the following files are not formatted according to the 'clang-format-6.0' check: Source/cmFileCommand.cxx. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.

    Please rewrite commits to fix the errors listed above (adding fixup commits will not resolve the errors) and force-push the branch again to update the merge request.

  • Author Contributor

    Do: reformat

  • This topic has been reformatted and pushed; please fetch from the source repository and reset your local branch to continue with further development on the reformatted commits.

  • Kitware Robot added 1 commit

    added 1 commit

    Compare with previous version

  • @brad.king I might have to bounce this one back to you, I may not get a chance to look at it properly for a few days. The one query I commented on earlier seems to have been addressed, so if you're happy with everything else then please go ahead with staging/merging.

  • assigned to @brad.king

  • Brad King resolved all discussions

    resolved all discussions

  • Do: test

  • Brad King added workflow:in-review label and removed workflow:wip label

    added workflow:in-review label and removed workflow:wip label

  • This merge request has been queued for testing. Test results may be viewed on:

    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: 12b471e8

  • Do: stage

  • Do: merge

  • Brad King mentioned in commit 5daf7d92

    mentioned in commit 5daf7d92

  • merged

  • Please register or sign in to reply
    Loading