jcupitt / docker-builds

Various sample dockerfiles
91 stars 31 forks source link

Upgrade webp, libvips + include poppler #6

Closed leaanthony closed 3 years ago

leaanthony commented 3 years ago

PDF support and updates to the library versions.

jcupitt commented 3 years ago

Hiya, looks good, though the danger of poppler is that it's GPL, so if you link it into your project, your whole project becomes GPL too. This makes it unsuitable for some commercial applications.

git master libvips has a fixed-up PDFium loader. If you need PDF support, it might be worth looking at that --- PDFium has a very liberal licence.

leaanthony commented 3 years ago

Do you have any suggestions on how to use that instead of poppler in the given Dockerfile?

jcupitt commented 3 years ago

There are some notes in the code:

https://github.com/libvips/libvips/blob/master/libvips/foreign/pdfiumload.c#L50-L79

Or in the issues which tracked fixing PDFium build.

jcupitt commented 3 years ago

... I added those notes to the PDFium section in the main README.

jcupitt commented 3 years ago

Sorry @leaanthony, this slipped my mind :(