Fix setup of datadir on installations with Conda (issue #4230) #4240
Conversation
…#4230) Signed-off-by: Stefan Weil <sw@weilnetz.de>
| tessdata_prefix = TESSDATA_PREFIX; | ||
| datadir = tessdata_prefix; | ||
| datadir += "/tessdata/"; | ||
| // in the binary, so it might be shorter. Recalculate its length. |
There was a problem hiding this comment.
Since std::string is initialized from "const char*", I wonder how it can be shorter?
There was a problem hiding this comment.
The length is calculated at compile time and not updated at runtime if the const char * was patched.
There was a problem hiding this comment.
If you have a locally built tesseract installed in /usr/local/bin (= with prefix /usr/local), you can patch the binary for example using perl -pi -e 's,/usr/l,/tmp/\0,' /usr/local/bin/tesseract. Then try tesseract --list-langs to get the endless recursion.
There was a problem hiding this comment.
As I understand, if the patched path is longer than the original one, the data in std::string is truncated. If so, it probably makes sense to call strlen() on the string literal rather than on c_str().
There was a problem hiding this comment.
It's not possible to replace the original TESSDATA_PREFIX by a longer one because that would destroy other data. Therefore Conda uses a very long placeholder for TESSDATA_PREFIX.
No description provided.