Skip to content

Decreased Curl dependency#854

Open
zizzo81 wants to merge 1 commit intotecnickcom:mainfrom
zizzo81:less-curl-dependency
Open

Decreased Curl dependency#854
zizzo81 wants to merge 1 commit intotecnickcom:mainfrom
zizzo81:less-curl-dependency

Conversation

@zizzo81
Copy link
Copy Markdown

@zizzo81 zizzo81 commented Feb 14, 2026

This modification makes possible to use TCPDF without forcing user to have the CURL extension enabled, provided you accept the limitation of not using any CURL-dependent functions.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Copy Markdown
Contributor

@williamdes williamdes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay, thanks

@nicolaasuni
Copy link
Copy Markdown
Member

Thank you very much, @zizzo81, for the careful work to make TCPDF usable in environments without the cURL extension - converting the class constants that contain cURL constants into lazy accessor functions is a clean approach, and the companion issue #841 makes the problem very clear.

TCPDF 6 is now deprecated and no further changes will be merged into this repository. Please see the pinned issue Thank You - and the Future of TCPDF for the full explanation and the path forward.

In tc-lib-pdf, cURL is an optional dependency and the library degrades gracefully when it is not available. If you encounter a situation where that is not the case, feel free to open a PR or issue there.

I am leaving this PR open in case you or anyone else wishes to continue the discussion, but it is unlikely that this PR will ever be merged.

Thank you again for your patience and understanding.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants