On Fri, 11 Aug 2023 01:49:57 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> On OSX, don't require that sudo be used to launch SA tools if developer mode > is enabled. More details are in the CR. > > Due to this change, the following tests are no longer being skipped if the > host has developer mode is enabled. They previously required running as root > because if sudo was used some files were created with root ownership and > could not be deleted: > > serviceability/sa/ClhsdbDumpclass.java > serviceability/sa/TestClassDump.java > > I tested on my home system (M1) with developer mode enabled and also > disabled. Also tested with CI, which appears to have developer mode disabled > on most hosts, but there are a few where it is enabled. This pull request has now been integrated. Changeset: 62ca0015 Author: Chris Plummer <cjplum...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/62ca00158c7ce7b40b5910562c1857b9f05ddf9f Stats: 55 lines in 1 file changed: 51 ins; 0 del; 4 mod 8313357: Revisit requiring SA tests on OSX to either run as root or use sudo Reviewed-by: dholmes, amenkov ------------- PR: https://git.openjdk.org/jdk/pull/15238
