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

Bundle pdftk with the gem #41

Open
SirRawlins opened this issue Aug 8, 2014 · 1 comment
Open

Bundle pdftk with the gem #41

SirRawlins opened this issue Aug 8, 2014 · 1 comment

Comments

@SirRawlins
Copy link

Hi All,

I'd like to propose that we bundle pdftk source code in with the gem to drop the requirement that it be pre-installed on the system. I think this would be beneficial for several reasons:

  1. Less work to get up and running, no need to install the dependancy separately in your environment.
  2. Versioning of the dependancy is controlled and defined within your app by the gem version, so no risk of the package being updated outside of your codebase and breaking things.
  3. Deployment to infrastructure which doesn't allow access to the underlying OS (such as Heroku) would be simplified.

There is already a fairly nice implementation for this kind of thing in the [pdftk-heroku](https://github.com/charliemaffitt/pdftk-heroku) gem. But I think there is a really solid use case for pulling this kind of functionality into active_pdftk natively.

If people are happy with this as a concept I'm happy to put the work in to a pull-request for it.

Thanks,

Robert

@elmatou
Copy link
Contributor

elmatou commented Feb 28, 2015

Hi Robert,
Sadly pdftk needs compilation to run on every OS, that mean each distrib has its flavor : x86, arm, win32, osx...
What you can do, is build a activ_pdtk-x86 gem and set the path while constructing the wrapper with the :path options.

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

No branches or pull requests

2 participants