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

Make training issue #1374

Closed
16seven opened this issue Mar 12, 2018 · 19 comments
Closed

Make training issue #1374

16seven opened this issue Mar 12, 2018 · 19 comments
Labels
Milestone

Comments

@16seven
Copy link

16seven commented Mar 12, 2018

Environment

  • Tesseract Version: 4.0.0-beta.1-16-g4b2bea7
  • Commit Number: 4b2bea
  • Platform: CentOS Linux 7 64-bit

Current Behavior:

Install tesseract from git source
Tesseract tools was installed fine
But compile training tools when exec
make training
occur
boxchar.cpp:67:42: error: 'U_RIGHT_TO_LEFT_ISOLATE' was not declared in this scope dir == U_ARABIC_NUMBER || dir == U_RIGHT_TO_LEFT_ISOLATE) {

Suggested Fix:

Remove the code?
|| dir == U_RIGHT_TO_LEFT_ISOLATE
Does have any effect?

@godofcheerup
Copy link

Hi . I have never seen before 4.00 - beta ...

Are you sure beta??? tesseract is yet Alpha

@Shreeshrii
Copy link
Collaborator

Shreeshrii commented Mar 12, 2018 via email

@Shreeshrii
Copy link
Collaborator

Platform: CentOS Linux 7 64-bit

Do you have the additional dependencies required for building training tools?

@Shreeshrii
Copy link
Collaborator

See #1179 (comment)

The training tools need icu version 52 and up. The icu version in RHEL & CentOS 7 is 50.

@16seven
Copy link
Author

16seven commented Mar 12, 2018

thx @Shreeshrii

@16seven 16seven closed this as completed Mar 12, 2018
@Shreeshrii
Copy link
Collaborator

Alex compiled an rpm package with tesseract-ocr for CentOS, Fedora, ScientificLinux, OpenSuse.

https://build.opensuse.org/project/show/home:Alexander_Pozdnyakov

@16seven Please check if it works on Centos7.

@stweil stweil added the bug label Dec 8, 2018
@stweil stweil added this to the 4.1.0 milestone Dec 8, 2018
@stweil stweil reopened this Dec 8, 2018
@stweil
Copy link
Member

stweil commented Dec 8, 2018

configure.ac should detect whether make training is supported. Currently it includes a check which wrongly assumes that there is a usable ICU package installed. Maybe that check can simply be removed. In addition, ICU_UC_LIBS should be set with the values detected by pkg-config.

@amitdo
Copy link
Collaborator

amitdo commented Dec 8, 2018

The issue was reported before 98747b3 was merged.

The version of the icu package in the official CentOS repo is 50.1, which is lower than the minimum required version 52.1.

http://mirror.centos.org/centos/7/os/x86_64/Packages/

@stweil
Copy link
Member

stweil commented Dec 8, 2018

Even with the latest version from Git the issue still exists as I noticed today. The version check fails, but the header check says that everything is fine and allows building training tools with the know build error.

@stweil
Copy link
Member

stweil commented Dec 8, 2018

The build was done on a Red Hat Enterprise Linux Server release 7.5.

@JensHumrich
Copy link

The issue still seems to exist. Any solutions yet?

@zdenop
Copy link
Contributor

zdenop commented Jan 31, 2019

What do you mean with "the issue still exist"? Is there issue with recent version of ICU (>52.1)?

@JensHumrich
Copy link

If icu is below 52.1, no error is thrown, when configuring. The error appears first during make-ing the training.

@zdenop
Copy link
Contributor

zdenop commented Feb 1, 2019

So why you asks for solution? amitdo already wrote you need to have ICU >= 52.1

@JensHumrich
Copy link

The missing requirements should be found, but is not. The missing ICU libraries should be complained during configure. The option to make training should not be displayed. Instead a warning: "Missing ICU..."
There have been some lines added to the configure check, which are not working correctly.

@zdenop
Copy link
Contributor

zdenop commented Feb 1, 2019

But it is not solution for you problem! You will not be able to build training tool with it. Solution is to use recent ICU version. And it is already described.

@stweil
Copy link
Member

stweil commented Feb 1, 2019

See pull request #2211 for a fix of this issue.

@JensHumrich
Copy link

Thanks a lot @stweil for your contribution. This is the solution I was hoping for.

@zdenop zdenop closed this as completed Feb 1, 2019
@abneetOSS
Copy link

abneetOSS commented Jan 25, 2020

@JensHumrich If you have trouble with ICU version you can add remi repo for CentOS 7 and install libicu62. I was having similar issue as i was not able to make training because icu <=51.2 in release4.1.1
$wget http://rpms.famillecollet.com/enterprise/remi-release-7.rpm
$rpm -Uvh remi-release-7.rpm

$vim /etc/yum.repos.d/remi.repo

find enable=0 and make it as enable=1
this will enable the repo

$yum install libicu62-devel
then you can do ./configure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants