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

build-package: Add -c option in help output #19725

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Biswa96
Copy link
Contributor

@Biswa96 Biswa96 commented Apr 10, 2024

No description provided.

@Biswa96 Biswa96 requested a review from Grimler91 as a code owner April 10, 2024 15:19
@@ -411,6 +411,7 @@ _show_usage() {
echo
echo "Available options:"
[ "$TERMUX_ON_DEVICE_BUILD" = "false" ] && echo " -a The architecture to build for: aarch64(default), arm, i686, x86_64 or all."
echo " -c Continue previous build, skip source extraction and configure step and go straight to termux_step_make."
Copy link
Member

Choose a reason for hiding this comment

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

if [ "$TERMUX_CONTINUE_BUILD" == "false" ]; then
termux_step_get_dependencies_python
termux_step_patch_package
termux_step_replace_guess_scripts
cd "$TERMUX_PKG_SRCDIR"
termux_step_pre_configure
fi
# Even on continued build we might need to setup paths
# to tools so need to run part of configure step
cd "$TERMUX_PKG_BUILDDIR"
termux_step_configure
if [ "$TERMUX_CONTINUE_BUILD" == "false" ]; then

In continue build, termux_step_configure will be executed, so if termux_step_configure is overridden by build.sh, configure step is not skipped.

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 have no idea what it does. I just add the option in help output which was brought up in a GitHub discussion.

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.

None yet

2 participants