Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 896416 - media-video/mjpegtools-2.2.1 fails to compile (MUSL-CLANG-SYSTEM): fastintfns.h:5:27: error: ISO C++17 does not allow register storage class specifier [-Wregister]
Summary: media-video/mjpegtools-2.2.1 fails to compile (MUSL-CLANG-SYSTEM): fastintfns...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Daniel Pielmeier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-25 07:28 UTC by Agostino Sarubbo
Modified: 2024-03-01 14:50 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,161.98 KB, text/plain)
2023-02-25 07:28 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2023-02-25 07:28:10 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: media-video/mjpegtools-2.2.1 fails to compile (MUSL-CLANG-SYSTEM).
Discovered on: amd64 (internal ref: tinderbox_musl)

NOTE:
(MUSL-CLANG-SYSTEM) in the summary means that bug was found on a machine that runs MUSL libc + clang16 but this bug MAY or MAY NOT BE related to musl/clang16.
Comment 1 Agostino Sarubbo gentoo-dev 2023-02-25 07:28:12 UTC
Created attachment 854522 [details]
build.log

build log and emerge --info
Comment 2 Agostino Sarubbo gentoo-dev 2023-02-25 07:28:14 UTC
Error(s) that match a know pattern in addition to what has been reported in the summary:


/var/tmp/portage/media-video/mjpegtools-2.2.1/work/mjpegtools-2.2.1/utils/motionsearch.c:756:12: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
/var/tmp/portage/media-video/mjpegtools-2.2.1/work/mjpegtools-2.2.1/utils/motionsearch.c:836:12: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
/var/tmp/portage/media-video/mjpegtools-2.2.1/work/mjpegtools-2.2.1/utils/motionsearch.c:889:12: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
/var/tmp/portage/media-video/mjpegtools-2.2.1/work/mjpegtools-2.2.1/utils/fastintfns.h:5:27: error: ISO C++17 does not allow 'register' storage class specifier [-Wregister]
Comment 3 Denis Pronin 2023-08-12 13:13:06 UTC
This commit is about to be closed with https://github.com/gentoo/gentoo/pull/32171
Comment 4 Denis Pronin 2023-08-12 13:13:48 UTC
(In reply to Denis Pronin from comment #3)
> This commit is about to be closed with
> https://github.com/gentoo/gentoo/pull/32171

I am sorry, this *bug*, not a commit, is about to be closed
Comment 5 Larry the Git Cow gentoo-dev 2023-08-12 13:15:09 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=85deb61cae13a121ad252bf49247b3314cb28787

commit 85deb61cae13a121ad252bf49247b3314cb28787
Author:     Denis Pronin <dannftk@yandex.ru>
AuthorDate: 2023-08-04 11:23:58 +0000
Commit:     Daniel Pielmeier <billie@gentoo.org>
CommitDate: 2023-08-12 13:13:18 +0000

    media-video/mjpegtools: fix compiling at least with c++17 standard
    
    add two patches those are:
    - fixing usage of 'register' specifier since, since c++17 standard,
    it's been removed
    - fixing usage pf 'auto_ptr' since, since c++17 standard, it's been
    removed
    
    Closes: https://bugs.gentoo.org/896416
    Signed-off-by: Denis Pronin <dannftk@yandex.ru>
    Closes: https://github.com/gentoo/gentoo/pull/32171
    Signed-off-by: Daniel Pielmeier <billie@gentoo.org>

 .../mjpegtools-2.2.1-c++17-no-auto_ptr-fix.patch     | 20 ++++++++++++++++++++
 .../files/mjpegtools-2.2.1-c++17-register-fix.patch  | 17 +++++++++++++++++
 media-video/mjpegtools/mjpegtools-2.2.1.ebuild       |  3 +++
 3 files changed, 40 insertions(+)