Opened 9 years ago

Closed 9 years ago

#47196 closed defect (fixed)

lparse @1.1.2_0: sundry fixes

Reported by: larryv (Lawrence Velázquez) Owned by: nerdling (Jeremy Lavergne)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: lparse

Description

The attached patches get lparse in working order (and fix #40940 and #41871). May I commit them?

Attachments (6)

0001-lparse-Fix-never-failing-makefile-recipes.patch (2.2 KB) - added by larryv (Lawrence Velázquez) 9 years ago.
Fix never-failing makefile recipes
0002-lparse-Use-the-right-compiler.patch (743 bytes) - added by larryv (Lawrence Velázquez) 9 years ago.
Use the right compiler
0003-lparse-Fix-detection-of-Bison-version.patch (2.7 KB) - added by larryv (Lawrence Velázquez) 9 years ago.
Fix detection of Bison version
0004-lparse-Avoid-namespace-collisions-41871.patch (14.0 KB) - added by larryv (Lawrence Velázquez) 9 years ago.
Avoid namespace collisions (fixes #41871)
0005-lparse-Make-configure-respect-CXXFLAGS-and-LDFLAGS.patch (2.1 KB) - added by larryv (Lawrence Velázquez) 9 years ago.
Make configure respect CXXFLAGS and LDFLAGS
0006-lparse-Fix-universal-build-40940.patch (829 bytes) - added by larryv (Lawrence Velázquez) 9 years ago.
Fix universal build (fixes #40940)

Download all attachments as: .zip

Change History (11)

Changed 9 years ago by larryv (Lawrence Velázquez)

Fix never-failing makefile recipes

Changed 9 years ago by larryv (Lawrence Velázquez)

Use the right compiler

Changed 9 years ago by larryv (Lawrence Velázquez)

Fix detection of Bison version

Changed 9 years ago by larryv (Lawrence Velázquez)

Avoid namespace collisions (fixes #41871)

Changed 9 years ago by larryv (Lawrence Velázquez)

Make configure respect CXXFLAGS and LDFLAGS

Changed 9 years ago by larryv (Lawrence Velázquez)

Fix universal build (fixes #40940)

comment:1 Changed 9 years ago by nerdling (Jeremy Lavergne)

Looks good; you'll update the other tickets too?

comment:2 Changed 9 years ago by nerdling (Jeremy Lavergne)

D'ooh, just saw it. For patch 5: O3 was previously forced by ./configure.

Last edited 9 years ago by nerdling (Jeremy Lavergne) (previous) (diff)

comment:3 in reply to:  1 Changed 9 years ago by larryv (Lawrence Velázquez)

Replying to snc@…:

Looks good; you'll update the other tickets too?

Yeah, I’ll take care of them.

comment:4 in reply to:  2 Changed 9 years ago by larryv (Lawrence Velázquez)

Replying to snc@…:

D'ooh, just saw it. For patch 5: O3 was previously forced by ./configure.

Right; I was just maintaining the current behavior. I don’t feel particularly strongly about it, but it’s nice to let the portfile dictate the optimization level.

comment:5 Changed 9 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.