Bug 2012172

Summary:	cyrus-sasl-2.1.25-no_rpath.patch sets wrong variable
Product:	[Fedora] Fedora	Reporter:	Wolfgang Breyha <wbreyha>
Component:	cyrus-sasl	Assignee:	Simo Sorce <ssorce>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	34	CC:	anon.amish, crypto-team, jjelen, ssorce, vanmeeuwen+fedora
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	cyrus-sasl-2.1.27-16.fc36	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-10-12 13:07:08 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Wolfgang Breyha 2021-10-08 13:43:14 UTC

Description of problem:
The source package contains a patch cyrus-sasl-2.1.25-no_rpath.patch
which tries to remove the runpath check and sets
andrew_runpath_switch="none"

But IMO this should be
andrew_cv_runpath_switch="none"

Version-Release number of selected component (if applicable):
detected in the current Fedora stable build
cyrus-sasl-2.1.27-8.fc34.src.rpm

How reproducible:
I tried to rebuild this package with a custome openssl path and wondered why it doesn't work. After a while I recognized that the LDFLAGS are built in the wrong way if andrew_cv_runpath_switch is empty and wondered why it is empty at all.

And then I found the patch removing the original check and setting the wrong value.

Comment 1 Simo Sorce 2021-10-11 21:43:47 UTC

Looks to me that you are right.
I am trying a scratch build with the patch fixed to add _cv_ in it.

Comment 2 Fedora Update System 2021-10-12 13:05:57 UTC

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

Comment 3 Fedora Update System 2021-10-12 13:07:08 UTC

FEDORA-2021-9e8fb2ee2f has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.