RDKB-63015: Fix to support repo without autogen script#32
RDKB-63015: Fix to support repo without autogen script#32GoutamD2905 merged 1 commit intodevelopfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the native component build flow to better handle autotools-based repositories that don’t ship an autogen.sh, by attempting to generate configure from autotools inputs.
Changes:
- Add an
autoreconffallback inbuild_component_autotools()whenautogen.shis missing andconfigurehas not been generated. - Gate the fallback on presence of
configure.ac.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| elif [[ ! -f "./configure" ]] && [[ -f "./configure.ac" ]]; then | ||
| # No autogen.sh and no configure, but configure.ac exists | ||
| # Run autoreconf to generate configure script | ||
| step "Running autoreconf to generate configure script" |
There was a problem hiding this comment.
The autoreconf fallback only checks for configure.ac. Some autotools projects use configure.in instead; in that case this branch is skipped and the next step tries to run ./configure even though it was never generated, causing an immediate build failure. Consider treating configure.in the same as configure.ac (or failing early with a clear error if neither is present and ./configure is missing).
Updating the build_native.sh to use autoreconf when autogen.sh is not available but configure.ac is available
guruchandru
force-pushed
the
feature/native-build-configure
branch
from
6b01785 to
c7b6534
Compare
Updating the build_native.sh to use autoreconf when autogen.sh is not available but configure.ac is available