Opened 10 years ago
Last modified 10 years ago
#44284 closed enhancement
python-related: switch from py*-pil to path-based dependency (openmaintainer) — at Version 21
Reported by: | Ionic (Mihai Moldovan) | Owned by: | petr@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | petr@…, Torsten.Maehne@… |
Port: | gramps asymptote comix ipe-tools py-opengl py-reportlab py-spyder-devel py-spyder py-astlib |
Description (last modified by mojca (Mojca Miklavec))
In accordance with communication on the MP users mailing list, I have changed py*-pil
dependencies to path:
-based ones satisfiable by both py*-pil
and py*-Pillow
.
This patchset is tracking and targeted at maintained openmaintainer
ports.
(Not CC'ing maintainers, as I have sent them eMails individually and people don't need to be bothered for other stuff but their own ports.)
asymptote | mojca@ | done | committed in r121918 |
comix
| |||
gramps | devans@ | done | dependency on py-pil replaced with py-Pillow in upgrade to version 4.0 r121866 |
gramps3 | devans@ | done | r121867 |
ipe-tools | Torsten.Maehne@ et al. | accepted | - |
py-astlib | aronnax@ | accepted | made openmaintainer |
py-opengl
| |||
py-reportlab | stromnov@ | done | committed in r121845, ticket #44249 |
py-spyder | eborisch@ | accepted | - |
py-spyder-devel | eborisch@ | accepted | - |
py-xhtml2pdf
|
Change History (21)
comment:1 Changed 10 years ago by Ionic (Mihai Moldovan)
comment:2 Changed 10 years ago by petrrr
Description: | modified (diff) |
---|
comment:3 Changed 10 years ago by petrrr
Owner: | changed from macports-tickets@… to petr@… |
---|
comment:4 Changed 10 years ago by petrrr
Is there a revision bump needed here? The content of the package does not change, and once the dependency is in place it does not change either. So revision bump has no real effect on the files installed.
comment:5 Changed 10 years ago by Ionic (Mihai Moldovan)
I assumed that the registry is saving dependencies given at install-time as well. Seems to have been wrong, only ports are recorded. In that case, yes, a revbump is not needed.
I'll update the patches.
comment:6 Changed 10 years ago by petrrr
Okay, I was not considering the registry. Not sure if something changes there. Maybe ask at the MP developer list for details or what impact the change has on the registry status.
comment:7 Changed 10 years ago by Ionic (Mihai Moldovan)
I did ask neverpanic, he should know this stuff. :) Waiting for a reply, but I'm pretty sure the registry will be unimpressed, as dependencies are not added literally, only port-based (and we don't really change the dependencies for already installed ports anyway.)
comment:8 Changed 10 years ago by Ionic (Mihai Moldovan)
comment:9 Changed 10 years ago by petrrr
BTW, why do you not use always there more compact path:${python.pkgd}/PIL
version?
comment:10 Changed 10 years ago by petrrr
Description: | modified (diff) |
---|
comment:11 Changed 10 years ago by Ionic (Mihai Moldovan)
I can't. python.pkgd
is only defined by the python PortGroup
. Trust me, you don't want to use the python PortGroup
for non-Python-apps... (it sets build.cmd
, use_configure no
and other stuff...)
comment:14 Changed 10 years ago by Ionic (Mihai Moldovan)
Updated patch to fix the issue reported by ipe-tools
maintainer, remove revbumps and py-reportlab
.
ipe-tools | Torsten.Maehne@*** et al | accepted | - |
py-spyder | eborisch@ | accepted | - |
py-spyder-devel | eborisch@ | accepted | - |
N.B.: I'd update the original report myself, but normal users are not allowed to do this
comment:15 Changed 10 years ago by dbevans (David B. Evans)
Description: | modified (diff) |
---|
comment:16 Changed 10 years ago by petrrr
Description: | modified (diff) |
---|
comment:17 Changed 10 years ago by dbevans (David B. Evans)
Description: | modified (diff) |
---|
comment:18 Changed 10 years ago by dbevans (David B. Evans)
Description: | modified (diff) |
---|
comment:19 Changed 10 years ago by Ionic (Mihai Moldovan)
This patchset update fixes comix
and py-opengl
by adding a Pillow
compat patch and py-xhtml2pdf
by updating to the newest version 0.0.6, which is compatible with both PIL
and Pillow
.
comment:20 Changed 10 years ago by petrrr
Please move the py-xhtml2pdf
update to a separate ticket and add reference to this one.
comment:21 Changed 10 years ago by mojca (Mojca Miklavec)
Description: | modified (diff) |
---|
asymptote
committed in r121918
asymptote
(mojca
): accepted