py-impressive: update to 0.10.5 and switch to hybrid `PIL`/`Pillow` dependency

[Ionic (Mihai Moldovan)]

py-impressive has been updated to version 0.10.5 which includes changes to make it Pillow-compatible by upstream.

Also switch to the new hybrid PIL/Pillow dependency style as suggested in #44281.

[Ionic (Mihai Moldovan)]

Update py-impressive to version 0.10.5 and change py*-pil dependency to new PIL/Pillow hybrid one.

[petrrr]

Just a detail, the revision 0, is the default. So it is not to be specified.

[Ionic (Mihai Moldovan)]

I know it is, but I personally like adding revision 0 anyway, because it makes revbumping easier should the revision need to be changed. (I.e., I won't have to write "revision" again after removing it.)

That's just a personal thing.

[petrrr]

BTW: Are other python versions, other then 2.6, now supported?

[Ionic (Mihai Moldovan)]

Yes, python 2.5 to 2.7 is supported. At least officially by the software author. I didn't update that though because I had no time to test if it's really working.

(py-impressive sounds awesome though)

[petrrr]

There might be an issue with port:pdftk on some systems. But that's a different story.

[petrrr]

Thanks! Committed in r121929.

[Ionic (Mihai Moldovan)]

Oh, I remember why I didn't look into the 2.7 issue:

Quote from portfile:

[...]
    python.link_binaries no
    # this is bad, a py27-impressive would conflict with py26-impressive
    # make non-conflicting or change name to just 'impressive'
    destroot {
[...]

Does it work nowadays?

[Ionic (Mihai Moldovan)]

Replying to petr@…:

Thanks! Committed in r121929.

Thanks for committing!

I added the py27 subport as well.

No, you actually didn't, but that's probably for the best (see my comment above.)

[petrrr]

I already noticed and change the comment above and now the commit message as well.

Yes, I will look at py27 subport later and so leave the ticket open for the moment. It looks like this is actually an application not a module so probably should be renamed.

[Ionic (Mihai Moldovan)]

Replying to petr@…:

Yes, I will look at py27 subport later and so leave the ticket open for the moment. It looks like this is actually an application not a module so probably should be renamed.

Is python/py-* reserved for modules only? I suspect there are lot of standalone applications in this category with that naming scheme.

[petrrr]

Cc Me!

[petrrr]

Renaming py-impressive (python package) to impressive (python app) is handled in #44538 and committed in r123616. Closing!