Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-16 Thread Michael Schwendt
On Tue, 14 Nov 2017 08:27:28 +0100, Milan Crha wrote:

>   Hi,
> the sum of changes is part of the release notes, which can be found
> here:
> https://github.com/libical/libical/releases

Would be great, if such a document were included within the package.

> > vcal_manager.c: In function 'vcal_manager_event_dump':
> > vcal_manager.c:398:30: warning: implicit declaration of function
> > 'icaltime_from_timet'; did you mean 'icaltime_as_timet'? [-Wimplicit-
> > function-declaration]
> >   icalproperty_vanew_dtstamp(icaltime_from_timet(time(NULL), TRUE),
> > (void*)0));  
> 
> In your case replace
>icaltime_from_timet()
> with
>icaltime_from_timet_with_zone()
> where the last argument, the zone to use, will be NULL.

Oh! So they only removed a method they could have kept as a wrapper call
to the new method.

Interestingly, in the headers in libical-devel.x86_64 2.0.0-12.fc27 both
functions exist already and no deprecation warning in there.

Thanks.
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Milan Crha
On Mon, 2017-11-13 at 10:01 -0800, Kevin Fenzi wrote:
> Orage (the Xfce calendar app) is also broken if you have time to take a
> look:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1512302
> https://bugzilla.xfce.org/show_bug.cgi?id=13997

Hi,
sure, I'll reply on the Red Hat bug.
Bye,
Milan
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Milan Crha
On Tue, 2017-11-14 at 01:36 +0100, Michael Schwendt wrote:
> Dunno yet what's the reason. Haven't had a look at it except for unpacking
> libical-devel in hope that its %doc section would contain a file that sums
> up the API changes. No such file in there. And the "Using Libical" guide
> claims it's from 2001, but comes with a fresh timestamp and doesn't tell
> about the changes either.

Hi,
the sum of changes is part of the release notes, which can be found
here:
https://github.com/libical/libical/releases

> vcal_manager.c: In function 'vcal_manager_event_dump':
> vcal_manager.c:398:30: warning: implicit declaration of function
> 'icaltime_from_timet'; did you mean 'icaltime_as_timet'? [-Wimplicit-
> function-declaration]
>   icalproperty_vanew_dtstamp(icaltime_from_timet(time(NULL), TRUE),
> (void*)0));

In your case replace
   icaltime_from_timet()
with
   icaltime_from_timet_with_zone()
where the last argument, the zone to use, will be NULL.

Bye,
Milan
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Michael Schwendt
On Mon, 13 Nov 2017 15:04:55 +0100, Milan Crha wrote:

> > The upgrade seems to be API incompatible, so more than rebuilding
> > dependencies is necessary.  
> 
>   Hi,
> that's true. libical upstream removed some deprecated symbols, most
> notably icaltimetype::is_utc.

Dunno yet what's the reason. Haven't had a look at it except for unpacking
libical-devel in hope that its %doc section would contain a file that sums
up the API changes. No such file in there. And the "Using Libical" guide
claims it's from 2001, but comes with a fresh timestamp and doesn't tell
about the changes either.


libtool: compile:  gcc -DHAVE_CONFIG_H -I. -I../../.. -I../../../src 
-I../../../src/common -I../../../src/common -I../../../src/gtk 
-I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -pthread 
-I/usr/include/gtk-2.0 -I/usr/lib64/gtk-2.0/include -I/usr/include/pango-1.0 
-I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/cairo 
-I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 
-I/usr/include/harfbuzz -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/atk-1.0 
-pthread -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 
-fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 
-grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 
-mtune=generic -Wno-unused-function -Wno-pointer-sign -Wall 
-I/usr/include/enchant -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include 
-pthread -c plugin.c -o vcalendar_la-plugin.o >/dev/null 2>&1
vcal_manager.c: In function 'vcal_manager_event_dump':
vcal_manager.c:398:30: warning: implicit declaration of function 
'icaltime_from_timet'; did you mean 'icaltime_as_timet'? 
[-Wimplicit-function-declaration]
   icalproperty_vanew_dtstamp(icaltime_from_timet(time(NULL), TRUE), (void*)0));
  ^~~
  icaltime_as_timet
vcal_manager.c:398:30: error: incompatible type for argument 1 of 
'icalproperty_vanew_dtstamp'
In file included from vcal_manager.c:35:0:
/usr/include/libical/ical.h:3306:35: note: expected 'struct icaltimetype' but 
argument is of type 'int'
 LIBICAL_ICAL_EXPORT icalproperty *icalproperty_vanew_dtstamp(struct 
icaltimetype v, ...);
   ^~
vcal_manager.c:426:30: error: incompatible type for argument 1 of 
'icalproperty_vanew_created'
   icalproperty_vanew_created(icaltime_from_timet(time(NULL), TRUE), (void*)0));
  ^~~
In file included from vcal_manager.c:35:0:
/usr/include/libical/ical.h:3246:35: note: expected 'struct icaltimetype' but 
argument is of type 'int'
 LIBICAL_ICAL_EXPORT icalproperty *icalproperty_vanew_created(struct 
icaltimetype v, ...);
   ^~
vcal_manager.c:428:35: error: incompatible type for argument 1 of 
'icalproperty_vanew_lastmodified'
   icalproperty_vanew_lastmodified(icaltime_from_timet(time(NULL), TRUE), 
(void*)0));
   ^~~
In file included from vcal_manager.c:35:0:
/usr/include/libical/ical.h:3371:35: note: expected 'struct icaltimetype' but 
argument is of type 'int'
 LIBICAL_ICAL_EXPORT icalproperty *icalproperty_vanew_lastmodified(struct 
icaltimetype v, ...);
   ^~~
vcalendar.c: In function 'create_meeting_from_message_cb_ui':
vcalendar.c:160:54: warning: implicit declaration of function 
'icaltime_from_timet'; did you mean 'icaltime_as_timet'? 
[-Wimplicit-function-declaration]
gchar *dtstart = g_strdup(icaltime_as_ical_string(icaltime_from_timet(t, 
FALSE)));
  ^~~
  icaltime_as_timet
vcalendar.c:160:54: error: incompatible type for argument 1 of 
'icaltime_as_ical_string'
In file included from vcalendar.c:30:0:
/usr/include/libical/ical.h:171:33: note: expected 'const struct icaltimetype' 
but argument is of type 'int'
 LIBICAL_ICAL_EXPORT const char *icaltime_as_ical_string(const struct 
icaltimetype tt);
 ^~~
vcalendar.c:161:52: error: incompatible type for argument 1 of 
'icaltime_as_ical_string'
gchar *dtend = g_strdup(icaltime_as_ical_string(icaltime_from_timet(t2, 
FALSE)));
^~~
In file included from vcalendar.c:30:0:
/usr/include/libical/ical.h:171:33: note: expected 'const struct icaltimetype' 
but argument is of type 'int'
 LIBICAL_ICAL_EXPORT const char *icaltime_as_ical_string(const struct 
icaltimetype tt);
 ^~~
make[5]: *** [Makefile:666: vcalendar_la-vcal_manager.lo] Error 1
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedor

Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Kevin Fenzi
On 11/13/2017 06:04 AM, Milan Crha wrote:
> On Mon, 2017-11-13 at 14:47 +0100, Michael Schwendt wrote:
>> The upgrade seems to be API incompatible, so more than rebuilding
>> dependencies is necessary.
> 
>   Hi,
> that's true. libical upstream removed some deprecated symbols, most
> notably icaltimetype::is_utc. It's replaced with icaltime_is_utc()
> function (available for a long time) and instead of writing to
> icaltimetype::is_utc one should set the timezone properly, thus when it
> had been set to 1 the icaltimetype::zone should be
> icaltimezone_get_utc_timezone(), when set to 0, then to something else.
> 
>> Claws Mail doesn't rebuild.
> 
> I do not see that failed build in koji, do you have a build log from
> there, please? I can help with patches, as I already did with
> kdepimlibs (a bug filled, patch attached). I do not have commit rights
> for all those still failing packages, thus my help ability is slightly
> limited.

Orage (the Xfce calendar app) is also broken if you have time to take a
look:

https://bugzilla.redhat.com/show_bug.cgi?id=1512302
https://bugzilla.xfce.org/show_bug.cgi?id=13997

kevin



signature.asc
Description: OpenPGP digital signature
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Milan Crha
On Mon, 2017-11-13 at 14:47 +0100, Michael Schwendt wrote:
> The upgrade seems to be API incompatible, so more than rebuilding
> dependencies is necessary.

Hi,
that's true. libical upstream removed some deprecated symbols, most
notably icaltimetype::is_utc. It's replaced with icaltime_is_utc()
function (available for a long time) and instead of writing to
icaltimetype::is_utc one should set the timezone properly, thus when it
had been set to 1 the icaltimetype::zone should be
icaltimezone_get_utc_timezone(), when set to 0, then to something else.

> Claws Mail doesn't rebuild.

I do not see that failed build in koji, do you have a build log from
there, please? I can help with patches, as I already did with
kdepimlibs (a bug filled, patch attached). I do not have commit rights
for all those still failing packages, thus my help ability is slightly
limited.
Bye,
Milan
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


Re: [Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-13 Thread Michael Schwendt
On Wed, 01 Nov 2017 12:26:12 +0100, Milan Crha wrote:

>   Hello,
> I'd like to give a heads up about a plan to update libical to its 3.0.0
> release in rawhide once I figure out some details about its build. This
> release also obsoletes libical-glib package, the project had been added
> into libical itself.
> 
> I currently plan to push the update on Monday, November 6th.

The upgrade seems to be API incompatible, so more than rebuilding
dependencies is necessary.

Claws Mail doesn't rebuild. And since texlive is waiting for a rebuild due
to poppler SONAME breakage, temporarily one would need to set %build_manual
to 0 in claws-mail.spec file.
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org


[Heads-up] libical 3.0.0 to reach rawhide, replacing libical-glib

2017-11-01 Thread Milan Crha
Hello,
I'd like to give a heads up about a plan to update libical to its 3.0.0
release in rawhide once I figure out some details about its build. This
release also obsoletes libical-glib package, the project had been added
into libical itself.

I currently plan to push the update on Monday, November 6th.

The list of dependencies according to:
   # dnf repoquery --whatrequires libical-devel --alldeps
follows:
   evolution-data-server-devel
   libical-glib-devel
   kf5-kcalendarcore-devel
   maya-calendar-devel
I have commit rights only for the first two, where the second is to be
obsoleted anyway.

The list of packages using libical is much longer:

   # dnf repoquery --whatrequires libical --alldeps
   akonadi-calendar-tools
   akonadiconsole
   almanah
   asterisk-calendar
   bijiben
   blogilo
   bluez-obexd
   cairo-dock-plug-ins-base
   california
   calligra-plan-libs
   claws-mail-plugins-vcalendar
   cyrus-imapd
   cyrus-imapd-utils
   digikam
   digikam-libs
   evolution
   evolution-bogofilter
   evolution-data-server
   evolution-data-server-tests
   evolution-ews
   evolution-mapi
   evolution-pst
   evolution-spamassassin
   gnokii
   gnome-calendar
   gnome-shell
   gnome-todo
   kalarm
   kdepim-addons
   kdepim-runtime
   kdepim-runtime-libs
   kdepimlibs
   kf5-akonadi-calendar
   kf5-akonadi-search
   kf5-calendarsupport
   kf5-eventviews
   kf5-incidenceeditor
   kf5-kalarmcal
   kf5-kblog
   kf5-kcalendarcore
   kf5-kcalendarutils
   kf5-ktnef
   kf5-messagelib
   kmail
   kmail-libs
   kmymoney
   knotes-libs
   korganizer
   korganizer-libs
   libical-glib
   libkgapi
   libkolab
   libopensync-plugin-evolution2
   maya-calendar
   openchange-client
   orage
   osmo
   python2-kolab
   renku
   syncevolution
   syncevolution-libs
   syncevolution-libs-akonadi
   wingpanel-indicator-datetime
   zanshin

I know they get the dependency mostly due to using one or more of
the packages from the first list, but I guess it will be safer to
rebuild them too. I have commit rights to many of those, but not to
all, thus I'd need help with rebuilds from the respective maintainers,
co-maintainers or proven packager(s), once I push the libical 3.0.0 to
the rawhide.

Bye,
Milan
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org