[PR #715] [CLOSED] [IMPROVEMENT] Add Autoconf scripts to make CCExtractor more linux standard compliant #1541

Open
opened 2026-01-29 17:17:03 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/715
Author: @fandango96
Created: 3/13/2017
Status: Closed

Base: masterHead: linux_standard


📝 Commits (10+)

  • 594122a Use the directory in which script is located as base for pre-build.sh; this allows relative paths to work even if the script is called from a different directory
  • a9600a6 Initial autotools scripts commit
  • 24c3fd6 Add AC_CONFIG_SRCDIR to configure.ac
  • 9ede921 Add AC_CONFIG_HEADERS macro
  • 1277fcb Remove original Makefile; Add lib, header and function checking to configure.ac
  • 5d4acd8 Add files required for proper building using configure/make/make install
  • 1d77c18 Rename all target in Makefile.am to all-local
  • 04102a7 Add local targets for install, uninstall
  • c16af07 Reverse the condition for AVFLAGS_ARE_EMPTY
  • 0d57923 Move libav* checks to configure.ac

📊 Changes

5 files changed (+164 additions, -59 deletions)

View changed files

📝 .gitignore (+15 -0)
📝 linux/Makefile.am (+40 -54)
linux/bootstrap (+14 -0)
linux/configure.ac (+89 -0)
📝 linux/pre-build.sh (+6 -5)

📄 Description

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

  • I have never used CCExtractor.
  • I have used CCExtractor just a couple of times.
  • I absolutely love CCExtractor, but have not contributed previously.
  • I am an active contributor to CCExtractor.

This PR add autoconf scripts (mainly configure.ac and Makefile.am) to allow the standard configure / make / make install process for building and installing CCExtractor on Linux systems.

Makefile.am is mostly derived from the existing Makefile (removed in this PR as it is no longer required) with target renaming as appropriate.

All the lib checking (tesseract, leptonica, etc.) and option parsing (--enable-ocr, --enable-ffmpeg, etc.) is now moved to the configure step after the addition of relevant macros in configure.ac

The other files added (Makefile.in, config.h.in, etc.) allow the building and installation to take place on systems where autotools is not installed.

pre-build.sh was modified to use relative paths with respect to the directory in which the shell script exists and not the current build directory.

Tested on
Kernel : Linux 4.4.48-1-MANJARO (x86_64)
C Library : GNU C Library version 2.24 (stable)
Default C Compiler : GNU C Compiler version 6.3.1 20170109 (GCC)
Distribution : Manjaro Linux

Part 1 of 3 to resolve Issue #678


🔄 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/715 **Author:** [@fandango96](https://github.com/fandango96) **Created:** 3/13/2017 **Status:** ❌ Closed **Base:** `master` ← **Head:** `linux_standard` --- ### 📝 Commits (10+) - [`594122a`](https://github.com/CCExtractor/ccextractor/commit/594122a38ea93e5b836f542b8e80e134348373da) Use the directory in which script is located as base for pre-build.sh; this allows relative paths to work even if the script is called from a different directory - [`a9600a6`](https://github.com/CCExtractor/ccextractor/commit/a9600a66a647cb133aa9030e47fb0fb8cb1d44e1) Initial autotools scripts commit - [`24c3fd6`](https://github.com/CCExtractor/ccextractor/commit/24c3fd6e216ce2160f028ed0305d6ebe102fb614) Add AC_CONFIG_SRCDIR to configure.ac - [`9ede921`](https://github.com/CCExtractor/ccextractor/commit/9ede9211219fe384d155e74f00579574eb0a3d4c) Add AC_CONFIG_HEADERS macro - [`1277fcb`](https://github.com/CCExtractor/ccextractor/commit/1277fcbb3c3b85b03254ad711e6800760043c0ef) Remove original Makefile; Add lib, header and function checking to configure.ac - [`5d4acd8`](https://github.com/CCExtractor/ccextractor/commit/5d4acd84df7b7adb6cfd6099d15008be6d96ee3f) Add files required for proper building using configure/make/make install - [`1d77c18`](https://github.com/CCExtractor/ccextractor/commit/1d77c1819e9d144db7d827989b4e88bee49d91a3) Rename all target in Makefile.am to all-local - [`04102a7`](https://github.com/CCExtractor/ccextractor/commit/04102a7b34962227d92c4af64ef8abc9c4700528) Add local targets for install, uninstall - [`c16af07`](https://github.com/CCExtractor/ccextractor/commit/c16af07af9e2281a68fdad5bd7332dcd4c204c09) Reverse the condition for AVFLAGS_ARE_EMPTY - [`0d57923`](https://github.com/CCExtractor/ccextractor/commit/0d579230e8f14471416c198b797c1882ec9e189c) Move libav* checks to configure.ac ### 📊 Changes **5 files changed** (+164 additions, -59 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+15 -0) 📝 `linux/Makefile.am` (+40 -54) ➕ `linux/bootstrap` (+14 -0) ➕ `linux/configure.ac` (+89 -0) 📝 `linux/pre-build.sh` (+6 -5) </details> ### 📄 Description Please prefix your pull request with one of the following: **[FEATURE]** **[FIX]** **[IMPROVEMENT]**. **In raising this pull request, I confirm the following (please check boxes):** - [x] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [x] I have checked that another pull request for this purpose does not exist. - [x] I have considered, and confirmed that this submission will be valuable to others. - [x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer. - [x] I give this submission freely, and claim no ownership to its content. **My familiarity with the project is as follows (check one):** - [ ] I have never used CCExtractor. - [x] I have used CCExtractor just a couple of times. - [ ] I absolutely love CCExtractor, but have not contributed previously. - [ ] I am an active contributor to CCExtractor. --- This PR add autoconf scripts (mainly configure.ac and Makefile.am) to allow the standard configure / make / make install process for building and installing CCExtractor on Linux systems. Makefile.am is mostly derived from the existing Makefile (removed in this PR as it is no longer required) with target renaming as appropriate. All the lib checking (tesseract, leptonica, etc.) and option parsing (--enable-ocr, --enable-ffmpeg, etc.) is now moved to the configure step after the addition of relevant macros in configure.ac The other files added (Makefile.in, config.h.in, etc.) allow the building and installation to take place on systems where autotools is not installed. pre-build.sh was modified to use relative paths with respect to the directory in which the shell script exists and not the current build directory. Tested on Kernel : Linux 4.4.48-1-MANJARO (x86_64) C Library : GNU C Library version 2.24 (stable) Default C Compiler : GNU C Compiler version 6.3.1 20170109 (GCC) Distribution : Manjaro Linux Part 1 of 3 to resolve Issue #678 --- <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:17:03 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#1541