<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>6813</bug_id>
          
          <creation_ts>2014-10-07 10:53:34 +0000</creation_ts>
          <short_desc>Large file support broken in python on 32-bit systems</short_desc>
          <delta_ts>2014-11-26 18:03:00 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>devtools / tool chain</component>
          <version>1.6</version>
          <rep_platform>All</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.6.2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Paul Eggleton">bluelightning</reporter>
          <assigned_to name="Paul Eggleton">bluelightning</assigned_to>
          <cc>john.toomey</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>46159</commentid>
    <comment_count>0</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2014-10-07 10:53:34 +0000</bug_when>
    <thetext>Python attempts to force large file support on where possible when building. However, when building python for a 32-bit target, because we hardcode ac_cv_sizeof_off_t to 4 for 32-bit targets in site/*, large file support is disabled when python&apos;s configure script checks for it.

We should remove ac_cv_sizeof_off_t from the site files; but when doing this we need to check this won&apos;t cause any regressions. Any autotools-based software whose recipe does not bypass the autoreconf execution and uses AC_CHECK_SIZEOF (as python&apos;s configure does) will work without ac_cv_sizeof_off_t hardcoded since the latter works when cross-compiling, but any that use hand-rolled checks instead that still look at ac_cv_sizeof_off_t might regress.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46309</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2014-10-14 09:31:02 +0000</bug_when>
    <thetext>I&apos;ve looked through all recipes in the OE-Core master, of which only the following have references to ac_cv_sizeof_off_t *and* don&apos;t use autotools_do_configure (and therefore don&apos;t autoreconf):

meta/recipes-support/db/db_6.0.30.bb
meta/recipes-support/db/db_5.3.28.bb
meta/recipes-devtools/rpm/rpm_5.4+cvs.bb
meta/recipes-devtools/rpm/rpm_5.4.14.bb
meta/recipes-devtools/binutils/binutils_2.24.bb
meta/recipes-devtools/gdb/gdb_7.7.1.bb

I checked the default versions of these and it seemed that all produced configure scripts with cross-compile compatible checks for the size; we should double check the other versions too, unless they get dropped in the mean time. However, when it came time to look outside the core (e.g. at meta-oe) the number of recipes that at least don&apos;t use autotools_do_configure suggests to me that we should probably be a bit conservative in fixing this for now.

I&apos;ve sent an alternate fix just to force ac_cv_sizeof_off_t=8 for Python (if &quot;largefile&quot; is in DISTRO_FEATURES) rather than dropping the value from the site files, which should also be safe to backport to stable branches; we can look at applying the proper fix in 1.8.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46587</commentid>
    <comment_count>2</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2014-10-27 15:16:13 +0000</bug_when>
    <thetext>The fix has gone into the master and dizzy branch for the 1.7 release:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=f95d3fce42fe1021d6fb373b6d7b6fe1e84a752b

Next up, backport to 1.6.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>47210</commentid>
    <comment_count>3</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2014-11-26 18:03:00 +0000</bug_when>
    <thetext>The python fix was merged into daisy for 1.6.2 some weeks ago:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?h=daisy&amp;id=19dc8bf950ae2dcca3e8165eaf122d3c1cb7006e

I think we can mark this bug as resolved now; the general issue of large file support enabling across the board is covered under bug 6803.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>