Skip to content
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

Fix vs2015 #75

Closed
wants to merge 8 commits into from
Closed

Fix vs2015 #75

wants to merge 8 commits into from

Conversation

SSE4
Copy link

@SSE4 SSE4 commented May 8, 2019

closes #66

SSE4 added 3 commits May 8, 2019 19:06
Signed-off-by: SSE4 <tomskside@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
@SSE4 SSE4 requested a review from jgsogo May 8, 2019 12:52
@jgsogo
Copy link

jgsogo commented May 8, 2019

Testing in a VM with only VS2015 installed (no 2017 or 2019 installed before) is needed. Working in 2015 is not required for the first version.

@jgsogo jgsogo added this to the next milestone May 8, 2019
@SSE4
Copy link
Author

SSE4 commented May 14, 2019

checked on VM with only VS2015 installed, works flawlessly

Prepare for release 1.0.x
@Gernatch
Copy link

@jgsogo Do you think this will make it to 1.1? We have someone having this issue and I would like to see if this resolves. Or is there a VSIX I could supply with this change for testing?

@SSE4
Copy link
Author

SSE4 commented Jun 17, 2019

@Gernatch you may try to install from here: https://ci.appveyor.com/project/ConanOrgCI/conan-vs-extension/builds/25177464/artifacts
please let us know if it works for you

@SSE4
Copy link
Author

SSE4 commented Jun 17, 2019

a bit beautified output:

    <description>Client rights determined: {"CurrentClientRights":{"ActivityId":"5210f107-241d-4d79-8eab-7b55ae79ab1c".
"AuthorizedVsEdition":2.
"AuthorizedVsEdition2":300.
"CrtokenDaysToExpire":30.
"DaysSinceCreation":0.
"DaysToExpire":30.
"IsLicenseGood":true.
"IsSubscriptionBased":true.
"IsTrialLicense":false.
"LicenseDescription":"MSDN Subscription".
"LicensedUserId":"ac88aa23-e23a-40f2-b1b4-4454e89428a0".
"LicensedUserName":"username@company(edited this).com".
"LicenseSource":1.
"LicenseSource2":1.
"LicenseStatus":4.
"LicenseType":12.
"LicenseUrl":"".
"LicensingAccount":"".
"MayUplevel":false.
"ReleaseType":1.
"RightsDaysToExpire":2147483647.
"RightsStatus":10.
"SubscriptionChannel":"VL".
"ClientRightsMetadata":{"SubscriptionChannel":"VL".
"NonLocalizedLicenseDescription":"MSDN".
"LicenseUrl":"".
"ExpirationDate":"\/Date(1563371882000)\/".
"ClientRightsVersion":{"Major":15.
"Minor":0.
"Build":28307.
"Revision":705.
"MajorRevision":0.
"MinorRevision":705}}}.
"CurrentLicenseSource":1.
"CurrentLicenseSource2":1.
"IDEUserClientRights":{"ActivityId":"5210f107-241d-4d79-8eab-7b55ae79ab1c".
"AuthorizedVsEdition":2.
"AuthorizedVsEdition2":300.
"CrtokenDaysToExpire":30.
"DaysSinceCreation":0.
"DaysToExpire":30.
"IsLicenseGood":true.
"IsSubscriptionBased":true.
"IsTrialLicense":false.
"LicenseDescription":"MSDN Subscription".
"LicensedUserId":"ac88aa23-e23a-40f2-b1b4-4454e89428a0".
"LicensedUserName":"username@company(edited this).com".
"LicenseSource":1.
"LicenseSource2":1.
"LicenseStatus":4.
"LicenseType":12.
"LicenseUrl":"".
"LicensingAccount":"".
"MayUplevel":false.
"ReleaseType":1.
"RightsDaysToExpire":2147483647.
"RightsStatus":10.
"SubscriptionChannel":"VL".
"ClientRightsMetadata":{"SubscriptionChannel":"VL".
"NonLocalizedLicenseDescription":"MSDN".
"LicenseUrl":"".
"ExpirationDate":"\/Date(1563371882000)\/".
"ClientRightsVersion":{"Major":15.
"Minor":0.
"Build":28307.
"Revision":705.
"MajorRevision":0.
"MinorRevision":705}}}.
"PIDClientRights":{"LicenseSource":0.
"LicenseSource2":0.
"IsSubscriptionBased":false.
"DaysSinceCreation":-1.
"LicenseType":2.
"AuthorizedVsEdition":2.
"AuthorizedVsEdition2":300.
"DaysToExpire":-1.
"IsLicenseGood":false.
"LicenseDescription":"30 day trial (for evaluation purposes only)".
"LicenseStatus":-1.
"LicenseUrl":"".
"MayUplevel":false.
"LicensedUserId":"00000000-0000-0000-0000-000000000000".
"LicensedUserName":"".
"LicensingAccount":"".
"ReleaseType":1}}; Account identity provider: AAD</description>

@Gernatch
Copy link

Gernatch commented Jun 17, 2019

@SSE4 I tried to delete my reply. Wasn't quick enough. Somehow my VS key became deactivated... or I was switched to a 30day MSDN subscription without realizing it. Sorry for the confusion.

I am seeing the netstandard error that was fixed on PR #89. That is the real cause of the issue. I assume this will be a non issue when this is all merged together.

PS: Will beautify future output ;P

@SSE4
Copy link
Author

SSE4 commented Jun 17, 2019

@Gernatch so to confirm, does it work for you with just #89 applied, or only with both #89 + #75?

@Gernatch
Copy link

Gernatch commented Jun 17, 2019

@SSE4 Each individually. I install the #89 using the VSIX, it works. I do the same for #75, it does not. Note again I'm running a VS2017 version.

@jgsogo
Copy link

jgsogo commented Jun 18, 2019

So, you are not blocked by this PR which is targeting VS2015, right?

I think we can release v1.1 (main change is #89) this week and leave #85 for the next one (it is one of the major/breaking changes I think we need to implement asap).

@kkripa84
Copy link

@SSE4 I see issues with this extension when I installed in VS2017 15.0.0 +26228.57 version.

image

After install, I clicked continue with the error message

Error I see in my activitylog

Could not load file or assembly 'Microsoft.VisualStudio.Threading, Version=15.6.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified. File name: &

Assembly binding logging is turned OFF.

@jgsogo jgsogo modified the milestones: next, 1.1 Jun 24, 2019
@jgsogo
Copy link

jgsogo commented Jun 24, 2019

@SSE4, can you merge dev, please. We will try to merge this for v1.1 too. Thanks

@SSE4 SSE4 changed the base branch from master to dev June 24, 2019 14:56
@SSE4
Copy link
Author

SSE4 commented Jun 24, 2019

@jgsogo done

@jgsogo
Copy link

jgsogo commented Jun 25, 2019

Thanks! I'll test it in clean Windows + VS2015 installation. If anyone else wants to try, here it is the link to the artifact: https://ci.appveyor.com/api/buildjobs/0ch7aaf8wak0ngym/artifacts/Conan.VisualStudio%2Fbin%2FRelease%2FConan.VisualStudio.vsix

@jgsogo
Copy link

jgsogo commented Jun 25, 2019

Tested with:

  • VS 2015 Community Update 3, installed in a VM from scratch only with Visual C++. It works 👌
  • VS 2019, it doesn't run 😞

Copy link

@jgsogo jgsogo left a comment

Choose a reason for hiding this comment

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

VS2019 fails in my machine 😞 , but VS2015 works

@Gernatch
Copy link

Kripa (above) tried installing this today. It seems like it loaded, but all functionality seems to be disabled. @kkripa84 is that right?

@jgsogo
Copy link

jgsogo commented Jun 25, 2019

This fails casting the Project to a VCProject (we need to convert it to a C++ type project to get its configuration and so). The VCProject relies on the VCProjectEngine.dll which is different from version to version (at least I found them to be incompatible between 14 and 16, which is the change needed from VS2015 to VS2019).

Not a bug: https://developercommunity.visualstudio.com/content/problem/568/systeminvalidcastexception-unable-to-cast-com-obje.html

There are some ideas to follow:

@jgsogo
Copy link

jgsogo commented Jun 25, 2019

Kripa (above) tried installing this today. It seems like it loaded, but all functionality seems to be disabled. @kkripa84 is that right?

Hi @Gernatch the problem here is summarized in this post from the MSFT team, we cannot have a single extension compatible with VS2015 and VS2017/9 (at least using the current approach).

@jgsogo jgsogo modified the milestones: 1.1.0, next Jun 25, 2019
@LBHawk
Copy link

LBHawk commented Jun 25, 2019

To clarify, we (myself, @Gernatch, @kkripa84) are not blocked by this PR for vs2015. I think we should keep the vs2017 issues we have (which are blocking for us) separate from this PR.

@jgsogo jgsogo modified the milestones: next, 1.2.0 Jun 27, 2019
@jgsogo jgsogo modified the milestones: 1.2.0, next Jul 19, 2019
@jgsogo jgsogo closed this in #154 Aug 7, 2019
@czoido czoido deleted the fix_vs2015 branch June 19, 2024 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[vs2015] Error loading in VS 2015
5 participants