Bug 2156397 - bristol rpm fails to build
Summary: bristol rpm fails to build
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: bristol
Version: 37
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-12-26 19:36 UTC by Josh Cogliati
Modified: 2023-01-15 14:56 UTC (History)
2 users (show)

Fixed In Version: bristol-0.60.11-22.fc37
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-01-15 01:58:43 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to remove need for -fcommon (6.85 KB, patch)
2022-12-26 19:36 UTC, Josh Cogliati
no flags Details | Diff
Patch to modify bristol.spec (857 bytes, patch)
2022-12-26 19:39 UTC, Josh Cogliati
no flags Details | Diff

Description Josh Cogliati 2022-12-26 19:36:50 UTC
Created attachment 1934525 [details]
Patch to remove need for -fcommon

Description of problem:
bristol currently fails to build: 
https://koji.fedoraproject.org/koji/buildinfo?buildID=2007040

Version-Release number of selected component (if applicable):
bristol-0.60.11-19.fc36.src.rpm

(Note this also is the root cause of Bug 2116055 )

How reproducible:
run rpmbuild on spec.

Steps to Reproduce:
1. Install fedora 37
2. rpmbuild -rb bristol-0.60.11-19.fc36.src.rpm

Actual results:
...
checking for suffix of executables... 
checking whether we are cross compiling... configure: error: in `/home/jjc/rpmbuild/BUILD/bristol-0.60.11':
configure: error: cannot run C compiled programs.
If you meant to cross compile, use `--host'.
See `config.log' for more details
error: Bad exit status from /var/tmp/rpm-tmp.fapy0b (%build)

RPM build warnings:
    user mockbuild does not exist - using root
    user mockbuild does not exist - using root
    user mockbuild does not exist - using root
    user mockbuild does not exist - using root
    user mockbuild does not exist - using root

RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.fapy0b (%build)


Expected results:
The rpm to build.

Additional info:
Investigation revealed that the problem is that bristol is using the legacy -fcommon option, which caused the configure to fail.

This can be fixed by removing the need for -fcommon with the attached patches and modifying the spec with:

--- bristol.spec	2021-08-30 05:55:25.000000000 -0600
+++ bristol_new.spec	2022-12-26 12:25:32.456339756 -0700
@@ -9,6 +9,7 @@
 Source1:    %{name}.desktop
 Patch0:     bristol-0.60.9-CVE-2010-3351.patch
 Patch1:     bristol-0.60.11-fix-build-with-alsa.patch
+Patch2:     bristol-0.60.11-fix-common.patch
 
 BuildRequires: gcc autoconf automake libtool
 BuildRequires: libX11-devel alsa-lib-devel jack-audio-connection-kit-devel desktop-file-utils
@@ -47,7 +48,7 @@
 
 %build
 autoreconf -if
-CFLAGS="-fcommon" ./configure --prefix=%{_prefix} --libdir=%{_libdir} --enable-static=no --disable-version-check
+./configure --prefix=%{_prefix} --libdir=%{_libdir} --enable-static=no --disable-version-check
 sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=""|g' libtool
 sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool

Comment 1 Josh Cogliati 2022-12-26 19:39:12 UTC
Created attachment 1934526 [details]
Patch to modify bristol.spec

Comment 2 Josh Cogliati 2022-12-27 18:00:00 UTC
Pull request: https://src.fedoraproject.org/rpms/bristol/pull-request/1

Comment 3 Fedora Update System 2023-01-06 16:09:55 UTC
FEDORA-2023-7e75aab3a8 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2023-7e75aab3a8

Comment 4 Fedora Update System 2023-01-07 02:20:48 UTC
FEDORA-2023-7e75aab3a8 has been pushed to the Fedora 37 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-7e75aab3a8`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-7e75aab3a8

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

Comment 5 Fedora Update System 2023-01-15 01:58:43 UTC
FEDORA-2023-7e75aab3a8 has been pushed to the Fedora 37 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 6 Josh Cogliati 2023-01-15 14:56:31 UTC
FYI: the fix has also been filed upstream: https://sourceforge.net/p/bristol/bugs/98/


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