Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 738509 - xulrunner forces soft-float ABI for all arm targets
Summary: xulrunner forces soft-float ABI for all arm targets
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: xulrunner
Version: 15
Hardware: other
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Gecko Maintainer
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: ARMTracker
TreeView+ depends on / blocked
 
Reported: 2011-09-15 02:54 UTC by Brendan Conoboy
Modified: 2018-04-11 14:41 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-12-23 14:28:51 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
This patch allows xulrunner to build on armv7hl systems (10.25 KB, patch)
2011-09-15 02:55 UTC, Brendan Conoboy 		no flags Details | Diff
Patch to properly configure the various arm platforms (1.43 KB, patch)
2011-12-22 10:44 UTC, Peter Robinson 		no flags Details | Diff
Show Obsolete (1) View All


Links
System ID Private Priority Status Summary Last Updated
Mozilla Foundation 632915 0 None None None Never

Description Brendan Conoboy 2011-09-15 02:54:03 UTC 
Description of problem:

The xulrunner package includes special handling for ARM systems that modify the flags passed to gcc and g++.  One of the flags set the floating point ABI to soft-float, even if the system is hard float.  When the hard float and soft float options are mixed together, gcc bails resulting in a FTBFS.

Version-Release number of selected component (if applicable):

Observed in the latest F15 update to xulrunner, xulrunner-6.0.2-1.fc15.

How reproducible:

Every time.  The symptom is when autoconf tries to identify the size of int * and gets a result of '0'.

Steps to Reproduce:
1. Install Fedora-ARM F15 rootfs
2. Try to build xulrunner.
3. Watch configure break.
  
Actual results:

Configure breaks.

Expected results:

Package should build.

Additional info:

The attached patch removes the setting of the float ABI, instead letting autoconf do its job and get the right flags from gcc (EG, --mfloat-abi=hardfp).  This patch is against xulrunner-6.0.2-1.fc15.  I'm not clear on Fedora package standards wrt configure regeneration, so this patch does include changes to the generated configure.  Taking just the changes to the two configure.in's and rerunning autoconf should give the same results.
Comment 1 Brendan Conoboy 2011-09-15 02:55:52 UTC 
Created attachment 523294 [details]
This patch allows xulrunner to build on armv7hl systems

This patch allows xulrunner to build on armv7hl systems.  It specifically stops configure from always specifying soft-float ABI and instead lets configure work out the right setting itself.
Comment 3 Niels de Vos 2011-12-22 09:44:14 UTC 
I think this is filed upstream already (currently still open):
- https://bugzilla.mozilla.org/show_bug.cgi?id=632915
Comment 4 Peter Robinson 2011-12-22 10:44:49 UTC 
Created attachment 549168 [details]
Patch to properly configure the various arm platforms

Based on xulrunner 9.0 there's proper configure options to set the various build options for the various ARM build options so we just need to set them so they get built. This patch to the current latest xulrunner.spec sets the options we want.
Comment 5 Peter Robinson 2011-12-23 14:28:51 UTC 
Pushed spec changes to xulrunnr in rawhdide.
Note You need to log in before you can comment on or make changes to this bug.