Opened 12 years ago

Closed 12 years ago

#36409 closed defect (fixed)

wml @2.0.11, Revision 2 Build error on Mountain Lion [PATCH]

Reported by: hahn.seb@… Owned by: cfaerber@…
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: haspatch Cc: neverpanic (Clemens Lang)
Port: wml

Description

On ML, wml doesn't build due to the changed behaviour of sed. I've hacked up a fix which sets LANG to "C" for any problematic sed calls, please check it out.

Attachments (3)

patch-fix_sed.diff (5.8 KB) - added by hahn.seb@… 12 years ago.
Portfile-wml.diff (731 bytes) - added by hahn.seb@… 12 years ago.
Portfile-wml-alternative.diff (605 bytes) - added by reg-macports.org@… 12 years ago.
LANG=C

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by hahn.seb@…

Attachment: patch-fix_sed.diff added

Changed 12 years ago by hahn.seb@…

Attachment: Portfile-wml.diff added

comment:1 Changed 12 years ago by neverpanic (Clemens Lang)

Cc: cal@… added; cfaerber@… removed
Owner: changed from macports-tickets@… to cfaerber@…

Appending "LANG=C" to the build environment in the Portfile seems to work, too. The revision bump isn't necessary, because wml either built successfully, or not at all, so we don't need to force a rebuild.

Last edited 12 years ago by neverpanic (Clemens Lang) (previous) (diff)

Changed 12 years ago by reg-macports.org@…

LANG=C

comment:2 Changed 12 years ago by reg-macports.org@…

It's easier to just add it to the Portfile. As an additional bonus, it avoids problems with other tools in the future.

comment:3 Changed 12 years ago by hahn.seb@…

Cool, I like that patch much better. Thanks!

comment:4 Changed 12 years ago by neverpanic (Clemens Lang)

Keywords: haspatch added
Resolution: fixed
Status: newclosed

Commited Portfile-wml-alternative.diff in r98283.

Note: See TracTickets for help on using tickets.