Page MenuHome

Cmake: Add option for doc readme
AbandonedPublic

Authored by Aaron Carlisle (Blendify) on Feb 14 2022, 6:47 AM.

Details

Summary

This adds an option to disable installing the readme.html.
This has been disabled for the developer config to avoid
installing the readme after every rebuild.

Diff Detail

Repository
rB Blender
Branch
cmake-doc-readme (branched from master)
Build Status
Buildable 20537
Build 20537: arc lint + arc unit

Event Timeline

Aaron Carlisle (Blendify) requested review of this revision.Feb 14 2022, 6:47 AM
Aaron Carlisle (Blendify) created this revision.
Campbell Barton (campbellbarton) requested changes to this revision.Feb 14 2022, 11:41 AM

It would be good to avoid this extra option if possible. Could you check on P2794 ?
This only updates the file if it's older than the files it depends on.

(requesting changes since I'd rather avoid this).

This revision now requires changes to proceed.Feb 14 2022, 11:41 AM

@Aaron Carlisle (Blendify) feel free to take the relevant code from D13863 and land it.

Ray Molenkamp (LazyDodo) requested changes to this revision.EditedFeb 14 2022, 9:00 PM

cmake's file(GENERATE and/or configure_file will only overwrite a file when the contents have changed, i'd prefer that over P2794 and this patch

for example use see build_files\cmake\platform\platform_win32_bundle_crt.cmake

Closing in favor of D13863