Skip to content

Code-Contribution: Add "Run bootstrap.sh" instruction#193

Closed
seanbright wants to merge 2 commits into
asterisk:mainfrom
seanbright:bootstrap
Closed

Code-Contribution: Add "Run bootstrap.sh" instruction#193
seanbright wants to merge 2 commits into
asterisk:mainfrom
seanbright:bootstrap

Conversation

@seanbright
Copy link
Copy Markdown
Contributor

When configure is removed from the repository, developers will need to run bootstrap.sh in order to generate build scripts on their own system.

When `configure` is removed from the repository, developers will need
to run `bootstrap.sh` in order to generate build scripts on their own
system.
@gtjoseph gtjoseph marked this pull request as draft May 22, 2026 22:36
@gtjoseph
Copy link
Copy Markdown
Member

Marking as draft to prevent unintentional merge until the asterisk PR is merged.

@seanbright
Copy link
Copy Markdown
Contributor Author

The content is written such that it shouldn’t matter when it is merged. If it won’t go in until the other PR does, then I will update it to remove the ambiguity.

@gtjoseph
Copy link
Copy Markdown
Member

gtjoseph commented May 22, 2026

You don't have to remove the ambiguity but you might update the tip noting that branch changes update configure.ac and may require re-running bootstrap.

@gtjoseph gtjoseph marked this pull request as ready for review May 22, 2026 22:57
Copy link
Copy Markdown
Member

@gtjoseph gtjoseph left a comment

Choose a reason for hiding this comment

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

Sorry, I just thought of one more warning...

Do NOT run bootstrap.sh if configure already exists unless you've made changes to configure.ac or any of the files in the autoconf directory. Doing so could result in unintended changes not related to your task.

@seanbright
Copy link
Copy Markdown
Contributor Author

This change is now moot

@seanbright seanbright closed this May 23, 2026
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.

3 participants