Bug 32972

Summary:	wine doesn't load its dependencies.
Product:	Mageia	Reporter:	Aurelian R <arusanu>
Component:	RPM Packages	Assignee:	Thierry Vignaud <thierry.vignaud>
Status:	NEW ---	QA Contact:
Severity:	normal
Priority:	Normal	CC:	yvesbrungard
Version:	Cauldron
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	wine-9.0-1.mga10.src.rpm	CVE:
Status comment:
Attachments:	wine.spec patch to fix wine32 package wine.spec patch to fix mingw dependencies wine for crosscompiling with mingw wine.spec patch build/compiling options

Description Aurelian R 2024-03-15 22:09:55 CET

"wine" has some issues with loading its dependencies on all supported architectures and especially on x86_64 with 32-bit support enabled. So, while I hope I'm not overstepping, I opened this bug to propose some solutions for the long-standing wine bugs in one place as they all have the same source, namely the wine.spec file not properly setting requirements/loading of system libraries.
Reference bugs: bug#28814, bug#28840, bug#30462(closed but some lingering issues remain)

As wine spec file is a little stuffy, this should be solved/reviewed in three steps:
1. fix wine32 rpm package setup/settings.
2. fix dependencies on mingw libraries during build and install.
3. some clean up/clarification of the building flags options and stages.

For each step I will upload a patch for the wine.spec file. These patches should be independent of each other and the order of applying them should not matter.
Patches will be against wine.spec revision=2042087 for Cauldron. 
I've been running wine with these kind of changes for a long time with no issues and unfortunately I can only confirm successful building for aarch64.

Comment 1 Aurelian R 2024-03-15 22:13:23 CET

Created attachment 14462 [details]
wine.spec patch to fix wine32 package

Patch 001* changes/fixes:
- wine32 dependencies
- loading the right wine-gecko library for each architecture
- the warning/notice to load wine32 for 32-bit support as it makes sense only for x86_64 systems
- the double listings of some manual files.

Comment 2 Aurelian R 2024-03-15 22:16:49 CET

Created attachment 14463 [details]
wine.spec patch to fix mingw dependencies

Patch 002* changes/fixes:
- wine32/wine64 properly require their 32/64-bit mingw packages
- use "%bcond_with mingw" for less nested/cluttered conditionals
- enabling/disabling mingw build will properly generate a PE/native build of wine
- rearrange/clean the rpm file list generated with or without using mingw.

Comment 3 Aurelian R 2024-03-15 22:20:12 CET

Created attachment 14464 [details]
wine for crosscompiling with mingw

This is an additional patch that enables use of Mageia mingw libraries instead of wine bundled libraries.

Comment 4 Aurelian R 2024-03-15 22:22:35 CET

Created attachment 14465 [details]
wine.spec patch build/compiling options

Patch 003* changes/fixes:
- clean/rearrange build options
- Use rpm macros to set/disable compiler and linker options(fortify, stack-protector ...)
- explicitly disable "oss", "capi" support in wine build( Mageia settings - oss too old, capi disabled)
- simplify "chrpath"

Comment 5 Lewis Smith 2024-03-16 08:47:05 CET

Well, thank you for all this work.

Assigning to tv who normally deals with Wine; CC'ing papoteur who pt up v9.0.

CC: (none) => yvesbrungard
Assignee: bugsquad => thierry.vignaud