Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Patch tesseract for automake and GNU libtoolize. #16917

Closed
wants to merge 1 commit into from
Closed

Patch tesseract for automake and GNU libtoolize. #16917

wants to merge 1 commit into from

Conversation

ianatha
Copy link

@ianatha ianatha commented Jan 6, 2013

Without this patch, attempting to install "tesseract", fails due to
"autoheader: error: AC_CONFIG_HEADERS not found in configure.ac".
In recent versions of automake, AM_CONFIG_HEADER has been replaced
with AC_CONFIG_HEADERS. This patch inreplaces 'configure.ac' to fix
this.

Furthermore, the 'libtool' brew installs 'libtoolize' as
'glibtoolize'. This patch inreplaces 'autogen.sh' to fix this.

Without this patch, attempting to install "tesseract", fails due to
"autoheader: error: AC_CONFIG_HEADERS not found in configure.ac".
In recent versions of automake, AM_CONFIG_HEADER has been replaced
with AC_CONFIG_HEADERS. This patch inreplaces 'configure.ac' to fix
this.

Furthermore, the 'libtool' brew installs 'libtoolize' as
'glibtoolize'. This patch inreplaces 'autogen.sh' to fix this.
@Sharpie
Copy link
Contributor

Sharpie commented Jan 6, 2013

Why are we even running autogen.sh in the first place? I don't see any replacements or patches that affect configure.ac so why do we need to regenerate the configure file?

@Sharpie
Copy link
Contributor

Sharpie commented Jan 6, 2013

Link to upstream issue:

http://code.google.com/p/tesseract-ocr/issues/detail?id=819

@Sharpie Sharpie closed this in 0e91da5 Jan 6, 2013
@Sharpie
Copy link
Contributor

Sharpie commented Jan 6, 2013

Removed call to autogen.sh instead. We were using it since the 3.01 release shipped without a Makefile.in. This oversight has been corrected upstream, so re-generating the build system is redundant.

@ianatha ianatha deleted the tesseract-fix branch January 6, 2013 11:14
@ianatha
Copy link
Author

ianatha commented Jan 6, 2013

Much better! Thank you!

dholm pushed a commit to dholm/homebrew that referenced this pull request Jan 14, 2013
A call to `autogen.sh` was introduced in 64bb607 because version 3.01 of
tesseract shipped without a `Makefile.in`. This has been corrected, so
re-generating the buildsystem is no longer necessary.

Also, the current `configure.ac` is no longer compatible with automake 1.13 so
builds were failing. Upstream issue:

  http://code.google.com/p/tesseract-ocr/issues/detail?id=819

Closes Homebrew#16917.
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants