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

tesseract/4.1.1: libarchive dep and portable build #4278

Merged
merged 17 commits into from
Jan 19, 2021

Conversation

theirix
Copy link
Contributor

@theirix theirix commented Jan 16, 2021

Specify library name and version: tesseract/4.1.1

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

Changes:

  • Add dependency to libarchive
  • Made a build portable. No SSE autodetection and march=native. But end-user can use options to enable optimizations.
  • Modernized recipe
  • Moved out patches
  • Added components
  • Test with target
  • Preparing for 5.x

@conan-center-bot
Copy link
Collaborator

Failure in build 1 (12814708329c6a54de97a3c7c65861f0a7dc3897):

  • Error processing recipe (ref 'tesseract/4.1.1'): Linux x86_64, Release, gcc 4.9, libstdc++ . Options: tesseract:shared-False
    Unexpected error happened:
ERROR: tesseract/4.1.1: Error in configure() method, line 69
	raise ConanInvalidConfiguration("%s requires a {} version >= {}" % (self.name, compiler, compiler_version))
	TypeError: not all arguments converted during string formatting

@conan-center-bot
Copy link
Collaborator

Some configurations of 'tesseract/4.1.1' failed in build 2 (392d195ca9c6b95847225802ba9ec950ff3e8575):

@conan-center-bot
Copy link
Collaborator

Some configurations of 'tesseract/4.1.1' failed in build 3 (c57ba0ce21c2f38a81147089400fde44f529e18c):

@conan-center-bot
Copy link
Collaborator

All green in build 4 (e63627e078792ee4103416eb0dd390b64d757272)! 😊

Comment on lines +115 to +116
# required for 5.0
tools.rmdir(os.path.join(self.package_folder, 'lib', 'cmake'))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's no 5.0 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was experimenting in packaging 5.0.x-alpha. Of course, I can drop it now and add for the final version.

Co-authored-by: Chris Mc <[email protected]>
@conan-center-bot
Copy link
Collaborator

All green in build 5 (e6844ee7747b34cf85c0a5e9aa6e56e35bdeab23)! 😊

@conan-center-bot
Copy link
Collaborator

All green in build 6 (ba82888459b25f35efc4a9fa34120d4cd2b012a8)! 😊

@dmn-star
Copy link
Contributor

dmn-star commented Jan 16, 2021

I was just wondering why the tesseract lib can be compiled at all on CCI for macOS (CCI uses all apple-clang versions in C++03 mode 🤨). https://github.com/tesseract-ocr/tesseract/pull/643/files

It's really a matter of luck with which c++ standard the libs are built.

@theirix
Copy link
Contributor Author

theirix commented Jan 16, 2021

I was just wondering why the tesseract lib can be compiled at all on CCI for macOS (CCI uses all apple-clang versions in C++03 mode 🤨). https://github.com/tesseract-ocr/tesseract/pull/643/files

It's really a matter of luck with which c++ standard the libs are built.

tesseract uses a clever introspection for C++ 11/14/17 and uses the latest supported standard supported by a compiler. There is no minimum required standard anywhere. If a compiler has C++-XY features, they will be enabled via CMAKE_CXX_STANDARD

@conan-center-bot
Copy link
Collaborator

All green in build 7 (7460bc5612958c925025deb3dcf96a23247dc96d)! 😊

@SSE4 SSE4 requested a review from uilianries January 17, 2021 04:52
@conan-center-bot conan-center-bot merged commit 75a3b6c into conan-io:master Jan 19, 2021
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

Successfully merging this pull request may close these issues.

6 participants