Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vips driver #1119

Open
anshu8858 opened this issue Oct 29, 2021 · 7 comments
Open

Add vips driver #1119

anshu8858 opened this issue Oct 29, 2021 · 7 comments
Assignees

Comments

@anshu8858
Copy link

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

@mstaack
Copy link

mstaack commented Jan 4, 2022

try this, works well for me:

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

@jcupitt
Copy link

jcupitt commented Dec 8, 2023

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

might be useful.

@mstaack
Copy link

mstaack commented Jan 25, 2024

@jcupitt oh great, thanks

@ARehmanMahi
Copy link

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.

@jcupitt
Copy link

jcupitt commented Feb 10, 2024

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

@ARehmanMahi
Copy link

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.

@olivervogel
Copy link
Member

olivervogel commented Feb 12, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants