python-related: switch from py*-pil to path-based dependency (openmaintainer)

[Ionic (Mihai Moldovan)]

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	perry@	accepted	on mention it's openmaintainer, r122193
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	committed in r122242
py-astlib	aronnax@	accepted	made openmaintainer, r122243
py-opengl	jmr@	done	committed in r121942
py-reportlab	stromnov@	done	committed in r121845, ticket #44249
py-spyder	eborisch@	accepted	r122243
py-spyder-devel	eborisch@	accepted	r122243

[Ionic (Mihai Moldovan)]

• asymptote (mojca): accepted

[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.

[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.

[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.

[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.)

[Ionic (Mihai Moldovan)]

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

[petrrr]

BTW, why do you not use always there more compact path:${python.pkgd}/PIL version?

[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...)

[petrrr]

Okay, sorry! Have not looking into the details

[maehne (Torsten Maehne)]

Cc Me!

[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

[dbevans (David B. Evans)]

gramps, gramps3 fixed in r121866, r121867. gramps is py-Pillow only due to need for python 3 support.

[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.

[petrrr]

Please move the py-xhtml2pdf update to a separate ticket and add reference to this one.

[mojca (Mojca Miklavec)]

asymptote committed in r121918

[Ionic (Mihai Moldovan)]

py-xhtml2pdf will be handled in #44309. Can be removed from this ticket. Patchset will be updated soon.

[Ionic (Mihai Moldovan)]

comix

perry@

accepted

on mention it's openmaintainer

[Ionic (Mihai Moldovan)]

py-opengl

jmr@

done

committed r121942

[Ionic (Mihai Moldovan)]

Switch from py*-pil to path:-based dependency satisfiable by both py*-pil and py*-Pillow. Fix issue reported by ipe-tools maintainer. Add Pillow compat patches to comix.

[Ionic (Mihai Moldovan)]

Seems like everyone either consented or committed themselves.

I'd go ahead and commit this updated bulk patch.

[petrrr]

comix done in r122193. I guess the port would deserve further maintenance. It should either support multiple python versions via variants or just switch to python27. It installs but some run time warnings appear. Not further tested.

[petrrr]

ipe-tools done in r122242.

I have rewritten the python related part to avoid code replication. It is a sort of python app recipe and inspired by the Fortran recipe and the Python portgroup. No revision should be needed here as the installed files remain unchanged.

[petrrr]

ports py-spyder-devel py-spyder py-astlib done in r122243.

[Ionic (Mihai Moldovan)]

Oh, the ipe-tools commit looks very interesting. It's generating python variants on the fly. Nice work!

[petrrr]

Replying to ionic@…:

Oh, the ipe-tools commit looks very interesting. It's generating python variants on the fly. Nice work!

It's an attempt for a pattern, which could be used either in an portgroup or at least an recipe. The portgroup needs to deal with delayed evaluation and probable requires some more work.