Skip to content

Commit

Permalink
fix: automatically set attached mode for delve based on OS (#86)
Browse files Browse the repository at this point in the history
* Automatically set attached mode for devle.

* Fix readme.

* Fix typo
  • Loading branch information
PerMalmberg authored Jul 4, 2024
1 parent a0c5a2b commit 3999f07
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ lua require('dap-go').setup {
-- whether the dlv process to be created detached or not. there is
-- an issue on Windows where this needs to be set to false
-- otherwise the dlv server creation will fail.
detached = true,
detached = vim.fn.has("win32") == 0,
-- the current working directory to run dlv from, if other than
-- the current working directory.
cwd = nil,
Expand Down
4 changes: 3 additions & 1 deletion lua/dap-go.lua
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,9 @@ local default_config = {
port = "${port}",
args = {},
build_flags = "",
detached = true,
-- Automativally handle the issue on Windows where delve needs
-- to be run in attched mode or it will fail (actually crashes).
detached = vim.fn.has("win32") == 0,
},
}

Expand Down

0 comments on commit 3999f07

Please sign in to comment.