KCalCore: remove local versit copy and use libical's icalvcal library instead
ClosedPublic

Authored by winterz on Feb 24 2018, 4:26 PM.

Details

Summary

This patch removes the ancient local copy of versit and replaces it with the same calls to the libical's icalvcal library.

Benefit: we let the libical people properly maintain.

Thanks to krop for adapting FindLibical.cmake

Test Plan

builds and make test still works

should continue work for people using ancient libical 1.0, but I would like to increase the min libical version to 2.0.

Diff Detail

Repository
R172 KCalendar Core
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
winterz created this revision.Feb 24 2018, 4:26 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptFeb 24 2018, 4:26 PM
Restricted Application added a subscriber: KDE PIM. · View Herald Transcript
winterz requested review of this revision.Feb 24 2018, 4:26 PM

"but I would like to increase the min libical version to 2.0." +1 I think that we can increase it.

dvratil requested changes to this revision.Feb 24 2018, 5:29 PM
dvratil added a subscriber: dvratil.

Should the entire versit directory be removed as well?

src/vcalformat.h
49

Could we keep the includes in the .cpp file?

This revision now requires changes to proceed.Feb 24 2018, 5:29 PM

Should the entire versit directory be removed as well?

yes, it is removed locally. I'll make a new patch to show that

src/vcalformat.h
49

yes. right. I'll do that.

winterz updated this revision to Diff 27939.Feb 24 2018, 6:13 PM

versit subdir is removed
relocate the includes from vcalformat.h to vcalformat.cpp

dvratil accepted this revision.Feb 24 2018, 7:53 PM

lgtm, thanks.

This revision is now accepted and ready to land.Feb 24 2018, 7:53 PM
This revision was automatically updated to reflect the committed changes.