Opened 10 years ago
Closed 10 years ago
#44307 closed defect (fixed)
py-pyepl: remove PIL dependency
Reported by: | Ionic (Mihai Moldovan) | Owned by: | petrrr |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | |
Port: | py-pyepl |
Description
py-pyepl
doesn't make use of PIL at all (i.e., it's not importing it.)
There is a comment stating
#import PIL.Image # catch error here!
but that's hardly sufficient to declare a dependency.
On the other hand, py-opengl
, a dependency of py-pyepl
does indeed need PIL/Pillow. py-opengl
already correctly depends on PIL
/Pillow
(cf. #44284.)
Remove the PIL
dependency from py-pyepl
.
x-linking #44281
Attachments (1)
Change History (9)
comment:1 follow-up: 2 Changed 10 years ago by petrrr
comment:2 Changed 10 years ago by Ionic (Mihai Moldovan)
comment:3 Changed 10 years ago by petrrr
Unrelated! Probably the port is dead for years. But the registry change would probably deserve a rev.bump, and the buildbots will complain.
comment:4 Changed 10 years ago by Ionic (Mihai Moldovan)
Nah, there's no revbump required. The registry will be fine. It just records resolved dependencies, so stuff will stay the same for users who already have the ports installed.
Changed 10 years ago by Ionic (Mihai Moldovan)
Attachment: | py-pyepl-remove-pil-dep.patch added |
---|
Remove py*-pil
dependency, not used by software. Revbump.
comment:5 Changed 10 years ago by Ionic (Mihai Moldovan)
Revbumped port to account for dependency change. (Even though it's still broken, whatever.)
comment:6 Changed 10 years ago by petrrr
comment:7 Changed 10 years ago by petrrr
Owner: | changed from macports-tickets@… to petr@… |
---|---|
Status: | new → assigned |
comment:8 Changed 10 years ago by petrrr
Cc: | petr@… removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
For me the port fails to build, and I doubt anyone really use it.