-
Notifications
You must be signed in to change notification settings - Fork 65
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
[TIKA-3420] Set tesseract ocr langauges as docker build args #2
base: master
Are you sure you want to change the base?
Conversation
Seems be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR, this will be a great addition to tika-docker!
I've got a few comments as it looks like this may not work as intended, so wondering if you could take a look at them.
Thanks,
Dave
Change the build for rest of parameters and fix Also some change in README.md using Markdown lint and this changes, change my PR if needed for grammer or etc problem. Thanks for your attention |
@dameikle Could you please look at this? Any problem, issue, changes? This simple patch will help me a lot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @mhf-ir I like this pull request. Please consider my feedback. Thank you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:like:
Seems |
@mhf-ir OK I tried out the new patch today
...
Is this the correct way to invoke |
@lewismc seems be problem for multiple packages name for build-args. i will try to find better way for that. |
@lewismc Try this, must be okey
|
This doesn't work either @mhf-ir
|
seems
try this:
|
Ability to user build docker with list of
tesseract-ocr-[lang]
as build args.