Add ccache support to KStars
ClosedPublic

Authored by ckertesz on May 27 2017, 2:44 PM.

Details

Reviewers
mutlaqja
Summary
  1. Install ccache and compile KStars with:

cmake . -DCCACHE=ON
make -j8

  1. Clean the files with "make clean"
  2. Compile again and notice that the object files are taken from the ccache cache instead of recompilation.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
ckertesz created this revision.May 27 2017, 2:44 PM
mutlaqja edited edge metadata.May 28 2017, 12:36 PM

Should this be restricted to UNIXes (Linux & OSX)?

ckertesz updated this revision to Diff 14945.May 29 2017, 4:09 PM

I made it conditional for Unix, Mac and Android.

ckertesz updated this revision to Diff 14946.May 29 2017, 4:13 PM

Oops. I noticed that I already added ccache support in my previous Android patches. Now I changed the existing version how it should be.

And the test case in the general description is wrong, it should be:

cmake . -DCCACHE_SUPPORT=ON
make -j8

mutlaqja accepted this revision.Jun 1 2017, 5:33 AM
This revision is now accepted and ready to land.Jun 1 2017, 5:33 AM
rkflx closed this revision.Sep 27 2017, 12:03 AM
rkflx added a subscriber: rkflx.

Committed in d0bd857d9dfb.


@ckertesz There are many more accepted Diffs for KStars which are still open. Speaking about KDE's Phabricator in general (not sure whether KStars has a different policy?), Diffs should be closed (automatically when using arc land) once accepted and committed.

It's not really urgent, but it would be great if those could be checked whether they are committed indeed, and set to closed eventually. Extra points for including the commit sha. Thanks for making KStars great :)

rkflx added a comment.Oct 26 2017, 8:08 PM
In D5988#149368, @rkflx wrote:

Any progress in closing those? Currently they are clogging Phabricator's list of accepted but uncommitted issues not only for KStars but everybody, while yours are probably already committed…

I already closed all dangling PRs months ago where I was the author.

rkflx added a comment.Oct 26 2017, 8:19 PM

I don't think you understand? Please look at the list I linked. You are the author, the state is "Accepted". The final state in Phabricator should be "Closed" once the commit has landed in the git repo. Are you saying those commits still have not landed?

Ok, I understand, I have to also press Submit otherwise it is just Preview state.

Done now from my side.

rkflx added a comment.Oct 26 2017, 8:30 PM

Almost :) Some are still open.

Ok. Now really.