[libical] Default value for ical_errors_are_fatal

[Suman Manjunath]

Hello,

[This is my first post to this list.. apologies if I've missed
following some etiquette :)]

As a part of merging the GNOME libical fork changes upstream[1], I
have bumped into an issue with the default setting of
ical_errors_are_fatal.

As of now, it is set to 1 (making libical crash on every call to
icalerror_*), due to which:

a) the Evolution guys have to set this variable to 0 in all related
processes at runtime - Evolution, Evolution-Data-Server(EDS) and
Evolution-Exchange
b) any other application depending on the EDS libraries (libecal and
libedata-cal) have to set it again to 0 as EDS itself might not be
running

This scales to any other application that does not want to go kaboom
on calls to icalerror_*. ALL of them need to do the same thing - set
ical_errors_are_fatal = 0

Now, as mentioned in the configure.in of libical, the setting is for
"test/regression.c (and maybe others) needs this defined". If it is
required by test and regression suites, why force it on applications
that depend on libical?

It is quite simple to make this setting configurable (see attached
patch). With the patch, ical_errors_are_fatal is set to 0 ONLY IF
explicitly specified using the configure option. This makes life
easier for the distros shipping libical (they can easily turn this
off) and does not hurt the libical developers either (fatality enabled
by default).

I propose this patch be taken upstream. Thoughts?

-Suman

[1] http://bugzilla.gnome.org/show_bug.cgi?id=541209
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icalerror-configurable.diff
Type: text/x-patch
Size: 1661 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/libical-interest/attachments/20081215/a8685710/attachment.bin>