On Tue, 3 Sep 2024 08:06:14 GMT, Kevin Walls <kev...@openjdk.org> wrote:

> Deprecation annotations and warnings on starting the tool(s).
> Handle man page in a separate issue.

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/HotSpotAgent.java:
Add annotation Deprecated(since="24", forRemoval=true) on the various 
startServer methods, and shutdownServer

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/SALauncher.java:
Add deprecation warnings in help messages.

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/DebugServer.java:
A standalone launcher, providing another main entry point (separate to jhsdb), 
which creates a HotSpotAgent() and calls startServer().  This class should have 
the same deprecation annotation, and print a warning in its main method.

Handle man page in a separate issue.


bash-4.2$ build/linux-x64/images/jdk/bin/jhsdb
    clhsdb              command line debugger
    hsdb                ui debugger                                             
                                                   
    debugd --help       to get more information (deprecated)
    jstack --help       to get more information
    jmap   --help       to get more information
    jinfo  --help       to get more information
    jsnap  --help       to get more information
bash-4.2$      

bash-4.2$ build/linux-x64/images/jdk/bin/jhsdb debugd --help                    
                                               
WARNING: debugd is deprecated and will be removed in a future release.          
                                                 
  --serverid <id>     ...etc...
  
  
bash-4.2$ build/linux-x64/images/jdk/bin/jhsdb jstack --help
    --locks                 To print java.util.concurrent locks.                
                                                 
    --mixed                 To print both Java and native frames (mixed mode).
    --pid <pid>             To attach to and operate on the given live process.
    --core <corefile>       To operate on the given core file.
    --exe <executable for corefile>
    --connect [<serverid>@]<host>[:registryport][/servername] To connect to a 
remote debug server (debugd).

WARNING: debugd (and therefore --connect) are deprecated and will be removed in 
a future release.

    The --core and --exe options must be set together to give the core..etc...


bash-4.2$ build/linux-x64/images/jdk/bin/jhsdb debugd --pid 123                 
                                               
WARNING: debugd is deprecated and will be removed in a future release.          
                                               
Attaching to process ID 123 and starting RMI services, please wait...  
...etc...


Additionally, update the html docs with the same warning:
src/jdk.hotspot.agent/doc/index.html
src/jdk.hotspot.agent/doc/transported_core.html

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

PR Comment: https://git.openjdk.org/jdk/pull/20830#issuecomment-2325899441

Reply via email to