From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io1-xd32.google.com (mail-io1-xd32.google.com [IPv6:2607:f8b0:4864:20::d32]) by sourceware.org (Postfix) with ESMTPS id 8A0203857803 for ; Tue, 25 Jul 2023 13:49:40 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 8A0203857803 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com Received: by mail-io1-xd32.google.com with SMTP id ca18e2360f4ac-7835c92c373so146772139f.0 for ; Tue, 25 Jul 2023 06:49:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1690292979; x=1690897779; h=to:content-transfer-encoding:mime-version:message-id:date:subject :from:from:to:cc:subject:date:message-id:reply-to; bh=1lqR1SzaJhcb3U5ZEe0lBlR9d0NM3X1xbCKhV2wz8Vo=; b=Z4zHds61ke3dhXDYnLXwZjQnx1JjVh12J7qt07iE2gQMFhjkjGv5hsAlYnxHSps4Mk Mfbn8jRqlEO/aAu57J6pKGhOK0PP0H21a/KvhK/FOOkAX/Axn61Mej30DsbcUM0wtEVv EHaKbABiXOyL8iY8iDZpTdrYA7RflOzq5DTNAN5xcC5mWgdloANJ+kKoCL1HUzNK8TQi bFoL4SJ0VfnaJyihUGyTZb+oxNcv9CXXuARc/pdjLNOXB1unJ0LTSKOE7/QuCqApaTqK Q2BCgUgv6nv7xwKEtwkzi6Oe1T4xJTawZcwT8s61fCj2T2TrHukClj5Pb2hq3zwZV5Y8 KS6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690292979; x=1690897779; h=to:content-transfer-encoding:mime-version:message-id:date:subject :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=1lqR1SzaJhcb3U5ZEe0lBlR9d0NM3X1xbCKhV2wz8Vo=; b=QWn+dilOiS9M28WHWi/jsOmFT92utpkQtPE8/UvofYuD3xn8oDqyk0Aeu3j8rsfQkY luNf/KCMPsk4smTRlmSZJOGrEoIwWU1fJ55LvCfnMUmB3REGCbHYIrAPgKMrSwNfgbem 67lXI/3ZDKIJ8dU2AUap+t89byF2MEheSp7E8lXx2AqJq8LYZ7flWLlu/5r6z+1jAZgR N+WxU963DSOhzJH1oZ9wwQbEQn+NvDhdWxihvQMkPss7cipcIxoh8afc4xiLcA292Uns bXei8F/6HNu+ZbkQDX991XxRSCG4VvXrTevQDQhYh8o3Nx+45HzPG+xWSgMYyHKpaD3y Jx4Q== X-Gm-Message-State: ABy/qLbZlwND/yods47jNyVMNxYtDq1q/F3AkUUAKnryULtA8w9phXhE c1uiBTCHlJZMxcnmaIy1uZRtPm33jpoayK47dVUP1Q== X-Google-Smtp-Source: APBJJlHTCiIvQYwkWN9TCrS7JH93f5mz6F2xtWVWmfLOtXUnfYm2P9GMjHgIqSXai7LLY6fpZAjAyQ== X-Received: by 2002:a05:6e02:12e4:b0:346:4f37:8a3 with SMTP id l4-20020a056e0212e400b003464f3708a3mr1665013iln.5.1690292979762; Tue, 25 Jul 2023 06:49:39 -0700 (PDT) Received: from localhost.localdomain (75-166-135-140.hlrn.qwest.net. [75.166.135.140]) by smtp.gmail.com with ESMTPSA id c3-20020a02a403000000b00418a5e0e93esm3595018jal.162.2023.07.25.06.49.39 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Jul 2023 06:49:39 -0700 (PDT) From: Tom Tromey Subject: [PATCH 0/6] Several DAP fixes for VSCode Date: Tue, 25 Jul 2023 07:49:37 -0600 Message-Id: <20230725-dap-bt-path-v1-0-bb015b0d8e54@adacore.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAPHSv2QC/x3MQQrCMBBG4auUWTuQJqjoVcTFpP01szCGSShC6 d1Nu/wW761UYYpK92Elw6JVv7ljPA00JclvsM7d5J0P7urPPEvh2LhISxw8bqOTAIQL9aIYXvo 7bo9nd5QKjiZ5SvvjI7XBaNv+2pSVKHYAAAA= To: gdb-patches@sourceware.org X-Mailer: b4 0.12.3 X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Vladimir Makaev tried the gdb DAP implementation with VSCode and found a few bugs. This series fixes these. It also adds a new DAP launch parameter -- I thought I was going to need this, and while I didn't, it still seemed like a good idea to add. I'm not completely happy with this series. See the "full paths" patch for some discussion. Essentially I think the FrameDecorator API is not very well thought out, and once again this is an area where it would be very nice if we could add new methods to the spec. I wonder whether we should. One difference between pretty-printers and frame decorators is that my sense is that there are many more of the former, so the smaller for the latter. Another idea I had for that patch is to just change the base behavior of FrameDecorator, based on a global flag -- that would only be set by DAP. --- Tom Tromey (6): Rename private member of FrameDecorator Refactor dap_launch Add "cwd" parameter to DAP launch request Full paths in DAP stackTrace responses Move DAP breakpoint event code to breakpoint.py Do not send "new breakpoint" event when breakpoint is set gdb/doc/gdb.texinfo | 8 +++ gdb/python/lib/gdb/FrameDecorator.py | 119 ++++++++++++++++++++------------- gdb/python/lib/gdb/dap/breakpoint.py | 64 +++++++++++++++++- gdb/python/lib/gdb/dap/events.py | 37 ---------- gdb/python/lib/gdb/dap/launch.py | 34 +++++----- gdb/python/lib/gdb/frames.py | 14 +++- gdb/testsuite/gdb.dap/args-env.exp | 2 +- gdb/testsuite/gdb.dap/basic-dap.exp | 11 +-- gdb/testsuite/gdb.dap/cwd.exp | 40 +++++++++++ gdb/testsuite/gdb.dap/stop-at-main.exp | 2 +- gdb/testsuite/lib/dap-support.exp | 72 +++++++++++++------- 11 files changed, 262 insertions(+), 141 deletions(-) --- base-commit: 695776dc2f43c56dd2ae2f7036fb7cf74e19b46b change-id: 20230725-dap-bt-path-32e910a3ee36 Best regards, -- Tom Tromey