Bug 2045404 - game-music-emu: FTBFS in Fedora rawhide/f36
Summary: game-music-emu: FTBFS in Fedora rawhide/f36
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gcc
Version: 36
Hardware: s390x
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jakub Jelinek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F36FTBFS F37FTBFS
TreeView+ depends on / blocked
 
Reported: 2022-01-25 16:36 UTC by Fedora Release Engineering
Modified: 2023-01-09 18:37 UTC (History)
13 users (show)

Fixed In Version: gcc-12.0.1-0.12.fc36
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-03-09 20:07:03 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
build.log (23.75 KB, text/plain)
2022-01-25 16:36 UTC, Fedora Release Engineering
no flags Details
root.log (32.00 KB, text/plain)
2022-01-25 16:36 UTC, Fedora Release Engineering
no flags Details
state.log (1008 bytes, text/plain)
2022-01-25 16:36 UTC, Fedora Release Engineering
no flags Details


Links
System ID Private Priority Status Summary Last Updated
GNU Compiler Collection 104775 0 P2 UNCONFIRMED [9/10/11/12 Regression] Failure to assemble on s390x with -fsanitize=undefined 2022-03-03 21:42:54 UTC

Description Fedora Release Engineering 2022-01-25 16:36:12 UTC
game-music-emu failed to build from source in Fedora rawhide/f36

https://koji.fedoraproject.org/koji/taskinfo?taskID=81771775


For details on the mass rebuild see:

https://fedoraproject.org/wiki/Fedora_36_Mass_Rebuild
Please fix game-music-emu at your earliest convenience and set the bug's status to
ASSIGNED when you start fixing it. If the bug remains in NEW state for 8 weeks,
game-music-emu will be orphaned. Before branching of Fedora 37,
game-music-emu will be retired, if it still fails to build.

For more details on the FTBFS policy, please visit:
https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/

Comment 1 Fedora Release Engineering 2022-01-25 16:36:16 UTC
Created attachment 1854196 [details]
build.log

Comment 2 Fedora Release Engineering 2022-01-25 16:36:18 UTC
Created attachment 1854197 [details]
root.log

file root.log too big, will only attach last 32768 bytes

Comment 3 Fedora Release Engineering 2022-01-25 16:36:20 UTC
Created attachment 1854198 [details]
state.log

Comment 4 Ben Cotton 2022-02-08 20:59:28 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 36 development cycle.
Changing version to 36.

Comment 5 Karel Volný 2022-03-03 15:23:17 UTC
to me, this looks like a compiler problem ...

gme code hasn't changed for ages and it compiled just fine in the past

... and still it compiles everywhere except for s390x

https://koji.fedoraproject.org/koji/taskinfo?taskID=83599073

Comment 6 Jonathan Wakely 2022-03-03 15:49:38 UTC
From the attached build.log:

{standard input}: Assembler messages:
{standard input}:26751: Error: syntax error; missing ')' after base register
{standard input}:26751: Error: junk at end of line: `%r10)'

Comment 7 Jakub Jelinek 2022-03-03 21:31:34 UTC
Reproduced and reducing now, the problematic assembly line is:
        clgte   %r4,0(%r1,%r10)
which the assembler doesn't like.
But, are you really using -fsanitize=undefined in fedora package builds (using it say for running the testsuite is ok)?
The sanitizers are debugging aids, not really meant for production use.

Comment 8 Jakub Jelinek 2022-03-09 20:07:03 UTC
gcc-12.0.1-0.12.fc37 in rawhide should fix this, f36 version still building...

Comment 9 Fedora Update System 2022-03-10 08:31:05 UTC
FEDORA-2022-9fb66de66d has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-9fb66de66d

Comment 10 Fedora Update System 2022-03-11 19:25:13 UTC
FEDORA-2022-42ea499a7d has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2022-42ea499a7d`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-42ea499a7d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 11 Karel Volný 2022-03-16 15:25:17 UTC
(In reply to Jakub Jelinek from comment #7)
> But, are you really using -fsanitize=undefined in fedora package builds
> (using it say for running the testsuite is ok)?

well, it comes from upstream

the relevant cmakefile parts are:

option(ENABLE_UBSAN "Enable Undefined Behavior Sanitizer error-checking" ON)

    if (ENABLE_UBSAN)
        # GCC needs -static-libubsan
        if (NOT BUILD_SHARED_LIBS AND CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
            set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=undefined -static-libubsan")
        else()
            set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=undefined")
        endif()
    endif()


... should I disable this in the package?


(In reply to Jakub Jelinek from comment #8)
> gcc-12.0.1-0.12.fc37 in rawhide should fix this, f36 version still
> building...

thanks, looks good

https://koji.fedoraproject.org/koji/taskinfo?taskID=84277845
https://koji.fedoraproject.org/koji/taskinfo?taskID=84278734

Comment 12 Jakub Jelinek 2022-03-16 15:40:03 UTC
(In reply to Karel Volný from comment #11)
> ... should I disable this in the package?

Yes, certainly.
As I said, it is a debugging tool, to be used when developing the package, not to be used in production use of it.

Comment 14 Fedora Update System 2022-03-17 17:10:11 UTC
FEDORA-2022-42ea499a7d has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2022-42ea499a7d`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-42ea499a7d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 15 Fedora Update System 2022-03-17 18:37:23 UTC
FEDORA-2022-42ea499a7d has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.