Bug 832358

Summary:	media-video/ffdiaporama-2.2_pre20140701-r2 fails to compile: cSoundBlockList.h:51:5: error: u_int8_t does not name a type; did you mean uint8_t?
Product:	Gentoo Linux	Reporter:	Agostino Sarubbo <ago>
Component:	Current packages	Assignee:	laurantino <olivier.laurantin>
Status:	RESOLVED FIXED
Severity:	normal	CC:	proxy-maint
Priority:	Normal	Keywords:	PullRequest
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://github.com/gentoo/gentoo/pull/24961 https://github.com/gentoo/gentoo/pull/25367
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log

Description Agostino Sarubbo gentoo-dev

2022-01-30 08:10:00 UTC

https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: media-video/ffdiaporama-2.2_pre20140701-r2 fails to compile.
Discovered on: amd64 (internal ref: tinderbox_musl)

NOTE:
This machine uses MUSL libc

Comment 1 Agostino Sarubbo gentoo-dev

2022-01-30 08:10:02 UTC

Created attachment 763933 [details]
build.log

build log and emerge --info

Comment 2 Agostino Sarubbo gentoo-dev

2022-01-30 08:10:03 UTC

Error(s) that match a know pattern in addition to what has been reported in the summary:


In file included from /usr/include/qt5/QtSql/QSqlError:1,
cDatabase.cpp:1060:46: warning: 'int QSqlError::number() const' is deprecated [-Wdeprecated-declarations[https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wdeprecated-declarations]]
cDatabase.cpp:1063:46: warning: 'int QSqlError::number() const' is deprecated [-Wdeprecated-declarations[https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wdeprecated-declarations]]
cDatabase.cpp:155:37: warning: 'int QSqlError::number() const' is deprecated [-Wdeprecated-declarations[https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wdeprecated-declarations]]
cDatabase.cpp:27:79: warning: 'int QSqlError::number() const' is deprecated [-Wdeprecated-declarations[https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wdeprecated-declarations]]
./engine/cSoundBlockList.h:51:5: error: 'u_int8_t' does not name a type; did you mean 'uint8_t'?

Comment 3 laurantino 2022-02-05 13:10:43 UTC

Hi,

Can you try with the following patch as I am not using musl? I will submit a pull request if it works.

https://raw.githubusercontent.com/laurantino/gentoo-dist/master/ffdiaporama-2.2-musl.patch

Thanks

Comment 4 Larry the Git Cow gentoo-dev

2022-05-13 20:04:06 UTC

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=97e71000c7110fb7d5827963a20f8f77c36074ee

commit 97e71000c7110fb7d5827963a20f8f77c36074ee
Author:     Olivier Laurantin <olivier.laurantin@laposte.net>
AuthorDate: 2022-04-09 09:05:54 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-05-13 20:03:54 +0000

    media-video/ffdiaporama: code clean up (fix build w/ newer ffmpeg, musl)
    
    Change sources location
    The code now includes all previous patches
    The code should compile fine with ffmpeg-5 and musl
    
    Closes: https://bugs.gentoo.org/834402
    Closes: https://bugs.gentoo.org/832358
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Olivier Laurantin <olivier.laurantin@laposte.net>
    Signed-off-by: Sam James <sam@gentoo.org>

 media-video/ffdiaporama/Manifest                   |  1 +
 .../ffdiaporama-2.2_pre20140701-r3.ebuild          | 59 ++++++++++++++++++++++
 2 files changed, 60 insertions(+)