> 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 incrementally with one additional 
commit since the last revision:

  Fold --without checks into more compact code

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7713/files
  - new: https://git.openjdk.java.net/jdk/pull/7713/files/c9699eb7..18ec991d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7713&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7713&range=01-02

  Stats: 42 lines in 1 file changed: 4 ins; 24 del; 14 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

Reply via email to