Opened 17 years ago

Closed 17 years ago

Last modified 16 years ago

#14104 closed defect (fixed)

pv 1.1.0 upgrade/build failure

Reported by: meissnem@… Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 1.6.0
Keywords: pv upgrade build failure Cc: meissnem@…, aaxin@…, wizard@…
Port:

Description

upgrade pv to 1.1.0 and fix compilation failure on Leopard

Attachments (2)

Portfile.diff (1.1 KB) - added by meissnem@… 17 years ago.
Portfile patch
Portfile.2.diff (979 bytes) - added by meissnem@… 17 years ago.

Download all attachments as: .zip

Change History (7)

Changed 17 years ago by meissnem@…

Attachment: Portfile.diff added

Portfile patch

comment:1 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to ryandesign@…

comment:2 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Thanks for the update! I added the missing platforms variable and the blank line after the PortSystem line in r33610. But I did not commit the update to 1.1.0 because it failed to build on my system (Intel, 10.4.11, Xcode 2.4.1). If I comment out your new post-patch section, the build succeeds. Are you sure the post-patch is necessary for Leopard? If so, we should wrap it in a platform darwin 9 section.

comment:3 Changed 17 years ago by meissnem@…

The post-patch is needed on Leopard -- that's my primary system and I can confirm it.

I got my hands on a 10.4.11 system myself and confirmed what you found, so I'm attaching an updated patch.

As an aside, it sure would be nice to have a 10.3, 10.4, and 10.5 machine available to maintainers for testing... or if not maintainers then at least committers!

Changed 17 years ago by meissnem@…

Attachment: Portfile.2.diff added

comment:4 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Committed in r33614. Thanks!

comment:5 Changed 16 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.