* [PATCH] libical: backport a patch to fix build with ICU 68.1
@ 2020-11-10 10:47 Ross Burton
2020-11-10 17:46 ` [OE-core] " Khem Raj
0 siblings, 1 reply; 2+ messages in thread
From: Ross Burton @ 2020-11-10 10:47 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../0001-Fix-build-with-icu-68.1.patch | 81 +++++++++++++++++++
meta/recipes-support/libical/libical_3.0.8.bb | 1 +
2 files changed, 82 insertions(+)
create mode 100644 meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
diff --git a/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch b/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
new file mode 100644
index 00000000000..79e1475c6c5
--- /dev/null
+++ b/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
@@ -0,0 +1,81 @@
+Upstream-Status: Backport
+Signed-off-by: Ross Burton <ross.burton@arm.com>
+
+From 52d603212173d4502800bf746436a93f806a8898 Mon Sep 17 00:00:00 2001
+From: Allen Winter <allen.winter@kdab.com>
+Date: Sat, 31 Oct 2020 17:38:01 -0400
+Subject: [PATCH] Fix build with icu-68.1
+
+Use stdbool.h to define true and false when possible
+
+Issue#448
+---
+ ConfigureChecks.cmake | 1 +
+ config.h.cmake | 3 +++
+ src/libical/icalrecur.c | 10 ++++++++--
+ 3 files changed, 12 insertions(+), 2 deletions(-)
+
+diff --git a/ConfigureChecks.cmake b/ConfigureChecks.cmake
+index e4d0e2ec..bba33fa9 100644
+--- a/ConfigureChecks.cmake
++++ b/ConfigureChecks.cmake
+@@ -10,6 +10,7 @@ check_include_files(sys/utsname.h HAVE_SYS_UTSNAME_H)
+ check_include_files(fcntl.h HAVE_FCNTL_H)
+ check_include_files(unistd.h HAVE_UNISTD_H)
+ check_include_files(wctype.h HAVE_WCTYPE_H)
++check_include_files(stdbool.h HAVE_STDBOOL_H)
+
+ include(CheckFunctionExists)
+ if(WIN32 AND MSVC)
+diff --git a/config.h.cmake b/config.h.cmake
+index 8fd3421b..c8008692 100644
+--- a/config.h.cmake
++++ b/config.h.cmake
+@@ -39,6 +39,9 @@
+ /* Define to 1 if you have the <dirent.h> header file. */
+ #cmakedefine HAVE_DIRENT_H 1
+
++/* Define to 1 if you have the <stdbool.h> header file. */
++#cmakedefine HAVE_STDBOOL_H 1
++
+ /* Define if we have pthread. */
+ #cmakedefine HAVE_PTHREAD_ATTR_GET_NP 1
+ #cmakedefine HAVE_PTHREAD_GETATTR_NP 1
+diff --git a/src/libical/icalrecur.c b/src/libical/icalrecur.c
+index 73178450..b3f054b8 100644
+--- a/src/libical/icalrecur.c
++++ b/src/libical/icalrecur.c
+@@ -145,6 +145,12 @@
+ #if defined(HAVE_LIBICU)
+ #include <unicode/ucal.h>
+ #include <unicode/ustring.h>
++#if defined(HAVE_STDBOOL_H)
++#include <stdbool.h>
++#else
++#define false 0
++#define true 1
++#endif
+ #define RSCALE_IS_SUPPORTED 1
+ #else
+ #define RSCALE_IS_SUPPORTED 0
+@@ -1018,7 +1024,7 @@ icalarray *icalrecurrencetype_rscale_supported_calendars(void)
+
+ calendars = icalarray_new(sizeof(const char **), 20);
+
+- en = ucal_getKeywordValuesForLocale("calendar", NULL, FALSE, &status);
++ en = ucal_getKeywordValuesForLocale("calendar", NULL, false, &status);
+ while ((cal = uenum_next(en, NULL, &status))) {
+ cal = icalmemory_tmp_copy(cal);
+ icalarray_append(calendars, &cal);
+@@ -1411,7 +1417,7 @@ static int initialize_rscale(icalrecur_iterator *impl)
+ }
+
+ /* Check if specified calendar is supported */
+- en = ucal_getKeywordValuesForLocale("calendar", NULL, FALSE, &status);
++ en = ucal_getKeywordValuesForLocale("calendar", NULL, false, &status);
+ while ((cal = uenum_next(en, NULL, &status))) {
+ if (!strcmp(cal, rule.rscale)) {
+ is_hebrew = !strcmp(rule.rscale, "hebrew");
+--
+2.25.1
+
diff --git a/meta/recipes-support/libical/libical_3.0.8.bb b/meta/recipes-support/libical/libical_3.0.8.bb
index efb9433412f..6230f09eca1 100644
--- a/meta/recipes-support/libical/libical_3.0.8.bb
+++ b/meta/recipes-support/libical/libical_3.0.8.bb
@@ -11,6 +11,7 @@ SECTION = "libs"
SRC_URI = " \
https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz \
file://0001-Use-our-hand-build-native-src-generator.patch \
+ file://0001-Fix-build-with-icu-68.1.patch \
"
SRC_URI[md5sum] = "41bd1f1fcdcb4779cea478bb55cf07bf"
SRC_URI[sha256sum] = "09fecacaf75ba5a242159e3a9758a5446b5ce4d0ab684f98a7040864e1d1286f"
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [OE-core] [PATCH] libical: backport a patch to fix build with ICU 68.1
2020-11-10 10:47 [PATCH] libical: backport a patch to fix build with ICU 68.1 Ross Burton
@ 2020-11-10 17:46 ` Khem Raj
0 siblings, 0 replies; 2+ messages in thread
From: Khem Raj @ 2020-11-10 17:46 UTC (permalink / raw)
To: openembedded-core
thanks this is better.
On 11/10/20 2:47 AM, Ross Burton wrote:
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
> .../0001-Fix-build-with-icu-68.1.patch | 81 +++++++++++++++++++
> meta/recipes-support/libical/libical_3.0.8.bb | 1 +
> 2 files changed, 82 insertions(+)
> create mode 100644 meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
>
> diff --git a/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch b/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
> new file mode 100644
> index 00000000000..79e1475c6c5
> --- /dev/null
> +++ b/meta/recipes-support/libical/libical/0001-Fix-build-with-icu-68.1.patch
> @@ -0,0 +1,81 @@
> +Upstream-Status: Backport
> +Signed-off-by: Ross Burton <ross.burton@arm.com>
> +
> +From 52d603212173d4502800bf746436a93f806a8898 Mon Sep 17 00:00:00 2001
> +From: Allen Winter <allen.winter@kdab.com>
> +Date: Sat, 31 Oct 2020 17:38:01 -0400
> +Subject: [PATCH] Fix build with icu-68.1
> +
> +Use stdbool.h to define true and false when possible
> +
> +Issue#448
> +---
> + ConfigureChecks.cmake | 1 +
> + config.h.cmake | 3 +++
> + src/libical/icalrecur.c | 10 ++++++++--
> + 3 files changed, 12 insertions(+), 2 deletions(-)
> +
> +diff --git a/ConfigureChecks.cmake b/ConfigureChecks.cmake
> +index e4d0e2ec..bba33fa9 100644
> +--- a/ConfigureChecks.cmake
> ++++ b/ConfigureChecks.cmake
> +@@ -10,6 +10,7 @@ check_include_files(sys/utsname.h HAVE_SYS_UTSNAME_H)
> + check_include_files(fcntl.h HAVE_FCNTL_H)
> + check_include_files(unistd.h HAVE_UNISTD_H)
> + check_include_files(wctype.h HAVE_WCTYPE_H)
> ++check_include_files(stdbool.h HAVE_STDBOOL_H)
> +
> + include(CheckFunctionExists)
> + if(WIN32 AND MSVC)
> +diff --git a/config.h.cmake b/config.h.cmake
> +index 8fd3421b..c8008692 100644
> +--- a/config.h.cmake
> ++++ b/config.h.cmake
> +@@ -39,6 +39,9 @@
> + /* Define to 1 if you have the <dirent.h> header file. */
> + #cmakedefine HAVE_DIRENT_H 1
> +
> ++/* Define to 1 if you have the <stdbool.h> header file. */
> ++#cmakedefine HAVE_STDBOOL_H 1
> ++
> + /* Define if we have pthread. */
> + #cmakedefine HAVE_PTHREAD_ATTR_GET_NP 1
> + #cmakedefine HAVE_PTHREAD_GETATTR_NP 1
> +diff --git a/src/libical/icalrecur.c b/src/libical/icalrecur.c
> +index 73178450..b3f054b8 100644
> +--- a/src/libical/icalrecur.c
> ++++ b/src/libical/icalrecur.c
> +@@ -145,6 +145,12 @@
> + #if defined(HAVE_LIBICU)
> + #include <unicode/ucal.h>
> + #include <unicode/ustring.h>
> ++#if defined(HAVE_STDBOOL_H)
> ++#include <stdbool.h>
> ++#else
> ++#define false 0
> ++#define true 1
> ++#endif
> + #define RSCALE_IS_SUPPORTED 1
> + #else
> + #define RSCALE_IS_SUPPORTED 0
> +@@ -1018,7 +1024,7 @@ icalarray *icalrecurrencetype_rscale_supported_calendars(void)
> +
> + calendars = icalarray_new(sizeof(const char **), 20);
> +
> +- en = ucal_getKeywordValuesForLocale("calendar", NULL, FALSE, &status);
> ++ en = ucal_getKeywordValuesForLocale("calendar", NULL, false, &status);
> + while ((cal = uenum_next(en, NULL, &status))) {
> + cal = icalmemory_tmp_copy(cal);
> + icalarray_append(calendars, &cal);
> +@@ -1411,7 +1417,7 @@ static int initialize_rscale(icalrecur_iterator *impl)
> + }
> +
> + /* Check if specified calendar is supported */
> +- en = ucal_getKeywordValuesForLocale("calendar", NULL, FALSE, &status);
> ++ en = ucal_getKeywordValuesForLocale("calendar", NULL, false, &status);
> + while ((cal = uenum_next(en, NULL, &status))) {
> + if (!strcmp(cal, rule.rscale)) {
> + is_hebrew = !strcmp(rule.rscale, "hebrew");
> +--
> +2.25.1
> +
> diff --git a/meta/recipes-support/libical/libical_3.0.8.bb b/meta/recipes-support/libical/libical_3.0.8.bb
> index efb9433412f..6230f09eca1 100644
> --- a/meta/recipes-support/libical/libical_3.0.8.bb
> +++ b/meta/recipes-support/libical/libical_3.0.8.bb
> @@ -11,6 +11,7 @@ SECTION = "libs"
> SRC_URI = " \
> https://github.com/${BPN}/${BPN}/releases/download/v${PV}/${BP}.tar.gz \
> file://0001-Use-our-hand-build-native-src-generator.patch \
> + file://0001-Fix-build-with-icu-68.1.patch \
> "
> SRC_URI[md5sum] = "41bd1f1fcdcb4779cea478bb55cf07bf"
> SRC_URI[sha256sum] = "09fecacaf75ba5a242159e3a9758a5446b5ce4d0ab684f98a7040864e1d1286f"
>
>
>
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-11-10 17:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-10 10:47 [PATCH] libical: backport a patch to fix build with ICU 68.1 Ross Burton
2020-11-10 17:46 ` [OE-core] " Khem Raj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox