> 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 one additional commit since the last revision: Minor help message improvements. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7708/files - new: https://git.openjdk.java.net/jdk/pull/7708/files/8edb6986..8abe4c66 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7708&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7708&range=01-02 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 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
