Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 455308 - app-emulation/wine-1.5.23: clang creates empty TEXTREL entries
Summary: app-emulation/wine-1.5.23: clang creates empty TEXTREL entries
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Wine Maintainers
URL: http://www.winehq.org/pipermail/wine-...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-03 16:26 UTC by Dennis Schridde
Modified: 2013-02-04 03:24 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
QA Notice (file_455308.txt,22.26 KB, text/plain)
2013-02-03 16:26 UTC, Dennis Schridde
Details
emerge --info (emerge.info,3.14 KB, text/plain)
2013-02-03 16:28 UTC, Dennis Schridde
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Schridde 2013-02-03 16:26:19 UTC
Portage reports QA notices after building wine-1.5.23.

[ebuild   R    ] app-emulation/wine-1.5.23  USE="X alsa cups gecko jpeg lcms ldap mono mp3 ncurses nls openal opengl osmesa oss perl png prelink pulseaudio samba ssl threads truetype udisks win32 win64 xcomposite xinerama xml -capi -custom-cflags -fontconfig -gnutls -gphoto2 -gsm (-gstreamer) -odbc -opencl -scanner (-selinux) {-test} -v4l" 0 kB

Reproducible: Always
Comment 1 Dennis Schridde 2013-02-03 16:26:44 UTC
Created attachment 337808 [details]
QA Notice
Comment 2 Dennis Schridde 2013-02-03 16:28:28 UTC
Created attachment 337810 [details]
emerge --info
Comment 3 Rafał Mużyło 2013-02-03 20:24:53 UTC
Source of the change is following commit:
http://source.winehq.org/git/wine.git/commitdiff/c14bdaf1ddb7d0e5587f63f1216b61c9ecb7a8c3

AFAICT, there's no opt-out.
The interesting thing about this is that those entries seem to be empty.
Comment 4 Alexandre Rostovtsev (RETIRED) gentoo-dev 2013-02-04 02:38:54 UTC
Thanks for reporting. Fixed by patching winebuild to respect the build-time value of $CCAS instead of just automagically looking for clang.

Patch submitted upstream at http://www.winehq.org/pipermail/wine-patches/2013-February/122040.html

+*wine-1.5.23-r1 (04 Feb 2013)
+
+  04 Feb 2013; Alexandre Rostovtsev <tetromino@gentoo.org>
+  +wine-1.5.23-r1.ebuild, +files/wine-1.5.23-winebuild-AS.patch,
+  wine-9999.ebuild:
+  Use the correct system 'as' command as the assembler; fixes text relocation in
+  wine binaries on systems with sys-devel/clang installed (bug #455308, thanks
+  to Dennis Schridde and Rafał Mużyło).
Comment 5 Rafał Mużyło 2013-02-04 03:09:40 UTC
TBH, I'd be more interested in hearing *why* such effect happens.
Comment 6 Alexandre Rostovtsev (RETIRED) gentoo-dev 2013-02-04 03:18:14 UTC
(In reply to comment #5)
> TBH, I'd be more interested in hearing *why* such effect happens.

@clang maintainers, any idea why clang-3.2 produces textrels in wine binaries when used as a linker?
Comment 7 Rafał Mużyło 2013-02-04 03:24:36 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > TBH, I'd be more interested in hearing *why* such effect happens.
> 
> @clang maintainers, any idea why clang-3.2 produces textrels in wine
> binaries when used as a linker?

More exactly, why does it produce *bogus* entries - scanelf says "has TEXTREL markings but doesnt appear to have any real TEXTREL's !?" and indeed readelf says the size of the entry is 0.