bugGNU gettext - Bugs: bug #43037, fix tests/autopoint-3,...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #43037: fix tests/autopoint-3, cross-compile libintl

Submitter:  Benedikt Morbach <bmorbach>
Submitted:  Wed 20 Aug 2014 11:58:51 PM UTC
   
 
Category:  Build Severity:  3 - Normal
Item Group:  None Status:  Fix Released
Privacy:  Public Assigned to:  haible
Open/Closed:  Closed

Thu 09 May 2019 07:18:05 PM UTC, comment #4: 

The fix is contained in the gettext-0.20 release.

Bruno Haible <haible>
Group administrator
Tue 02 Jan 2018 06:40:11 PM UTC, comment #3: 

Thanks for the report. Fixed through
http://git.savannah.gnu.org/gitweb/?p=gettext.git;a=commitdiff;h=d1e71423b911a70a36dc7b764e46954ba7f4f668
and
http://git.savannah.gnu.org/gitweb/?p=gettext.git;a=commitdiff;h=d527cb6e70d82d109c08c331238e63dfa9c36a77

> This will also need to be added to archive.dir.tar, but afaics that is generated on release?


Yes, this is part of the release preparation steps.

Bruno Haible <haible>
Group administrator
Fri 05 Sep 2014 06:46:22 AM UTC, comment #2: 

By the way, if you just want to avoid the test failure, the safest solution would be using libtool.  I've tweaked the test to do so:
http://git.savannah.gnu.org/cgit/gettext.git/commit/?h=maint&id=d1e71423b911a70a36dc7b764e46954ba7f4f668

Daiki Ueno <ueno>
Group administrator
Fri 05 Sep 2014 03:42:05 AM UTC, comment #1: 

Thanks, the patch makes sense.  I'd like to make a couple of minor modifications:

- Add a guard around AC_CHECK_TOOLS, to make sure if $AR is not defined.  It might already set to other command, like "cc -c.a" on Minix:
http://git.savannah.gnu.org/cgit/gnulib.git/tree/m4/gnulib-common.m4#n272

- Supply the default value to AC_CHECK_TOOLS, so @AR@ be always substituted.

Daiki Ueno <ueno>
Group administrator
Wed 20 Aug 2014 11:58:51 PM UTC, original submission:  

currently libintl fails to build in tests/autopoint-3 here, as it uses AR = ar, but our cross-compile toolchain only has host-/target-prefixed tools.

CC, RANLIB, .. use the proper host-prefixed names to allow cross compilation.

The attached patch adds the same behavior for ar.

This will also need to be added to archive.dir.tar, but afaics that is generated on release?

Benedikt Morbach <bmorbach>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by haible (Updated the item)
  • -email is unavailable- added by ueno (Posted a comment)
  • -email is unavailable- added by bmorbach (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-01-04 haible StatusFixed Fix Released
    2018-01-02 haible StatusNone Fixed
        Assigned toNone haible
        Open/ClosedOpen Closed
    2016-11-27 haible CategoryNone Build
    2014-08-20 bmorbach Attached File- Added gettext-0.19.2-look-for-host-prefixed-ar.patch, #31930

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code