> Strip currently has its flags set by guessing based on the OS, it would be 
> more appropriate to instead set them based on properly checking the strip 
> binary instead of speculating what the correct flags are based on the 
> compiler and OS.

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 16 additional commits 
since the last revision:

 - Merge branch 'openjdk:master' into patch-1
 - Comment can be made more accurate
 - Merge branch 'openjdk:master' into patch-1
 - Merge branch 'openjdk:master' into patch-1
 - Merge branch 'openjdk:master' into patch-1
 - Add more documentation
 - Replace with the actual new check
 - Little documentation doesn't hurt
 - Prepare for actual changes
 - Update flags-other.m4
 - ... and 6 more: https://git.openjdk.org/jdk/compare/3e5e6e0e...df46df9d

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10882/files
  - new: https://git.openjdk.org/jdk/pull/10882/files/d831ca59..df46df9d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=03-04

  Stats: 173 lines in 11 files changed: 89 ins; 33 del; 51 mod
  Patch: https://git.openjdk.org/jdk/pull/10882.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10882/head:pull/10882

PR: https://git.openjdk.org/jdk/pull/10882

Reply via email to