-
Notifications
You must be signed in to change notification settings - Fork 38
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Item13079: Item13081: Better bootstrap for tools/configure
Change configure to make use of the Load::bootstrapConfig() function. This will apply consistent guesses in shell and web environments. And use the guesses from bootstrapConfig as the defaults in the prompts. That cleans up the ../ relative paths. Add an "optional" flag to the _prompt routine, so that settings permitting empty or undefined can be configured. There are still a few things that should be done. It would be good if Load::bootstrapConfig() could use the $reporter instead of returning strings. Also if for some reason bootstrap is unable to guess a critical directory, it will still die. That needs to be handled better. Fixed up the docs for -set. Maybe it's a GetOpt issue, but I can't get -set {key} 'value' to work. It has to be -set {key}='value', the equal sign appears to be required.
- Loading branch information
Showing
1 changed file
with
45 additions
and
17 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters