Require out-of-source builds
ClosedPublic

Authored by ngraham on Sat, May 16, 7:47 PM.

Details

Summary

In addition to just being a good idea in general, the dynamic icon
generation bit doesn't work properly with in-source builds. Let's
formally require out-of-source builds.

BUG: 421637
FIXED-IN: 5.71

Test Plan

Try out of source build, get yelled at

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Sat, May 16, 7:47 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptSat, May 16, 7:47 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
ngraham requested review of this revision.Sat, May 16, 7:47 PM
ognarb accepted this revision.Sat, May 16, 7:50 PM
This revision is now accepted and ready to land.Sat, May 16, 7:50 PM
davidre requested changes to this revision.Sat, May 16, 7:55 PM
davidre added a subscriber: davidre.

I don't think we want to require in source builds

This revision now requires changes to proceed.Sat, May 16, 7:55 PM

I believe a few sentences in the the commit message contradicts each other.

ngraham retitled this revision from Require in-source build to Require out-of-source builds.Sat, May 16, 8:09 PM
davidre accepted this revision.Sat, May 16, 8:18 PM
This revision is now accepted and ready to land.Sat, May 16, 8:18 PM
apol accepted this revision.Sun, May 17, 12:31 PM
nicolasfella accepted this revision.Sun, May 17, 1:09 PM
ndavis accepted this revision.Sun, May 17, 2:00 PM
tfella accepted this revision.Sun, May 17, 2:05 PM
filipf accepted this revision.Sun, May 17, 2:17 PM
cblack accepted this revision.Sun, May 17, 4:45 PM

I think this patch meets the world record for number of acceptances!

This revision was automatically updated to reflect the committed changes.