Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

add go-debug #410

Merged
merged 1 commit into from
Aug 16, 2016
Merged

add go-debug #410

merged 1 commit into from
Aug 16, 2016

Conversation

lloiser
Copy link
Collaborator

@lloiser lloiser commented Mar 30, 2016

Haven't touched CI scripts .travis.yml and appveyor.yml.

@andrewrgoss
Copy link

Can we get a timeline on when this will be merged into the go-plus package? Dying to have an interactive debugger for Golang akin to the experience Visual Studio provides...

@joefitzgerald
Copy link
Owner

@andrewrgoss you're not blocked at all? Just install go-debug via the Atom UI or via apm install go-debug on the command line (requires a restart if you use apm.

@joefitzgerald
Copy link
Owner

To answer your question about a timeline for merging: I want to ensure we have a way to automate the installation of delve on the target machine. Without it, we'll get buried with questions about why debugging isn't working. Alternatively we could ship with support for godebug as it is "go gettable" and allow for the user to select delve as an option (until we figure out how to automate the installation).

@andrewrgoss
Copy link

@joefitzgerald appreciate the quick response here, I did install go-debug via the Atom UI, however I can't get delve successfully installed after multiple attempts. Unfortunately the documentation is extremely poor for both go-debug and delve, which is why I was hoping this would be integrated into go-plus.

@joefitzgerald
Copy link
Owner

Hehe, this is precisely why this PR has not been merged yet. Suggest you work with @lloiser on getting things going (but first just work with the delve community to get delve installed correctly) and then maybe we can find a way to automate the steps you had to take. Ideally delve would just be released as a binary that could be downloaded from github releases.

@andrewrgoss
Copy link

Having delve be a simple downloaded binary would be a beautiful, simple thing indeed. I'll see what I can find out. Thanks again for the help.

@lloiser
Copy link
Collaborator Author

lloiser commented Apr 6, 2016

@andrewrgoss feel free to contact me on slack https://gophersinvite.herokuapp.com/ (go-plus channel) or direct message if you need help

@joefitzgerald
Copy link
Owner

:shipit:

@joefitzgerald joefitzgerald merged commit 992de05 into joefitzgerald:master Aug 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants