Bug 196916 - [CMake] Remove WEBKIT_WRAP_SOURCELIST
Summary: [CMake] Remove WEBKIT_WRAP_SOURCELIST
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CMake (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Don Olmstead
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-04-15 10:35 PDT by Don Olmstead
Modified: 2020-12-07 12:28 PST (History)
19 users (show)

See Also:


Attachments
Patch (9.22 KB, patch)
2019-04-15 10:43 PDT, Don Olmstead
no flags Details | Formatted Diff | Diff
Patch (9.57 KB, patch)
2019-04-19 14:54 PDT, Don Olmstead
no flags Details | Formatted Diff | Diff
Patch (9.54 KB, patch)
2019-04-19 14:58 PDT, Don Olmstead
no flags Details | Formatted Diff | Diff
Patch (9.09 KB, patch)
2020-12-07 11:26 PST, Don Olmstead
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Don Olmstead 2019-04-15 10:35:24 PDT
There doesn't appear to be any value for this macro so let's remove it.
Comment 1 Don Olmstead 2019-04-15 10:43:15 PDT Comment hidden (obsolete)
Comment 2 Konstantin Tokarev 2019-04-15 14:17:30 PDT
Comment on attachment 367424 [details]
Patch

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

> ChangeLog:7
> +

Justification is needed here, otherwise anyone looking at git or svn log, or glancing over diff, may wonder why is it a valid change. I think something like thiss will be fine:

"WEBKIT_WRAP_SOURCELIST macro is used only to adjust source groups in Visual Studio project without any impact on build process. Its references variable specific to particular target (WebCore) which contradicts our goal of having target-oriented CMake project. It can be reintroduced later in a more clean way, in case anyone needs to have such grouping"
Comment 3 Don Olmstead 2019-04-19 14:54:57 PDT
Created attachment 367834 [details]
Patch

Adding Changelog
Comment 4 EWS Watchlist 2019-04-19 14:56:37 PDT
Attachment 367834 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 8 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Don Olmstead 2019-04-19 14:58:31 PDT
Created attachment 367837 [details]
Patch
Comment 6 Konstantin Tokarev 2019-04-19 17:08:54 PDT
Comment on attachment 367837 [details]
Patch

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

> ChangeLog:12
> +        CMake project. It can be reintroduced later in a more clean way, in case anyone needs
> +        to have such grouping.

This comment is present only in one changelog file. If you are going to apply it via cq, I have no idea if it will use it for commit message or not (the former is desired)
Comment 7 Don Olmstead 2020-12-07 11:26:38 PST
Created attachment 415567 [details]
Patch
Comment 8 EWS 2020-12-07 12:28:15 PST
Committed r270516: <https://trac.webkit.org/changeset/270516>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 415567 [details].