Opened 17 years ago

Closed 17 years ago

Last modified 15 years ago

#11604 closed defect (fixed)

BUG: lzo2-2.02 headers are installed in the incorrect directory

Reported by: yunzheng.hu@… Owned by: pipping@…
Priority: Normal Milestone:
Component: ports Version: 1.3.2
Keywords: gnutls openvpn2 lzo lzo2 Cc: yunzheng.hu@…, boeyms@…
Port:

Description

The current Portfile for lzo2 installs it's headers to ${prefix}/include/lzo2 but this should be ${prefix}/include/lzo This is where it is expected to be.

When this is fixed it can also fix the following ports:

  • gnutls (see #10645 why it fails to build, will need to add dependency on port:lzo2 to fix this)
  • openvpn2 (already builds correctly, but we can then remove a unneccesary reinplace)

mplayer can also be updated to use lzo2 i think as lzo(1) is kinda outdated.

Attachments (2)

lzo2-Portfile.diff (1.2 KB) - added by yunzheng.hu@… 17 years ago.
lzo2 Portfile diff
openvpn2-Portfile.diff (414 bytes) - added by yunzheng.hu@… 17 years ago.
updated openvpn2 Portfile with the removed reinplace

Download all attachments as: .zip

Change History (7)

Changed 17 years ago by yunzheng.hu@…

Attachment: lzo2-Portfile.diff added

lzo2 Portfile diff

Changed 17 years ago by yunzheng.hu@…

Attachment: openvpn2-Portfile.diff added

updated openvpn2 Portfile with the removed reinplace

comment:1 Changed 17 years ago by pipping@…

Milestone: Port Bugs

comment:2 Changed 17 years ago by markd@…

See #10645 for more info.

comment:3 Changed 17 years ago by markd@…

Oops, I didn't see there was already a reference to 10645.

comment:4 Changed 17 years ago by boeyms@…

Cc: boeyms@… added
Resolution: fixed
Status: newclosed

Patch to lzo2 committed in r25465; I finally realised that the lzo port puts its headers into ${prefix}/include rather than ${prefix}/include/lzo as lzo2 does, so everything works just fine. Thanks!

comment:5 Changed 15 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.