image icon indicating copy to clipboard operation
image copied to clipboard

Add vips driver

Open anshu8858 opened this issue 4 years ago • 12 comments

Please add vips for driver. It takes less RAM. Stack Overflow PECL

anshu8858 avatar Oct 29 '21 11:10 anshu8858

try this, works well for me:

https://github.com/osiemsiedem/intervention-image-vips-driver

mstaack avatar Jan 04 '22 16:01 mstaack

Hello, I had a quick look at that driver and made an issue with a couple of things I spotted: https://github.com/osiemsiedem/intervention-image-vips-driver/issues/27

might be useful.

jcupitt avatar Dec 08 '23 18:12 jcupitt

@jcupitt oh great, thanks

mstaack avatar Jan 25 '24 12:01 mstaack

I would advise against the osiemsiedem driver. The maintainer doesn't cooperate with intervention devs & doesn't even reply to the other maintainers of the vips libs itself @jcupitt. Thinks intervention is dead yet tries to have an adapter for the "dead" lib.

There have been talks of such adapters for 5 some years. Yet no progress on it. Seems like a lost cause. I'm just a junior php dev, please don't come at me with PRs lol.

Intervention & vips are both amazing projects. Would love to have them working together.

ARehmanMahi avatar Feb 10 '24 08:02 ARehmanMahi

The osiemsiedem driver is MIT licenced, so it could be forked and maintained by intervention, if necessary. I'm not offering to do the work myself, of course ahem

jcupitt avatar Feb 10 '24 10:02 jcupitt

The osiemsiedem driver is MIT licenced, so it could be forked and maintained by intervention, if necessary. I'm not offering to do the work myself, of course ahem

That is why I'm going to remove Intervention and directly use vips-php, not that difficult to use standalone.

ARehmanMahi avatar Feb 10 '24 10:02 ARehmanMahi

That is why I'm going to remove Intervention and directly use vips-php, not that difficult to use standalone.

I agree. Intervention Image is basically just a layer of comfort and convenience. If one intend to use only VIPS (or any other processor) anyway, there is no reason not to the this processor directly.

olivervogel avatar Feb 12 '24 13:02 olivervogel