bugSwarm - Bugs: bug #41491, Build fails with BSD sed

 
 

bug #41491: Build fails with BSD sed

Submitter:  Ryan Carsten Schmidt <ryandesign>
Submitted:  Fri 07 Feb 2014 04:33:33 AM UTC
Votes: 1
 
Category:  Swarm Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Wont Fix Privacy:  Public
Assigned to:  None Open/Closed:  Open
* Mandatory Fields

Post a Comment

Add a New Comment Rich Markup
   

Discussion

Wed 12 Feb 2014 09:14:10 AM UTC, comment #2: 

Thanks for confirming the issue. I did add a GNU sed dependency for swarm in MacPorts so we're all set for now.

I see that the GNU sed requirement is documented in the file macosx/README.MacOSX. I had not looked for this file before. Even if I had, I might've skipped reading it when realizing that it is over 9 years old.

Is there a chance you could add a check to the configure script for this? It would be a lot more helpful to have the configure script exit with an error that says something like "GNU sed is required but only BSD sed was found" than to have configure succeed and the build fail with a confusing errors like "collections.xm:6: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'OffsetOutOfRange'" which is what we get currently.

Ryan Carsten Schmidt <ryandesign>
Sat 08 Feb 2014 06:12:14 PM UTC, comment #1: 

Yes this is true. Swarm has always needed GNU sed. If building for macports then you should add GNU sed as a dependency for building.

Scott Christley <schristley>
Group administrator
Fri 07 Feb 2014 04:33:33 AM UTC, original submission:  

Building swarm 2.4.1 fails when BSD sed is found by the configure script, e.g. on Mac OS X. Here is a log from the MacPorts automated build system which shows the problem:

https://build.mac ... ompile/logs/stdio

Installing GNU sed allows the build to succeed. I think one of the regular expressions in src/defobj/make-h2x is using a GNU-sed-specific syntax, but there are a lot of regular expressions and I have not analyzed them.

Ryan Carsten Schmidt <ryandesign>

 

Attached Files

This item currently has no attached files.

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

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

Carbon-Copy List
  • -email is unavailable- added by egallager (Voted in favor of this item)
  • -email is unavailable- added by schristley (Posted a comment)
  • -email is unavailable- added by ryandesign (Submitted the item)
  •  

    Votes

    There is 1 vote 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.

     

    History

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-02-14 egallager Carbon-Copy- Added egallager
    2014-02-08 schristley StatusNone Wont Fix

    Back to the top

    Powered by Savane 3.16.
    Corresponding source code