Opened 11 years ago
Closed 11 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)
Change History (9)
comment:1 Changed 11 years ago by cjones051073 (Chris Jones)
Changed 11 years ago by cjones051073 (Chris Jones)
| Attachment: | pythia.Portfile.diff added |
|---|
comment:2 Changed 11 years ago by cjones051073 (Chris Jones)
I've updated the diff to fix two other issues
- The port would opportunistically use zlib and boost, so I have added dependencies against these.
- The xmldoc data was not being installed. This is now installed under ${prefix}/share/doc/pythia/
cheers Chris
comment:3 Changed 11 years ago by andre.david@…
Works for me, in the context of #38344 together with this.
comment:4 Changed 11 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 11 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 11 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:8 Changed 11 years ago by larryv (Lawrence Velázquez)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Committed in r104094, with minor tweaks.
ROOT update submitted as ticket #38344