Require dompdf through Composer

[philsturgeon]

Creating a wrapper for L4 is a nice idea, but you really shouldn't be modifying the original package which you are wrapping!

Just lits it as a dependency in your package.

https://github.com/dompdf/dompdf

It's right here, so its easy to install.

[philsturgeon]

Sorry, you are. Ignore this.

[barryvdh]

Maybe you meant this package? https://github.com/thujohn/pdf-l4

[philsturgeon]

No I just looked at the PDF class and responded too quickly. I'm spring-loaded on "shout at people making L4 specific code which shouldn't be L4 specific" and "wrappers should be wrappers not forks", and pulled the trigger to quickly.

Good work :)

Sent from my iPhone

On Jul 31, 2013, at 3:15 AM, "Barry vd. Heuvel" notifications@github.com wrote:

Maybe you meant this package? https://github.com/thujohn/pdf-l4

—
Reply to this email directly or view it on GitHub.