> Some of the --without options are not properly handled and will crash when > processed (For example, --without-version-string), in other cases the > --without-* option will actually silently produce incorrect results instead > of actually doing what --without-* implies (For example, --without-build-user > and all the --with-vendor-* options). The most elegant way to solve this > would simply be to handle such cases and display warnings when they're > encountered (or if the option is critical to the build process, throwing an > error) > > Even if it doesn't make sense to pass said option however, it should display > a warning instead of letting configure exit with a confusing error when it's > run
Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into without - MACOSX_BUNDLE_BUILD_VERSION cannot be empty - Fix syntax errors - Fold --without checks into more compact code - Handle remaining options and change critical options to error when --without is passed - Initial proper handling of some --without arguments ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7713/files - new: https://git.openjdk.java.net/jdk/pull/7713/files/4eb5c85a..4cecda2b Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7713&range=05 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7713&range=04-05 Stats: 6806 lines in 298 files changed: 4386 ins; 662 del; 1758 mod Patch: https://git.openjdk.java.net/jdk/pull/7713.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7713/head:pull/7713 PR: https://git.openjdk.java.net/jdk/pull/7713