> jdb has 3 types of arguments: > > 1. Those that are jdb specific, such as -attach, -launch, and -listconnectors > 2. Those that are passed to the JVM used to run jdb. These are all prefixed > with -J, and any valid JVM argument can be passed in this manner. > 3. Those that are passed to the debuggee JVM when it is launched, such as > -classpath, -D<property>, and any option that starts with -X (including -XX) > > The problem with the 3rd group is that (other than for -D and -X), jdb will > only pass through arguments that it recognizes, and that list is very > limited. When you want to launch the debuggee with an argument that jdb > doesn't recognize, you have no choice but to launch the debuggee yourself > (using a separate command line and using the -agentlib:jdwp argument) and > then tell jdb to attach to the debuggee process. It's much easier when you > can just let jdb launch the debuggee, and our nsk/jdb testing currently > relies on this feature. > > This PR adds --enable-preview and --add-modules to the list of arguments that > jdb recognizes and passes through to the debuggee JVM. These seem to be the > two most glaring omissions, and are two that will likely be needed soon in > order for loom nsk/jdb testing to work properly.
Chris Plummer has updated the pull request incrementally with three additional commits since the last revision: - Re-add copyright change. - Remove copyright change. - Support -R instead of --enable-preview and --add-modules. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7708/files - new: https://git.openjdk.java.net/jdk/pull/7708/files/5ce3a7fc..8edb6986 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7708&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7708&range=00-01 Stats: 37 lines in 5 files changed: 19 ins; 14 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/7708.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7708/head:pull/7708 PR: https://git.openjdk.java.net/jdk/pull/7708
