#44588 closed defect (fixed)
libpurple: add perl dependency
| Reported by: | Ionic (Mihai Moldovan) | Owned by: | dbevans (David B. Evans) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | haspatch | Cc: | nerdling (Jeremy Lavergne) |
| Port: | libpurple |
Description
Both libpurple and pidgin can be built with perl support, providing a means of scripting of the protocol/networking or UI portion (c.f. the pidgin developer documentation).
Perl support is currently enabled for pidgin, but not libpurple.
The configure script bails out if perl was not explicitly disabled but unavailable, leading to a build error in trace mode.
devans is in favor of unconditionally enabling Perl support. I am backing this decision.
This patch moves perl5.16 to a common dependency of both pidgin and libpurple.
Revbump on libpurple to account for that change, even for users not using trace mode, as the possibility of users only installing libpurple but not pidgin exists and those might be missing perl5.16.
Attachments (1)
Change History (6)
comment:1 Changed 10 years ago by dbevans (David B. Evans)
| Cc: | devans@… removed |
|---|---|
| Keywords: | tracemode removed |
| Owner: | changed from macports-tickets@… to devans@… |
| Status: | new → assigned |
comment:2 Changed 10 years ago by Ionic (Mihai Moldovan)
Testing it right now. :)
I needed a bug # first to put it as a reminder for the revbump comment.
Changed 10 years ago by Ionic (Mihai Moldovan)
| Attachment: | libpurple-perl.diff added |
|---|
Move perl5.16 dependency to the common deps for both pidgin and libpurple. Revbump only libpurple.
comment:3 Changed 10 years ago by Ionic (Mihai Moldovan)
There it is. (I wonder why trac doesn't send emails on attachment changes...)
comment:4 Changed 10 years ago by dbevans (David B. Evans)
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Committed in r123539. Thanks for catching this.
If you add the patch, I'll commit it.