Bug 2045404

Created attachment 1854196 [details]
build.log

Created attachment 1854197 [details]
root.log

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

Created attachment 1854198 [details]
state.log

This bug appears to have been reported against 'rawhide' during the Fedora 36 development cycle.
Changing version to 36.

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

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)'

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.

gcc-12.0.1-0.12.fc37 in rawhide should fix this, f36 version still building...

FEDORA-2022-9fb66de66d has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-9fb66de66d

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.

(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

(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.

FTR, https://src.fedoraproject.org/rpms/game-music-emu/c/70b69a5ea59700d534fc6bbcddb8b0ec0480ebf1?branch=rawhide

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.

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.