bugGNU gettext - Bugs: bug #40083, 0.18.3 autopoint leaks error...

 
 

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

bug #40083: 0.18.3 autopoint leaks error messages from m4_esyscmd

Submitter:  Gary V. Vaughan <gary>
Submitted:  Sun 22 Sep 2013 01:56:56 AM UTC
 
Category:  Programmer tools Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  ueno
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Tue 03 Dec 2013 10:34:15 PM UTC, comment #2: 

Wouldn't it also work to add m4_esyscmd to the set of m4 macros that are disabled by the preamble you feed into autom4te, rather than ignoring stderr?  Particularly true if the m4_esyscmd has more side effects than just printing a version string - you probably don't want to be executing any other commands when doing a trace.

Eric Blake <ericb>
Tue 24 Sep 2013 05:29:59 AM UTC, comment #1: 

Thanks for the suggestion.  Pushed a workaround:
http://git.savannah.gnu.org/cgit/gettext.git/commit/?h=maint&id=1de46f193e6b7beaaadc4bd551c984ef6de2cfc7

By the way any chance of extract-trace and your bootstrap rewrite being merged into gnulib?  We actually wanted to use it in gettext instead of copying around our own func_trace :)

Daiki Ueno <ueno>
Project Administrator
Sun 22 Sep 2013 01:56:56 AM UTC, original submission:  

In configure.ac I use:

  m4_define([M4_VERSION],
    m4_esyscmd([build-aux/git-version-gen .tarball-version]))

When bootstrap runs autopoint (before gnulib-tool has had chance to copy git-version-gen into srcdir), every m4 --trace it runs shows a shell error for:

  sh: build-aux/git-version-gen: No such file or directory

libtoolize 2.4.2 and earlier simply discard the m4 process' stderr during tracing to avoid that problem; git master uses the extract-trace helper script shared by my gnulib bootstrap rewrite, and is more sophisticated by redirecting stderr to stdout, so that the output massaging SED script can operate on the error messages before scanning the results if necessary.

I worked around this by adding a pre-autopoint bootstrap function to copy git-version-gen into place to avoid the error message.

Gary V. Vaughan <gary>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

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 ericb (Posted a comment)
  • -email is unavailable- added by ueno (Posted a comment)
  • -email is unavailable- added by gary (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.

    Only logged-in users can vote.

     

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-11-27 haible CategoryNone Programmer tools
        Assigned toNone ueno
    2013-09-24 ueno StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top


    Powered by Savane 3.9