buglibiconv - Bugs: bug #36748, patch to fix ./autogen.sh under...

 
 

bug #36748: patch to fix ./autogen.sh under MinGW shell

Submitter:  None
Submitted:  Fri 29 Jun 2012 10:03:17 AM UTC
   
 
Category:  Build Severity:  3 - Normal
Item Group:  None Status:  Fix Released
Privacy:  Public Assigned to:  haible
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 01 Jul 2012 12:39:27 AM UTC, comment #1: 


> under MinGW, only stdout+stderr do.


Correct.

> This has the side effect of making debugging under gdb somewhat easier, too.


Good point.

Thanks for the suggestion. Implemented at
http://git.savannah.gnu.org/gitweb/?p=libiconv.git;a=commitdiff;h=1f574c8a69e51e4d36698e803be6e510c9d45637
http://git.savannah.gnu.org/gitweb/?p=libiconv.git;a=commitdiff;h=5977e1b0c6e2c9ad9fa019698ff28dc11ef6d076

Forgive me, I couldn't use your patch directly. For doing that
I would have needed
  - your name,
  - a copyright assignment from you to the FSF.

Bruno Haible <haible>
Group administrator
Fri 29 Jun 2012 10:03:17 AM UTC, original submission:  

Running the autogen.sh script under MinGW's shell fails, because genaliases tools quite reasonably expect POSIX behavior and redirect 3rd and 4th file descriptors into generated files. This unfortunately doesn't work under MinGW, only stdout+stderr do.

This patch changes the behavior to pass the names of files to generate as command line arguments and updates Makefile.devel accordingly. For compatibility, in case something else depends on these tools, the old behavior is preserved if you don't pass the files as arguments.

This has the side effect of making debugging under gdb somewhat easier, too.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #26118:  iconv-mingw.patch added by None (9KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by haible (Posted a comment)
  •  

    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 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-01-04 haible StatusFixed Fix Released
    2016-11-23 haible CategoryNone Build
    2012-07-01 haible StatusNone Fixed
        Assigned toNone haible
        Open/ClosedOpen Closed
    2012-06-29 None Attached File- Added iconv-mingw.patch, #26118

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code