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 15

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 dbevans (David B. Evans))

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@ accepted -
comix
gramps devans@ rejected will remove dependency on py-pil in an upcoming commit
ipe-tools
py-astlib aronnax@ accepted made openmaintainer
py-opengl
py-reportlab stromnov@ done committed in r121845, ticket #44249
py-spyder-devel
py-spyder
py-xhtml2pdf

Change History (15)

comment:1 Changed 10 years ago by Ionic (Mihai Moldovan)

  • asymptote (mojca): accepted

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)

py-reportlab (stromnov): committed in r121845 (ticket #44249)

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:12 Changed 10 years ago by petrrr

Okay, sorry! Have not looking into the details

comment:13 Changed 10 years ago by maehne (Torsten Maehne)

Cc: Torsten.Maehne@… added

Cc Me!

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-toolsTorsten.Maehne@*** et alaccepted-
py-spydereborisch@accepted-
py-spyder-develeborisch@accepted-

N.B.: I'd update the original report myself, but normal users are not allowed to do this

Last edited 10 years ago by Ionic (Mihai Moldovan) (previous) (diff)

comment:15 Changed 10 years ago by dbevans (David B. Evans)

Description: modified (diff)
Note: See TracTickets for help on using tickets.