Skip to contentSkip to conversation

libical: upgrade to 3.0.5

PATCH Superseded Ross Burton 3 participants 9 messages Jul 22, 2019 – Jul 23, 2019

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 libical: upgrade to 3.0.5

Note that this upgrade includes some API breakage.

---
 ...ibical.pc.in-fix-iculibs-remove-full.patch | 44 -------------------
 .../{libical_2.0.0.bb => libical_3.0.5.bb}    | 20 +++++----
 2 files changed, 12 insertions(+), 52 deletions(-)
 delete mode 100644 meta/recipes-support/libical/libical/0001-CMakeLists.txt-libical.pc.in-fix-iculibs-remove-full.patch
 rename meta/recipes-support/libical/{libical_2.0.0.bb => libical_3.0.5.bb} (53%)
meta/recipes-support/libical/libical/0001-CMakeLists.txt-libical.pc.in-fix-iculibs-remove-full.patch deleted +0 -44
deleted file mode 100644
index 6db75f5086c..00000000000
@@ -1,44 +0,0 @@
-From 1a41069c0dc054e59cd76fc3d8cf7cc2a78b5e8d Mon Sep 17 00:00:00 2001
-From: Allen Winter <[email protected]>
-Date: Sat, 3 Sep 2016 16:56:29 -0400
-Subject: [PATCH] CMakeLists.txt, libical.pc.in - fix iculibs (remove full path) ISSUE: 227
-
-Upstream-Status: Backport
-
-Signed-off-by: Maxin B. John <[email protected]>
-----
-diff -Naur libical-2.0.0-orig/CMakeLists.txt libical-2.0.0/CMakeLists.txt
---- libical-2.0.0-orig/CMakeLists.txt	2016-09-08 18:05:06.166049345 +0300
-+++ libical-2.0.0/CMakeLists.txt	2016-09-08 18:05:45.201561334 +0300
-@@ -128,6 +128,7 @@
- #  RSCALE info at http://tools.ietf.org/html/rfc7529
- find_package(ICU)
- if(ICU_FOUND)
-+  set(ICUUC_LIBS "-licuuc") #for libical.pc
-   set(HAVE_LIBICU 1)
-   if(ICU_MAJOR_VERSION VERSION_GREATER 50)
-     set(HAVE_ICU_DANGI TRUE)
-@@ -137,6 +138,7 @@
- endif()
- if(ICU_I18N_FOUND)
-   set(HAVE_LIBICU_I18N 1)
-+  set(ICUI18N_LIBS "-licui18n") #for libical.pc
- endif()
- 
- # MSVC specific definitions
-diff -Naur libical-2.0.0-orig/libical.pc.in libical-2.0.0/libical.pc.in
---- libical-2.0.0-orig/libical.pc.in	2015-12-28 23:44:53.000000000 +0200
-+++ libical-2.0.0/libical.pc.in	2016-09-08 18:09:12.991963597 +0300
-@@ -3,10 +3,10 @@
- libdir=@libdir@
- includedir=@includedir@
- threadslib=@PTHREAD_LIBS@
--iculib=@ICU_LIBRARIES@ @ICU_I18N_LIBRARIES@
-+iculibs=@ICUUC_LIBS@ @ICUI18N_LIBS@
- 
- Name: libical
- Description: An implementation of basic iCAL protocols
- Version: @VERSION@
--Libs: -L${libdir} -lical -licalss -licalvcal ${threadslib} ${iculib}
-+Libs: -L${libdir} -lical -licalss -licalvcal ${threadslib} ${iculibs}
- Cflags: -I${includedir}
meta/recipes-support/libical/libical_3.0.5.bb renamed from meta/recipes-support/libical/libical_2.0.0.bb +12 -8
similarity index 53%
rename from meta/recipes-support/libical/libical_2.0.0.bb
rename to meta/recipes-support/libical/libical_3.0.5.bb
index 7dffdcfb013..6489ccb9323 100644
@@ -1,18 +1,18 @@
 SUMMARY = "iCal and scheduling (RFC 2445, 2446, 2447) library"
 HOMEPAGE = "https://github.com/libical/libical"
 BUGTRACKER = "https://github.com/libical/libical/issues"
-LICENSE = "LGPLv2.1 | MPL-1.0"
-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
-                    file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+                    file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+                    file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"
 SECTION = "libs"
 
-SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz \
-           file://0001-CMakeLists.txt-libical.pc.in-fix-iculibs-remove-full.patch \
-           "
+SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz"
+
 DEPENDS = "icu"
 
-SRC_URI[md5sum] = "6bf8e5f5a3ba88baf390d0134e05d76e"
-SRC_URI[sha256sum] = "654c11f759c19237be39f6ad401d917e5a05f36f1736385ed958e60cf21456da"
+SRC_URI[md5sum] = "9be4e8a1739a9c27e06aa8ce66b8cb34"
+SRC_URI[sha256sum] = "7ad550c8c49c9b9983658e3ab3e68b1eee2439ec17b169a6b1e6ecb5274e78e6"
 UPSTREAM_CHECK_URI = "https://github.com/libical/libical/releases"
 
 inherit cmake pkgconfig
@@ -20,6 +20,10 @@ inherit cmake pkgconfig
 # No need to use perl-native, the host perl is sufficient.
 EXTRA_OECMAKE += "-DPERL_EXECUTABLE=${HOSTTOOLS_DIR}/perl"
 
+# The glib library can't be cross-compiled, disable for now.
+# https://github.com/libical/libical/issues/394
+EXTRA_OECMAKE += "-DICAL_GLIB=false"
+
 do_install_append_class-target () {
     # Remove build host references
     sed -i \
Signature · 1 line
2.20.1
#1 lore raw
Quoted 7 lines · Ross Burton
(quoted diff) +0 -2
-LICENSE = "LGPLv2.1 | MPL-1.0"
-LIC_FILES_CHKSUM =

"file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =
Quoted 5 lines

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

https://autobuilder.yoctoproject.org/typhoon/#/builders/37/builds/862/steps/8/logs/warnings

Thanks,
Anuj

#2 lore raw
Quoted 3 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 5 lines
Quoted 5 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6" +LICENSE = "LGPLv2.1 | MPL-2"

Quoted 8 lines
Quoted 6 lines

+LIC_FILES_CHKSUM = "file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

Quoted 12 lines

https://autobuilder.yoctoproject.org/typhoon/#/builders/37/builds/862/steps/8/logs/warnings

Thanks,
Anuj

--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

#3 lore raw
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 5 lines
Quoted 5 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6" +LICENSE = "LGPLv2.1 | MPL-2"

Quoted 8 lines
Quoted 6 lines

+LIC_FILES_CHKSUM = "file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

Thanks.

#4 lore raw
Quoted 23 lines · Anuj
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 13 lines
Quoted 11 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 4 lines
Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

reply quote confused me. yes it is wrong. It should be MPL-2.0

Quoted 2 lines

Thanks.

#5 lore raw

this is causing a failure in meta-xfce

https://errors.yoctoproject.org/Errors/Details/253910/

Quoted 34 lines · Khem Raj
Quoted 23 lines · Anuj
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 13 lines
Quoted 11 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 4 lines
Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

reply quote confused me. yes it is wrong. It should be MPL-2.0

Quoted 2 lines

Thanks.

#6 lore raw

The last release of Orage was four years ago: I suggest removing that from meta-xfce instead of keeping everyone else using an old version of libical.

Ross

Quoted 40 lines · Khem Raj

this is causing a failure in meta-xfce

https://errors.yoctoproject.org/Errors/Details/253910/

Quoted 34 lines · Khem Raj
Quoted 23 lines · Anuj
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 13 lines
Quoted 11 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 4 lines
Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

reply quote confused me. yes it is wrong. It should be MPL-2.0

Quoted 2 lines

Thanks.

#7 lore raw
Quoted 4 lines · Ross

The last release of Orage was four years ago: I suggest removing that from meta-xfce instead of keeping everyone else using an old version of libical.

thats the latest unfortunately. I think a simple fix would be to drop is_utc field setting
in orage since it seems to gone in libical3, I will try to cookup a patch and try

Quoted 44 lines

Ross

Quoted 40 lines · Khem Raj

this is causing a failure in meta-xfce

https://errors.yoctoproject.org/Errors/Details/253910/

Quoted 34 lines · Khem Raj
Quoted 23 lines · Anuj
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 13 lines
Quoted 11 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 4 lines
Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

reply quote confused me. yes it is wrong. It should be MPL-2.0

Quoted 2 lines

Thanks.

#8 lore raw
Quoted 12 lines · Khem Raj
Quoted 4 lines · Ross

The last release of Orage was four years ago: I suggest removing that from meta-xfce instead of keeping everyone else using an old version of libical.

thats the latest unfortunately. I think a simple fix would be to drop is_utc field setting
in orage since it seems to gone in libical3, I will try to cookup a patch and try

sent here
https://patchwork.openembedded.org/patch/163311/

we can merge libical3 update

Quoted 44 lines
Quoted 44 lines

Ross

Quoted 40 lines · Khem Raj

this is causing a failure in meta-xfce

https://errors.yoctoproject.org/Errors/Details/253910/

Quoted 34 lines · Khem Raj
Quoted 23 lines · Anuj
Quoted 21 lines · Khem Raj
Quoted 2 lines · Anuj
Quoted 1 line · Ross Burton

-LICENSE = "LGPLv2.1 | MPL-1.0"

this should be MPL-2.0 | LGPL-2.1

Quoted 13 lines
Quoted 11 lines

-LIC_FILES_CHKSUM = "file://COPYING;md5=d4fc58309d8ed46587ac63bb449d82f8 \
- file://LICENSE;md5=d1a0891cd3e582b3e2ec8fe63badbbb6"

(quoted diff) +2 -0
+LICENSE = "LGPLv2.1 | MPL-2"
+LIC_FILES_CHKSUM =

"file://LICENSE;md5=1910a2a76ddf6a9ba369182494170d87 \
+ file://LICENSE.LGPL21.txt;md5=933adb561f159e7c3da079536f0ed871 \
+ file://LICENSE.MPL2.txt;md5=9741c346eef56131163e13b9db1241b3"

Should this be MPL-2.0 instead? This is causing warnings:

Quoted 4 lines
Quoted 2 lines

the license is MPL-2.0 but the license file name is MPL2.txt so this seems fine.

I meant the field in LICENSE. Quoted the wrong text.

reply quote confused me. yes it is wrong. It should be MPL-2.0

Quoted 2 lines

Thanks.