Bug#905697: kdepimlibs: don't depend on libical
What about porting it to libical3? That might be easier. I tried to build it
and it does need some changes, but I don't think keeping libical2 is feasible.
See the attached for the error.
Scott K
On Friday, February 08, 2019 10:46:21 PM Sandro Knauß wrote:
> Hey,
>
> for me it looks like we won't be able to get kdepimlibs without libical 2
> for buster.
> Keep in mind, that kdepimlibs is an old grufted lib set that we also would
> like to kill. But we also have other packages depending on it like:
> * basket
> * kopete (it has a QT5 version in experimental, but this is suitable as
> replacement yet).
> and other that may can be killed.
>
> extract libical 2 out of kdepimlibs is not that simple, as the whole
> interface will leak into akonadi-calendar, calcore, calutils,...
>
> In short I won't have the time to do the work and test it in the near
> future.
>
> I added pino to this bug report, as pino successfully extracted other parts
> of old kdelibs/kdepimlibs.
>
> hefee
>
> --
>
> On Donnerstag, 17. Januar 2019 12:11:46 CET Emilio Pozuelo Monfort wrote:
> > On 11/01/2019 13:37, Emilio Pozuelo Monfort wrote:
> > > On 08/08/2018 10:38, Emilio Pozuelo Monfort wrote:
> > >> Source: kdepimlibs
> > >> Version: 4:4.14.10-10
> > >> Severity: serious
> > >> Control: block 884128 with -1
> > >>
> > >> Hi,
> > >>
> > >> libical2 from src:libical is superseded by libical3 (src:libical3).
> > >>
> > >> Please either port kdepimlibs to libical3 or try to disable the
> > >> libical support, so that we can only ship src:libical3 in buster.
> > >
> > > Could someone who knows kdepimlibs take a look at this? Can we disable
> > > libical support in kdepimlibs for buster? This is the last blocker for
> > > the libical 2 removal.
> >
> > Hi Sandro,
> >
> > Lisandro said you may be able to help with this. Do you know if libical
> > support could be disabled in kdepimlibs, or if the newer version libical3
> > could be used?
> >
> > >From [1] it seems that libical is kind of optional, though I don't know
> > >if
> > >we
> >
> > can disable the bits to drop that or if those bits have rdeps. Since
> > kdepimlibs is the last rdep of the old libical version, fixing this would
> > allow us to drop that one from buster. Otherwise the RC bugs that it has
> > will need to be fixed and we'll have to ship with two versions of the
> > library.
> >
> > Thanks,
> > Emilio
> >
> > [1]
> > https://sources.debian.org/src/kdepimlibs/4:4.14.10-10/CMakeLists.txt/#L81
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp: In static member function 'static icaltimetype KCalCore::ICalFormatImpl::writeICalDate(const QDate&)':
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp:2304:7: error: 'icaltimetype' {aka 'struct icaltimetype'} has no member named 'is_utc'; did you mean 'is_date'?
t.is_utc = 0;
^~~~~~
is_date
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp: In static member function 'static icaltimetype KCalCore::ICalFormatImpl::writeICalDateTime(const KDateTime&)':
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp:2326:7: error: 'icaltimetype' {aka 'struct icaltimetype'} has no member named 'is_utc'; did you mean 'is_date'?
t.is_utc = datetime.isUtc() ? 1 : 0;
^~~~~~
is_date
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp: In static member function 'static icalproperty* KCalCore::ICalFormatImpl::writeICalDateTimeProperty(icalproperty_kind, const KDateTime&, KCalCore::ICalTimeZones*, KCalCore::ICalTimeZones*)':
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp:2401:12: error: 'icaltimetype' {aka 'struct icaltimetype'} has no member named 'is_utc'; did you mean 'is_date'?
if (!t.is_utc) {
^~~~~~
is_date
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp: In static member function 'static KDateTime KCalCore::ICalFormatImpl::readICalDateTime(icalproperty*, const icaltimetype&, KCalCore::ICalTimeZones*, bool)':
/home/kdepimlibs-4.14.10/kcalcore/icalformat_p.cpp:2434:11: error: 'const icaltimetype' {aka 'const struct icaltimetype'} has no member named 'is_utc'; did you mean 'is_date'?
if (t.is_utc || t.zone == icaltimezone_get_utc_timezone()) {
^~~~~~
is_date
Reply to: