Skip to content

Add vsdbg for .net core#23

Merged
quoctruong merged 2 commits intoGoogleContainerTools:duct-tapefrom
quoctruong:netcore
May 8, 2019
Merged

Add vsdbg for .net core#23
quoctruong merged 2 commits intoGoogleContainerTools:duct-tapefrom
quoctruong:netcore

Conversation

@quoctruong
Copy link
Copy Markdown
Contributor

COPY --from=delve /go/bin/dlv go/bin/
COPY --from=python27 /ptvsd/ python/
COPY --from=python37 /ptvsd/ python/
COPY --from=netcore /vsdbg/ netcore/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will vsdbg support future versions like .NET 5? Should we version this directory?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vsdbg seems to work with all versions of .NET Core so far (1.0, 2.0 and soon to be released 3.0) so I don't think that will be a problem.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That being said, I just realized vsdbg is runtime specific so this version of vsdbg will only work for Linux container.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's true of the python support files too. We'll have to publish separate images when/if we support alternative os/arch combinations.

Copy link
Copy Markdown
Member

@briandealwis briandealwis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@quoctruong quoctruong merged commit e50e192 into GoogleContainerTools:duct-tape May 8, 2019
@quoctruong quoctruong deleted the netcore branch May 8, 2019 20:25
@quoctruong
Copy link
Copy Markdown
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants