search

gkovacs / pdfocr

Adds text to PDF files using the cuneiform OCR software
MIT License
325 stars 49 forks source link

Update pdfocr.rb #22

Closed thomasX closed 9 years ago

thomasX commented 9 years ago

fixed: if running with tesseract and couldn't ocr scan one page of the source-pdf , this page was removed from the destination-pdf fixed: if running with tesseract on large pdf's we could run into OutOfDiscSpace

gkovacs commented 9 years ago

is the -R in "rm -Rf" actually necessary for files? I thought that was only needed for removing directories and not files. just want to be sure this doesn't risk somehow doing an rm -Rf / or so. looks good otherwise

thomasX commented 9 years ago

thank's ! thats wright , we need only rm -f ... i have changed this in my fork and made a new pullrequest

2015-05-19 3:16 GMT+02:00 Geza Kovacs notifications@github.com:

is the -R in "rm -Rf" actually necessary for files? I thought that was only needed for removing directories and not files. just want to be sure this doesn't risk somehow doing an rm -Rf / or so. looks good otherwise

— Reply to this email directly or view it on GitHub https://github.com/gkovacs/pdfocr/pull/22#issuecomment-103290265.

Mit freundlichen Grüssen | Best Regards

Ing. Thomas Margreiter Turbinenweg 36 A-6250 Kundl