[Buildroot] [PATCH] package/libedit: fix compile error

[Buildroot] [PATCH] package/libedit: fix compile error

[Waldemar Brodkorb]

Header file uses ssize_t so sys/types.h must be included.

Fixes:
 - http://autobuild.buildroot.net/results/439/439538b23776a58ff81c38754a19ee92c3590a1f

Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
---
Author was informed via email about the issue.
---
 package/libedit/0002-add-sys-types_h.patch | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
 create mode 100644 package/libedit/0002-add-sys-types_h.patch

diff --git a/package/libedit/0002-add-sys-types_h.patch b/package/libedit/0002-add-sys-types_h.patch
new file mode 100644
index 0000000000..26aa906ee3
--- /dev/null
+++ b/package/libedit/0002-add-sys-types_h.patch
@@ -0,0 +1,15 @@
+Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
+Upstream: N/A only reachable via mail
+
+diff -Nur libedit-20221030-3.1.orig/src/sys.h libedit-20221030-3.1/src/sys.h
+--- libedit-20221030-3.1.orig/src/sys.h	2022-10-30 06:35:40.000000000 +0100
++++ libedit-20221030-3.1/src/sys.h	2023-08-12 09:52:28.137223311 +0200
+@@ -40,7 +40,7 @@
+ #ifndef _h_sys
+ #define	_h_sys
+ 
+-#if defined(HAVE_SYS_TYPES_H) && defined(__sun)
++#if defined(HAVE_SYS_TYPES_H)
+ #include <sys/types.h>
+ #endif
+ 
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libedit: fix compile error

[Thomas Petazzoni via buildroot]

Hello Waldemar,

On Sat, 12 Aug 2023 10:21:34 +0200
Waldemar Brodkorb <wbx@openadk.org> wrote:

> Header file uses ssize_t so sys/types.h must be included.
> 
> Fixes:
>  - http://autobuild.buildroot.net/results/439/439538b23776a58ff81c38754a19ee92c3590a1f
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> ---
> Author was informed via email about the issue.
> ---
>  package/libedit/0002-add-sys-types_h.patch | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>  create mode 100644 package/libedit/0002-add-sys-types_h.patch

I have applied to master, but after significantly improving the commit
log. For all the "fixes" that you send, there is almost always a
critical information that is missing: since when is the problem
occurring. Indeed, we want to know if the fix needs to be backported to
one of our stable branches or not.

So here, I had to do that research myself, to realize it's the version
bump in commit cc69dbf8fdef6619c4ea09a4aaa97cd5ad3ee423 that introduced
the regression. Since this commit is no yet in any release (other than
2023.08-rc1), there is no need to backport your fix to a previous
stable Buildroot branch. This is really a critical piece of information
that Peter Korsgaard will need when reviewing patches for backport to
stable branches.

So could you please always, always, always to this research and include
the relevant information in the commit log? It will make things a lot
easier when reviewing/merging the patches.

The final commit log looks like this:

commit fe42fa8017b10602ed57ee066f8a9710f0f8e7ca (HEAD -> master, official/master)
Author: Waldemar Brodkorb <wbx@openadk.org>
Date:   Sat Aug 12 10:21:34 2023 +0200

    package/libedit: fix compile error with uClibc
    
    In Buildroot commit cc69dbf8fdef6619c4ea09a4aaa97cd5ad3ee423
    ("package/libedit: bump version to 20221030-3.1"), the version of
    libedit was bumped from 20210910-3.1 to 20221030-3.1. This broke the
    build with uClibc:
    
    ./sys.h:96:1: error: unknown type name 'ssize_t'; did you mean 'size_t'?
    
    Header file uses ssize_t so sys/types.h must be included.
    
    This commit adds a patch that fixes this issue.
    
    Fixes:
    
      http://autobuild.buildroot.net/results/439538b23776a58ff81c38754a19ee92c3590a1f
    
    Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
    Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/libedit: fix compile error

[Waldemar Brodkorb]

Hi Thomas,
Thomas Petazzoni wrote,

> Hello Waldemar,
> 
> On Sat, 12 Aug 2023 10:21:34 +0200
> Waldemar Brodkorb <wbx@openadk.org> wrote:
> 
> > Header file uses ssize_t so sys/types.h must be included.
> > 
> > Fixes:
> >  - http://autobuild.buildroot.net/results/439/439538b23776a58ff81c38754a19ee92c3590a1f
> > 
> > Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> > ---
> > Author was informed via email about the issue.
> > ---
> >  package/libedit/0002-add-sys-types_h.patch | 15 +++++++++++++++
> >  1 file changed, 15 insertions(+)
> >  create mode 100644 package/libedit/0002-add-sys-types_h.patch
> 
> I have applied to master, but after significantly improving the commit
> log. For all the "fixes" that you send, there is almost always a
> critical information that is missing: since when is the problem
> occurring. Indeed, we want to know if the fix needs to be backported to
> one of our stable branches or not.
> 
> So here, I had to do that research myself, to realize it's the version
> bump in commit cc69dbf8fdef6619c4ea09a4aaa97cd5ad3ee423 that introduced
> the regression. Since this commit is no yet in any release (other than
> 2023.08-rc1), there is no need to backport your fix to a previous
> stable Buildroot branch. This is really a critical piece of information
> that Peter Korsgaard will need when reviewing patches for backport to
> stable branches.
> 
> So could you please always, always, always to this research and include
> the relevant information in the commit log? It will make things a lot
> easier when reviewing/merging the patches.
> 
> The final commit log looks like this:
> 
> commit fe42fa8017b10602ed57ee066f8a9710f0f8e7ca (HEAD -> master, official/master)
> Author: Waldemar Brodkorb <wbx@openadk.org>
> Date:   Sat Aug 12 10:21:34 2023 +0200
> 
>     package/libedit: fix compile error with uClibc
>     
>     In Buildroot commit cc69dbf8fdef6619c4ea09a4aaa97cd5ad3ee423
>     ("package/libedit: bump version to 20221030-3.1"), the version of
>     libedit was bumped from 20210910-3.1 to 20221030-3.1. This broke the
>     build with uClibc:
>     
>     ./sys.h:96:1: error: unknown type name 'ssize_t'; did you mean 'size_t'?
>     
>     Header file uses ssize_t so sys/types.h must be included.
>     
>     This commit adds a patch that fixes this issue.
>     
>     Fixes:
>     
>       http://autobuild.buildroot.net/results/439538b23776a58ff81c38754a19ee92c3590a1f
>     
>     Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
>     Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> 
> Thanks!

Okay, sorry for that. I promise I try to find the needed information
for my next patches. See my conmon patch I sent today ;)

best regards
 Waldemar
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot