Differential D4144

Install also CamelCase forward headers for classes from multi-class headers
ClosedPublic
Actions

Authored by kossebau on Jan 15 2017, 5:21 PM.

Details

Reviewers
cfeck
Group Reviewers
Frameworks
Commits
R236:1473382cbd22: Install also CamelCase forward headers for classes from multi-class headers

Diff Detail

Repository
R236 KWidgetsAddons
Branch
installMoreCamelCaseForwardHeaders
Lint
No Linters Available
Unit
No Unit Test Coverage
kossebau updated this revision to Diff 10188.Jan 15 2017, 5:21 PM
kossebau retitled this revision from to Install also CamelCase forward headers for classes from multi-class headers.
kossebau added a reviewer: Frameworks.
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 15 2017, 5:21 PM
cfeck accepted this revision.Jan 17 2017, 1:39 AM
cfeck added a reviewer: cfeck.
cfeck added a subscriber: cfeck.

I have no idea if a comma is a valid separator here, but I am sure you tested it :)

This revision is now accepted and ready to land.Jan 17 2017, 1:39 AM
kossebau added a comment.Jan 17 2017, 1:56 AM

Yes, both tested and the docs of the macro also have it explicitely: https://api.kde.org/ecm/module/ECMGenerateHeaders.html :) Thanks for review.

Closed by commit R236:1473382cbd22: Install also CamelCase forward headers for classes from multi-class headers (authored by kossebau). · Explain WhyJan 19 2017, 6:08 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/CMakeLists.txt (8 lines)

Diff 10188

View Options

src/CMakeLists.txt

Loading...