[PR #1829] [MERGED] build(autoconf): add tesseract/leptonica linking for HARDSUBX #2584

New Issue

claunia · 2026-01-29T17:22:55Z

claunia commented

2026-01-29 17:22:55 +00:00

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1829
Author: @cfsmp3
Created: 12/15/2025
Status: ✅ Merged
Merged: 12/15/2025
Merged by: @cfsmp3

Base: master ← Head: fix/autoconf-hardsubx-tesseract

📝 Commits (1)

d8698dc build(autoconf): add tesseract/leptonica linking for HARDSUBX

📊 Changes

1 file changed (+6 additions, -0 deletions)

View changed files

📝 linux/Makefile.am (+6 -0)

📄 Description

Summary

This is the autoconf equivalent of the CMake fix in PR #1760.

Adds explicit tesseract/leptonica include flags (${tesseract_CFLAGS}, ${lept_CFLAGS}) to the HARDSUBX block in linux/Makefile.am
Adds explicit tesseract/leptonica library linking (${tesseract_LIBS}, ${lept_LIBS}) to the HARDSUBX block

Background

PR #1760 fixed the CMake build system to properly link tesseract/leptonica when building with -DWITH_HARDSUBX=ON -DWITH_OCR=OFF. The autoconf build system had a similar structure where tesseract/leptonica were only explicitly linked in the OCR block.

Note: The autoconf configure.ac already sets OCR_IS_ENABLED when HARDSUBX is enabled, so the build would technically work via the OCR block. However, this change makes the dependency explicit and consistent with the CMake fix.

PR #1760 (CMake fix, now merged)
Issue #1719 (original bug report)

Test plan

Build with --enable-hardsubx without --enable-ocr
Verify no undefined reference errors for tesseract/leptonica symbols

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/1829 **Author:** [@cfsmp3](https://github.com/cfsmp3) **Created:** 12/15/2025 **Status:** ✅ Merged **Merged:** 12/15/2025 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `fix/autoconf-hardsubx-tesseract` --- ### 📝 Commits (1) - [`d8698dc`](https://github.com/CCExtractor/ccextractor/commit/d8698dc9cb4840d6ee902ba0f311a4da7d4704ec) build(autoconf): add tesseract/leptonica linking for HARDSUBX ### 📊 Changes **1 file changed** (+6 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `linux/Makefile.am` (+6 -0) </details> ### 📄 Description ## Summary This is the autoconf equivalent of the CMake fix in PR #1760. - Adds explicit tesseract/leptonica include flags (`${tesseract_CFLAGS}`, `${lept_CFLAGS}`) to the HARDSUBX block in `linux/Makefile.am` - Adds explicit tesseract/leptonica library linking (`${tesseract_LIBS}`, `${lept_LIBS}`) to the HARDSUBX block ## Background PR #1760 fixed the CMake build system to properly link tesseract/leptonica when building with `-DWITH_HARDSUBX=ON -DWITH_OCR=OFF`. The autoconf build system had a similar structure where tesseract/leptonica were only explicitly linked in the OCR block. Note: The autoconf `configure.ac` already sets `OCR_IS_ENABLED` when HARDSUBX is enabled, so the build would technically work via the OCR block. However, this change makes the dependency explicit and consistent with the CMake fix. ## Related - PR #1760 (CMake fix, now merged) - Issue #1719 (original bug report) ## Test plan - [ ] Build with `--enable-hardsubx` without `--enable-ocr` - [ ] Verify no undefined reference errors for tesseract/leptonica symbols 🤖 Generated with [Claude Code](https://claude.ai/code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-29 17:22:55 +00:00

claunia closed this issue

2026-01-29 17:22:56 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#2584