Bug 208952 – [CMake] Warning when ccache is not installed

• Status: RESOLVED
• Resolution: FIXED
• Component: CMake

[Michael Catanzaro]

This warning is not useful:

CMake Warning at Source/cmake/WebKitCCache.cmake:25 (message):
  Enabling ccache: Couldn't find ccache program.  Not enabling it.
Call Stack (most recent call first):
  Source/cmake/WebKitCommon.cmake:51 (include)
  CMakeLists.txt:169 (include)

[Michael Catanzaro]

Created attachment 393301 [details]
Patch

[Daniel Bates]

Comment on attachment 393301 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=393301&action=review

> ChangeLog:10
> +        Change the warning when ccache is not installed to a status message. We shouldn't be

Same things as in bug 208970, comment 3, Again patch is OK as-is. NO CHANGE NEEDED.

[[
This is ok as-is. No change is needed. The optimal ChangeLog would move all the text in this line and later ABOVE the file name so as to be the description of the change because:

1. There is only one file changed in this patch so there's no need for per file comments.

2. Per file/function comments should be concise, no longer than a single paragraph. They are meant to provide more fine grain descriptions of the charges in the patch.
]]

[Michael Catanzaro]

Thanks Daniel.

I see I'm getting rusty with ChangeLogs....

[Michael Catanzaro]

Created attachment 393584 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 393584 [details]
Patch for landing

Clearing flags on attachment: 393584

Committed r258467: <https://trac.webkit.org/changeset/258467>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.