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
Item Group:  None Status:  Wont Fix
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

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.macports.org/builders/buildports-mtln-x86_64/builds/12043/steps/compile/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>

 

(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 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)
  •  

    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.

     

    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.13-f8d8.
    Corresponding source code