Bug 3764 - Use external libical instead of the internal copy
Summary: Use external libical instead of the internal copy
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: Plugins/vCalendar (show other bugs)
Version: 3.15.0
Hardware: PC Linux
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2017-01-21 14:19 UTC by flo.xfce
Modified: 2017-01-27 16:59 UTC (History)
0 users

See Also:


Attachments
the patch (109.79 KB, application/gzip)
2017-01-21 18:20 UTC, Paul
no flags Details
Original patch sans removal of the builtin libical (15.07 KB, patch)
2017-01-21 20:22 UTC, Andrej Kacian
no flags Details | Diff

Description flo.xfce 2017-01-21 14:19:11 UTC
The vcalendar plugin relies on libical, which is provided in-tree. However, this included version is outdated. I am not aware of any security related issues with libical at the moment, but using the upstream version of libical can prevent some security issues should they arise (users only have to update libical, not claws-mail).
The attached patch removes the in-tree version of libical and fixes the build to use an external version of libical. This patch also bumps the minimum dependency of libical to version 2.0 and a few code lines were adjusted accordingly.
The patch is only tested on Linux. I do not know if Windows or OS X build were broken.
Comment 1 flo.xfce 2017-01-21 14:30:14 UTC
So the patch is a bit too large to be attached here directly, so please see this paste:
https://bpaste.net/raw/9a4c01fcd5ba
Comment 2 Paul 2017-01-21 18:20:18 UTC
Created attachment 1713 [details]
the patch

the patch linked in comment 1, gzipped and attached
Comment 3 Andrej Kacian 2017-01-21 20:22:38 UTC
Created attachment 1714 [details]
Original patch sans removal of the builtin libical

For convenience. :)

This (removal of our old, decrepit libical) is definitely something we want. Windows build can be tested, as for Mac version, that is out of our hands - its maintainer will have to take care of testing.
Comment 4 users 2017-01-27 16:57:03 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

++ ChangeLog	2017-01-27 16:57:03.545437245 +0100
http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=d76642c6c7cb6dfb8c1d94308d725bcb6a9d18ae
Merge: ff4298a 8925ab2
Author: Colin Leroy <colin@colino.net>
Date:   Fri Jan 27 16:57:02 2017 +0100

    Merge branch 'master' of file:///home/git/claws

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=8925ab226ba8bb18928d61393d45e7d97f32777b
Author: Paul <paul@claws-mail.org>
Date:   Fri Jan 27 15:56:47 2017 +0000

    vCalendar plugin: use external libical
    
    closes bug #3764
    patch by flo.xfce@gmx-topmail.de
Comment 5 Paul 2017-01-27 16:59:40 UTC
Thanks!

Note You need to log in before you can comment on or make changes to this bug.