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

Submitted by:  Gary V. Vaughan <gary>
Submitted on:  Sun 22 Sep 2013 01:56:56 AM UTC  
 
Category: Programmer toolsSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Daiki Ueno <ueno>
Open/Closed: Closed

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 AdministratorIn charge of this item.
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>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by haible (Updated the item)
  • -unavailable- added by ericb (Posted a comment)
  • -unavailable- added by ueno (Posted a comment)
  • -unavailable- added by gary (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 27 Nov 2016 01:49:43 AM UTChaibleCategoryNone=>Programmer tools
      Assigned toNone=>ueno
    Tue 24 Sep 2013 05:29:59 AM UTCuenoStatusNone=>Fixed
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1