Opened 12 years ago

Closed 12 years ago

#38341 closed update (fixed)

Update Pythia + fix conflict with lib event

Reported by: cjones051073 (Chris Jones) Owned by: mattiafrancescomoro@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: cooljeanius (Eric Gallager)
Port: pythia

Description

The attached Portfile diff updates the pythia port to the latest version, 8.175, and also fixes the conflict with libevent by installing the headers to /opt/local/include/pythia. Note that the patch file is not needed with this update and can also be removed.

This change only affects one other port, root, which has a variant that uses pythia. The fix is trivial and I will be submitting another update that fixes this, assuming this one is committed.

I need to remove the conflict with libevent, as I am working on another update to ROOT that will use a new port, xrootd, that would benefit from using libevent, thus I need libevent and pythia to not conflict.

Chris

Attachments (1)

pythia.Portfile.diff (2.0 KB) - added by cjones051073 (Chris Jones) 12 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by cjones051073 (Chris Jones)

ROOT update submitted as ticket #38344

Changed 12 years ago by cjones051073 (Chris Jones)

Attachment: pythia.Portfile.diff added

comment:2 Changed 12 years ago by cjones051073 (Chris Jones)

I've updated the diff to fix two other issues

  1. The port would opportunistically use zlib and boost, so I have added dependencies against these.
  1. The xmldoc data was not being installed. This is now installed under ${prefix}/share/doc/pythia/

cheers Chris

comment:3 Changed 12 years ago by andre.david@…

Works for me, in the context of #38344 together with this.

comment:4 Changed 12 years ago by mf2k (Frank Schima)

Cc: gmail.com:mattiafrancescomoro removed
Owner: changed from macports-tickets@… to mattiafrancescomoro@…
Version: 2.1.3

Trac requires regularly formatted email addresses.

comment:5 Changed 12 years ago by cjones051073 (Chris Jones)

Thanks for the corrections ;)

So, any thoughts on this. Can you foresee any issues I have missed ? The only minor one I can come up with is if anyone is building there own applications that link against pythia, they will have to update their build options with the new path for the includes. Trivial fix, as I did for the ROOT port, so not a big deal and worth I think the improvement in having the includes better isolated in their own sub-directory under ${prefix}/include.

I would like to get the ROOT update for this rolled ut, so if this could be committed soon, that would be great.

Chris

comment:6 Changed 12 years ago by mf2k (Frank Schima)

This looks good to me. We just need to wait for maintainer approval or timeout in 3 days since yesterday.

comment:7 Changed 12 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:8 Changed 12 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: newclosed

Committed in r104094, with minor tweaks.

Note: See TracTickets for help on using tickets.