Opened 13 years ago
Closed 10 years ago
#34618 closed enhancement (fixed)
swig: add swig-php53 and swig-php54 subports
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | michaelld (Michael Dickens) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.1.1 |
Keywords: | Cc: | jyrkiwahlstedt, cooljeanius (Eric Gallager) | |
Port: | swig |
Description
Currently there is a swig-php5 subport, which depends on path:bin/php:php5. The php5 and php5-devel ports are going away, and ${prefix}/bin/php will no longer be installed by a port. php5 and php5-devel are to be replaced by the new php53 and php54 subports, which install ${prefix}/bin/php53 and ${prefix}/bin/php54 respectively. I'm not sure how swig works, but I assume we should have new swig-php53 and swig-php54 ports, which let you build something for use with php53 and php54 respectively.
Change History (5)
comment:1 Changed 11 years ago by cooljeanius (Eric Gallager)
Cc: | egall@… added |
---|
comment:2 Changed 10 years ago by michaelld (Michael Dickens)
Owner: | changed from kevin@… to michaelld@… |
---|
comment:3 Changed 10 years ago by michaelld (Michael Dickens)
The swig PHP subports need to be worked on: yes! I'll look into these, but I agree that it's probably adding swig-php5X and then making them conflict with each other; pick one and go with it. Could do this similarly for the gcj ones (gcj4X, to match +gcc4X).
comment:4 Changed 10 years ago by michaelld (Michael Dickens)
So here's what I'm proposing doing:
1) Set swig-php5 as replaced_by swig-php.
2) Add +php5X variants to swig-php (similar to +gcc4X for gcj).
The above seems to make sense to me; and, they install correctly in my testing -- but, I do not otherwise use these ports so I cannot comment as to how well they work once installed. I'll push these changes once I have confirmation that they make sense, or a description of what -does- make sense.
comment:5 Changed 10 years ago by michaelld (Michael Dickens)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Should be fixed in r120853. This revision works for me and should be OS version agnostic, so I'm closing this ticket as fixed.
Cc Me!