Bug 22584

Summary: libX11 does not cross compile
Product: xorg Reporter: Paul Bender <pebender>
Component: Lib/XlibAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium CC: dottedmag
Version: unspecifiedKeywords: patch
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
cross compilation fixes none

Description Paul Bender 2009-07-01 09:52:26 UTC 
Created attachment 27301 [details] [review]
cross compilation fixes

While some changes have been to libX11 made to fix cross compilation, there are still things missing.

First, while CC has been replaced by CC_FOR_BUILD, CPPFLAGS, CFLAGS and LDFLAGS are still used. These should be replaced by CPPFLAGS_FOR_BUILD, CFLAGS_FOR_BUILD and LDFLAGS_FOR_BUILD.

Second, the pkg-config program is used directly rather than using the PKG_CONFIG variable.

Third, pkg-config's PKG_CONFIG_SYSROOT_DIR is not used.

The attached patch fixes these three issues.
Comment 1 Alan Coopersmith 2010-06-03 17:09:31 UTC 
The portions of the patch for the first two issues were updated to
apply to the libX11 1.3.x branch and included for the libX11-1.3.4
release.

The reviewers dicsussing this on IRC weren't sure about the 
PKG_CONFIG_SYSROOT_DIR changes, so those were left out for
now - they can be added later if you can explain why they're
needed.
Comment 2 Matt Turner 2010-12-03 16:39:47 UTC 
I think this is fixed. Please send a patch to xorg-devel@lists.x.org if it is not. Putting patches on bugzilla is a great way for them to get lost.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.