Update to libical 0.46

RESOLVED FIXED in 1.0b4

Status

Calendar
Internal Components
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

Dependency tree / graph

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Assignee)

Description

7 years ago
Created attachment 515391 [details] [diff] [review]
update to reference libical, no local changes

We should update to the latest libical, there are a few bugs (especially w.r.t the ATTACH property) that are fixed there.
Attachment #515391 - Flags: review?(nomisvai)
(Assignee)

Comment 1

7 years ago
Created attachment 515392 [details] [diff] [review]
local changes as before patch

This patch re-applies all the local changes we had before. There is one exception, in design-data/parameters.csv:

-"RECEIVED-SEQUENCE","const char*",
-"RECEIVED-DTSTAMP","const char*",
-"SCHEDULE-AGENT","const char*",
-"SCHEDULE-STATUS","const char*",
-"SCHEDULE-FORCE-SEND","const char*",

These cases should be taken care of by libical's addtion here:

+"#In practice any unknown paramater that is not an xparam is treated as an ianaparam"
+"IANA","33","const char*",
+"ANY","0",,

I'd appreciate a test here.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #515392 - Flags: review?(nomisvai)
(Assignee)

Updated

7 years ago
Blocks: 632352
(Assignee)

Comment 2

7 years ago
For reference, the last upgrade happened in bug 394902.

Comment 3

7 years ago
Makes me think if we should follow other examples in mozilla-central regarding handling of external libraries. For easier track of changes and probably easier update with future library releases the patches applied to the vanilla library version are stored together with the source code and documented in a readme, e.g.
http://mxr.mozilla.org/comm-central/source/mozilla/media/libogg/
http://mxr.mozilla.org/comm-central/source/mozilla/media/libsydneyaudio/
(Assignee)

Comment 4

7 years ago
While this will introduce a lot of junk files that we don't need (which in turn causes comm-central checkout times to increase for everyone), it might be worth considering. I think we should postpone this to "next time" though.

I remember dbo saying that applying patches on libraries has caused maintenance issues with openoffice, I'll ask him about this again when I find time.

Comment 5

7 years ago
Created attachment 516575 [details] [diff] [review]
[maybe beta]Compilation tests and modifications

I tried to compile after applying the patches but I got errors.
First error comes with missing 'icaltimezoneimpl.h' file.
After adding the file icaltimezoneimpl.h (from libical 0.46), errors are obtained when linking.
'unresolved external symbol _snprintf'
After removing 'libicals_w32_vsnprintf_replacement.c' file and making some changes, compilation works.
I do not know if this is correct, but the compilation succeeded.
I started testing Lightning with this compilation and initial tests go well.
Attachment #516575 - Flags: review?(philipp)
(Assignee)

Comment 6

7 years ago
Hm too bad the snprintf stuff doesnt work out of the box. We used to have a similar local change to fix snprintf in the past, I had hoped this was fixed in the newest libical. I'll integrate your changes into the patch.
(Assignee)

Updated

7 years ago
Blocks: 557782

Comment 7

7 years ago
Will this fix bug #322458?
(Assignee)

Comment 8

7 years ago
Comment on attachment 515392 [details] [diff] [review]
local changes as before patch

Philippe, maybe you can take a look at this to make sure I'm doing the right thing?
Attachment #515392 - Flags: review?(nomisvai) → review?(philippe.martinak)
(Assignee)

Updated

7 years ago
Attachment #515391 - Flags: review?(nomisvai) → review?(philippe.martinak)

Comment 9

7 years ago
Comment on attachment 515392 [details] [diff] [review]
local changes as before patch

Attachment 515392 [details] [diff] from 29/03 and 26/02 are the same.
Applying patches 515391 and 515392 does not allow me to compile without error.
(Assignee)

Updated

7 years ago
Attachment #516575 - Attachment description: Compilation tests and modifications → [maybe beta]Compilation tests and modifications
(Assignee)

Comment 10

7 years ago
Created attachment 534770 [details] [diff] [review]
local changes as before patch - v2

This patch combines attachment 515392 [details] [diff] [review] and attachment 516575 [details] [diff] [review]. I'm marking it r+, I think we can count replying with a patch as a review.
Attachment #515392 - Attachment is obsolete: true
Attachment #516575 - Attachment is obsolete: true
Attachment #515392 - Flags: review?(philippe.martinak)
Attachment #516575 - Flags: review?(philipp)
Attachment #534770 - Flags: review+
(Assignee)

Comment 11

7 years ago
Comment on attachment 515391 [details] [diff] [review]
update to reference libical, no local changes

r+ similar to previous comment.
Attachment #515391 - Flags: review?(philippe.martinak) → review+
(Assignee)

Comment 12

7 years ago
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/c68b4de81618>
-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Trunk
(Assignee)

Comment 13

7 years ago
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/8b7cdb6dd357>
-> FIXED
(Assignee)

Comment 14

7 years ago
Backported to comm-miramar <http://hg.mozilla.org/releases/comm-miramar/rev/69c9da461448>
Target Milestone: Trunk → 1.0b4
You need to log in before you can comment on or make changes to this bug.