diff mbox series

[1/1] package/vorbis-tools: fix build with gcc >= 14

Message ID 20240721200430.3070270-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/vorbis-tools: fix build with gcc >= 14 | expand

Commit Message

Fabrice Fontaine July 21, 2024, 8:04 p.m. UTC
Fix the following build failure with gcc >= 14:

codec_skeleton.c: In function 'skeleton_process_fisbone_message_header':
codec_skeleton.c:119:9: error: implicit declaration of function 'utf8_decode' [-Wimplicit-function-declaration]
  119 |     if (utf8_decode(header, &decoded) < 0) {
      |         ^~~~~~~~~~~

Fixes:
 - http://autobuild.buildroot.org/results/bb5d56d836e7d0f2a62daa9954878ad6e0b190ed

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 .../0001-fix-build-with-gcc-14.patch          | 51 +++++++++++++++++++
 1 file changed, 51 insertions(+)
 create mode 100644 package/vorbis-tools/0001-fix-build-with-gcc-14.patch

Comments

Arnout Vandecappelle July 22, 2024, 8:02 a.m. UTC | #1
On 21/07/2024 22:04, Fabrice Fontaine wrote:
> Fix the following build failure with gcc >= 14:
> 
> codec_skeleton.c: In function 'skeleton_process_fisbone_message_header':
> codec_skeleton.c:119:9: error: implicit declaration of function 'utf8_decode' [-Wimplicit-function-declaration]
>    119 |     if (utf8_decode(header, &decoded) < 0) {
>        |         ^~~~~~~~~~~
> 
> Fixes:
>   - http://autobuild.buildroot.org/results/bb5d56d836e7d0f2a62daa9954878ad6e0b190ed
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

  Applied to master, thanks.

  Regards,
  Arnout

> ---
>   .../0001-fix-build-with-gcc-14.patch          | 51 +++++++++++++++++++
>   1 file changed, 51 insertions(+)
>   create mode 100644 package/vorbis-tools/0001-fix-build-with-gcc-14.patch
> 
> diff --git a/package/vorbis-tools/0001-fix-build-with-gcc-14.patch b/package/vorbis-tools/0001-fix-build-with-gcc-14.patch
> new file mode 100644
> index 0000000000..a05ce8a584
> --- /dev/null
> +++ b/package/vorbis-tools/0001-fix-build-with-gcc-14.patch
> @@ -0,0 +1,51 @@
> +From f4a31fe975ea37537f39ace4420aee893b7e0b1a Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Sun, 21 Jul 2024 21:57:38 +0200
> +Subject: [PATCH] fix build with gcc >= 14
> +
> +Add missing includes to avoid the following build failure with gcc >=
> +14:
> +
> +codec_skeleton.c: In function 'skeleton_process_fisbone_message_header':
> +codec_skeleton.c:119:9: error: implicit declaration of function 'utf8_decode' [-Wimplicit-function-declaration]
> +  119 |     if (utf8_decode(header, &decoded) < 0) {
> +      |         ^~~~~~~~~~~
> +
> +Fixes:
> + - http://autobuild.buildroot.org/results/bb5d56d836e7d0f2a62daa9954878ad6e0b190ed
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Upstream: https://github.com/xiph/vorbis-tools/pull/48
> +---
> + ogginfo/codec_skeleton.c | 2 ++
> + share/utf8.c             | 1 +
> + 2 files changed, 3 insertions(+)
> +
> +diff --git a/ogginfo/codec_skeleton.c b/ogginfo/codec_skeleton.c
> +index a27f8da..f906f8c 100644
> +--- a/ogginfo/codec_skeleton.c
> ++++ b/ogginfo/codec_skeleton.c
> +@@ -28,6 +28,8 @@
> +
> + #include "private.h"
> +
> ++#include "utf8.h"
> ++
> + typedef struct {
> +     bool supported;
> +     uint16_t version_major;
> +diff --git a/share/utf8.c b/share/utf8.c
> +index be378d1..e819b67 100644
> +--- a/share/utf8.c
> ++++ b/share/utf8.c
> +@@ -28,6 +28,7 @@
> + #include <stdlib.h>
> + #include <string.h>
> +
> ++#include "charset.h"
> + #include "utf8.h"
> +
> +
> +--
> +2.43.0
> +
Peter Korsgaard Aug. 30, 2024, 12:21 p.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Fix the following build failure with gcc >= 14:
 > codec_skeleton.c: In function 'skeleton_process_fisbone_message_header':
 > codec_skeleton.c:119:9: error: implicit declaration of function 'utf8_decode' [-Wimplicit-function-declaration]
 >   119 |     if (utf8_decode(header, &decoded) < 0) {
 >       |         ^~~~~~~~~~~

 > Fixes:
 >  - http://autobuild.buildroot.org/results/bb5d56d836e7d0f2a62daa9954878ad6e0b190ed

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2024.05.x, thanks.
diff mbox series

Patch

diff --git a/package/vorbis-tools/0001-fix-build-with-gcc-14.patch b/package/vorbis-tools/0001-fix-build-with-gcc-14.patch
new file mode 100644
index 0000000000..a05ce8a584
--- /dev/null
+++ b/package/vorbis-tools/0001-fix-build-with-gcc-14.patch
@@ -0,0 +1,51 @@ 
+From f4a31fe975ea37537f39ace4420aee893b7e0b1a Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Sun, 21 Jul 2024 21:57:38 +0200
+Subject: [PATCH] fix build with gcc >= 14
+
+Add missing includes to avoid the following build failure with gcc >=
+14:
+
+codec_skeleton.c: In function 'skeleton_process_fisbone_message_header':
+codec_skeleton.c:119:9: error: implicit declaration of function 'utf8_decode' [-Wimplicit-function-declaration]
+  119 |     if (utf8_decode(header, &decoded) < 0) {
+      |         ^~~~~~~~~~~
+
+Fixes:
+ - http://autobuild.buildroot.org/results/bb5d56d836e7d0f2a62daa9954878ad6e0b190ed
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Upstream: https://github.com/xiph/vorbis-tools/pull/48
+---
+ ogginfo/codec_skeleton.c | 2 ++
+ share/utf8.c             | 1 +
+ 2 files changed, 3 insertions(+)
+
+diff --git a/ogginfo/codec_skeleton.c b/ogginfo/codec_skeleton.c
+index a27f8da..f906f8c 100644
+--- a/ogginfo/codec_skeleton.c
++++ b/ogginfo/codec_skeleton.c
+@@ -28,6 +28,8 @@
+ 
+ #include "private.h"
+ 
++#include "utf8.h"
++
+ typedef struct {
+     bool supported;
+     uint16_t version_major;
+diff --git a/share/utf8.c b/share/utf8.c
+index be378d1..e819b67 100644
+--- a/share/utf8.c
++++ b/share/utf8.c
+@@ -28,6 +28,7 @@
+ #include <stdlib.h>
+ #include <string.h>
+ 
++#include "charset.h"
+ #include "utf8.h"
+ 
+ 
+-- 
+2.43.0
+