Re: [Buildroot] [PATCH 1/1] package/dvb-apps: fix build with gcc >= 14

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

On Tue, 23 Jul 2024 18:25:33 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Workaround for glibc 2.31 added by commit
> bbe170dbcf4ddb63e76299214e2c18b16e5cf016 back in 2020 is incorrect as it
> writes new_time (a pointer on time_t) into timespec.tv_sec (i.e. time_t).
> This mistake raises the following build failure with gcc >= 14:
> 
> dvbdate.c: In function 'set_time':
> dvbdate.c:313:18: error: assignment to '__time_t' {aka 'long int'} from 'time_t *' {aka 'long int *'} makes integer from pointer without a cast [-Wint-conversion]
>   313 |         s.tv_sec = new_time;
>       |                  ^
> 
> It shall be noted that gentoo also spotted this mistake:
> https://gitweb.gentoo.org/repo/gentoo.git/commit/media-tv/linuxtv-dvb-apps/files?id=81e99f1c753d1cb564be29b22dcd8927830c4b9a
> 
> Fixes: bbe170dbcf4ddb63e76299214e2c18b16e5cf016
>  - http://autobuild.buildroot.org/results/e99b9ebf602d307fbc32f2a367e95177f53a68f9
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/dvb-apps/0006-fix-glibc-2.31.patch | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Applied to master, thanks. However, I'm wondering if we shouldn't also
remove this package. It is considered legacy
by https://www.linuxtv.org/projects.php. Its repository at
https://linuxtv.org/legacy-hg/dvb-apps is broken (no CSS) and has
seen no commit since 2014. So I believe we could drop it.

Best regards,

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