-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for VS 2019 #60
base: master
Are you sure you want to change the base?
Conversation
hi @peterM any complied vsix file link? |
hi @hippasus, you just need clone my branch and build it. Then |
Thanks for this idea @peterM! I was able to clone, build, and install from the .VSIX as you mentioned. I only have two issues: 1) VS 2019 is complaining about this extension loading synchronously, and 2) I couldn't resolve the error "The service 'Microsoft.Internal.VisualStudio.Shell.Interop.SVsUIThreadInvokerPrivate' must be installed for this feature to work. Ensure that this service is available" when trying to run the tests. But for now, this appears to be a viable short-term solution. |
Hi @3milio, Thank you for your input. I will try to look on issues. Maybe i will be able to solve them. Could you more describe 2nd issue (steps to reproduce)? I did not experience it. |
@peterM, sorry for the delay in responding. |
@3milio I will try. To be honest on my computer did not work tests .. but when i commited code and pull requests checks triggered build it was after several attempts green :) Last week I tried migrate extension to support Asynchronous loading, but is not so easy to change inheritance from |
BTW: Visual studio 2019.1 will probably stop support loading synchronous packages so it will need to implement |
new to this GitHub, how do I pull your branch? |
execute |
Any way this could get merged or a new repository added so others can access it via VS 2019? Thanks peterM for doing this! I just pulled down your version to run the install and it worked great, but would be nice not to have to do that. |
What is the plan for official release for VS 2019? |
Thanks for doing this @peterM Here's the build log:
|
Here is built extension |
Hmm is hard to say because i think original author do not have time to maintain this. Hopefuly someone will take it. Currently i also do not have time to properly migrate this extension to fully support VS2019 |
After installed T4Toolbox.0.0.0.0.zip in vs 2019 preview 16.4.0 there is some problem with include once="true" |
Thanks @peterM this works for me - although I only use a limited set of features here (basically I use this primarily to trigger a rebuid of a .tt template when an xml file is changed) I do notice however that it can only be loaded via allowing synchronous packages to load. Is this something that is simple to update? Also maybe @olegsych could accept and make this official? |
I tried to migrate it to use |
I'm giving it a go based on PeterM's fork. The AsyncPackage is a bit of a bind. In a lot of places the ServiceLocator is passed down into the objects and stored. At a later point, in a synchronous method, it then tries to resolve a service. Under the AsyncPackage you can't do this, because resolving services becomes an asynchronous process. (GetServiceAsync vs GetService) So I'm refactoring the relevant objects to accept their dependencies through constructors. I'll do the service resolution asynchronously BEFORE object creation. Once I've slain that dragon, I'll work out how to get the Visual Studio Integration tests working. MSTest v1 had some special sauce for running those which no longer exists in MSTest v2. |
I tried to compile the projects but I have the following problem: First a compilation error:
VSConstants is not found and I find out that I have to install the package Microsoft.VisualStudio.Shell.Framework. But it seems one its dependencies is not compatible with .NET 4.72: Install-Package : Could not install package 'Microsoft.ServiceHub.Analyzers 2.6.92'. You are trying to install this package into a |
I was able to compile it and build the VSIX for Visual Studio 2019. Download it here: http://dl.compdj.us/download/index/T4Toolbox15003VS2019vsix |
@dbalciunas Have you been able to resolve /work around this issue or found the cause? I'm seing something similar when trying to update nuget packages of another vsix project. Thanks! |
No, I don't. I didn't try anymore. Sorry. |
Do you think one should open a separate issue on this? Where would be the right place for that? |
Why does this break with every release of Visual Studio? The T4Toolbox is such a great utility for what it does, this is really a pain. Do we have a working build yet? |
Hi,
I updated source code of T4Toolbox to be possible use it with VS 2019.
I also think this should not be merged to current repository and instead of this should have its own repository.
I am also not for 100% sure if i changed all needed fields.
Thanks
#59