bugGNU roff - Bugs: bug #49649, Unable to build groff-1.22.3 on...

 
 

bug #49649: Unable to build groff-1.22.3 on MacOS X

Submitter:  None
Submitted:  Sat 19 Nov 2016 03:14:11 AM UTC
   
 
Category:  General Severity:  3 - Normal
Item Group:  Build/Installation Status:  Fixed
Privacy:  Public Assigned to:  bgarrigues
Open/Closed:  Closed Planned Release:  1.22.4
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 15 Jun 2022 06:36:23 PM UTC, comment #6: 


commit 07b73a459eb715383e0f995bedd06ffb78a6d8b7
Author: Bertrand Garrigues <bertrand.garrigues@laposte.net>
Date:   Sat Sep 2 00:03:57 2017 +0200

    Fix build on MacOS X

    * Makefile.am: define `SED' command with LANG and LC_ALL set to C.
    * contrib/mm/mm.am: use it.
    * doc/doc.am: Ditto.

    Fix bug https://savannah.gnu.org/bugs/?49649


G. Branden Robinson <gbranden>
Group administrator
Fri 01 Sep 2017 10:06:07 PM UTC, comment #5: 

Patch commited (with a slight difference: I've put the definition of SED in the top-level Makefile.am file).  I did not find a good way to set LANG and LC_ALL globally.

Bertrand Garrigues <bgarrigues>
Group administrator
Sun 20 Nov 2016 07:26:17 AM UTC, comment #4: 

Thanks.  I still wonder whether LANG and LC_ALL can be set globally for the whole build and installation process.  I guess that other tools might benefit also (even if not currently necessary).

Werner LEMBERG <wl>
Group administrator
Sat 19 Nov 2016 11:42:17 AM UTC, comment #3: 

Revised patch against repository source and build procedure submitted.

Anonymous
Sat 19 Nov 2016 08:46:36 AM UTC, comment #2: 

The problem was encountered on a version of groff before the Automake migration, and the patched file no longer exist.  Could you please try to build the latest sources from the git repository?

Bertrand Garrigues <bgarrigues>
Group administrator
Sat 19 Nov 2016 06:33:12 AM UTC, comment #1: 

Thanks for the patch.  However, I think it can be simplified a lot by setting `LANG' and `LC_ALL' globally while building groff.

Bertrand, please have a look.

Werner LEMBERG <wl>
Group administrator
Sat 19 Nov 2016 03:14:11 AM UTC, original submission:  

The attached patch fixes build failure with the distributed source MacOS X.  sed is picky about LANG and LC_ALL settings when handling files with non-UTF-8 characters in it.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #39019:  groff-sed.diff added by None (2KiB - application/octet-stream - reworked minimal patch against a repository build as requested)
file #39016:  groff-1.22.3-build.patch added by None (5KiB - application/octet-stream - Source patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by bgarrigues (Posted a comment)
  • -email is unavailable- added by wl (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 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-06-15 gbranden CategoryNone General
    2017-09-01 bgarrigues StatusNone Fixed
        Open/ClosedOpen Closed
        Planned ReleaseNone 1.22.4
    2017-09-01 bgarrigues Item GroupIncorrect behaviour Build/Installation
    2016-11-19 None Attached File- Added groff-sed.diff, #39019
    2016-11-19 wl Assigned toNone bgarrigues
    2016-11-19 None Attached File- Added groff-1.22.3-build.patch, #39016

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code