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

VS2019 initial support #5864

Merged
merged 1 commit into from
Dec 10, 2018
Merged

VS2019 initial support #5864

merged 1 commit into from
Dec 10, 2018

Conversation

NN---
Copy link
Contributor

@NN--- NN--- commented Dec 7, 2018

Allow installation of VSIX in VS2019.
https://madskristensen.net/blog/how-to-upgrade-extensions-to-support-visual-studio-2019/

  • Git Extensions 3.00.00.4433
  • Build fca7cf2
  • Git 2.19.2.windows.1
  • Microsoft Windows NT 10.0.17763.0
  • .NET Framework 4.7.3260.0
  • DPI 120dpi (125% scaling)

@vbjay
Copy link
Contributor

vbjay commented Dec 7, 2018 via email

@NN---
Copy link
Contributor Author

NN--- commented Dec 7, 2018

Haven't tried it yet.
Usually extensions don't work if you don't allow installation for newer VS version.
This blog post is from VS extensions developer, he has a lot of experience in this area.

@gerhardol
Copy link
Member

Seem reasonable. How did you test it?

@gerhardol gerhardol added this to the 3.00.01 milestone Dec 7, 2018
@NN---
Copy link
Contributor Author

NN--- commented Dec 7, 2018

Installed VS2019 preview https://visualstudio.microsoft.com/vs/preview/
Running the VSIX I don't see option to integrate into 2019, only 2017 is listed.

@vbjay
Copy link
Contributor

vbjay commented Dec 7, 2018 via email

@codecov
Copy link

codecov bot commented Dec 10, 2018

Codecov Report

Merging #5864 into master will decrease coverage by 0.5%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #5864      +/-   ##
=========================================
- Coverage    37.3%   36.8%   -0.51%     
=========================================
  Files         618     618              
  Lines       47391   49172    +1781     
  Branches     6299    6836     +537     
=========================================
+ Hits        17679   18096     +417     
- Misses      28935   30259    +1324     
- Partials      777     817      +40

@RussKie RussKie merged commit 034fc25 into gitextensions:master Dec 10, 2018
@RussKie
Copy link
Member

RussKie commented Dec 10, 2018

Thank you

@gerhardol
Copy link
Member

I want to have this in the VSIX update for the market place (@RussKie, I added to the 3.00.01 milestone)
@NN--- Can you submit a PR for release/3.00 too?

@RussKie
Copy link
Member

RussKie commented Dec 10, 2018 via email

@nul800sebastiaan
Copy link

Is the VSIX available yet somewhere?

@vbjay
Copy link
Contributor

vbjay commented Jan 17, 2019

Note the milestone. We are working on releasing 3.0.1.

@nul800sebastiaan
Copy link

@vbjay Great, thanks! Now that 3.0.1 is out, where would I find the VISX though? :-) The MSI installer doesn't offer integration with VS19 and I can't find an update in the VS marketplace.

I've tried updating the old VSIX with the manifest changes from this PR but it refuses to install but it doesn't give me a good reason other than "prerequisites" that are missing.

image

@nul800sebastiaan
Copy link

nul800sebastiaan commented Jan 22, 2019

Alright, that was a typo on my part, got a little further now but it crashes on a missing reference to Microsoft.VisualStudio.Component.CoreEditor.

Manifest looks like:

<?xml version="1.0" encoding="utf-8"?>
<PackageManifest Version="2.0.0" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011">
    <Metadata>
        <Identity Publisher="GitExt Team" Version="2.50.2" Language="en-US" Id="GitExtensions..550013F4-337C-4750-8967-DDEEEC2EB497" />
        <DisplayName>GitExtensions</DisplayName>
        <Description xml:space="preserve" >Git Extensions is a graphical user interface for Git that allows you to control Git without using the command-line</Description>
        <MoreInfo>https://gitextensions.github.io/</MoreInfo>
        <License>Resources\LICENSE.md</License>
        <GettingStartedGuide>https://git-extensions-documentation.readthedocs.io/en/latest/</GettingStartedGuide>
        <Icon>Resources\git-extensions-logo-32px.png</Icon>
        <PreviewImage>Resources\git-extensions-logo-256px.png</PreviewImage>
        <Tags>Git, Git Extensions</Tags>
    </Metadata>
    <Installation>
        <InstallationTarget Version="[14.0,17.0)" Id="Microsoft.VisualStudio.Community" />
    </Installation>
    <Dependencies>
        <Dependency Id="Microsoft.Framework.NDP" DisplayName="Microsoft .NET Framework" Version="4.0" />
        <Dependency Id="Microsoft.VisualStudio.MPF.14.0" DisplayName="Visual Studio MPF 14.0" Version="[14.0,15.0)" />
    </Dependencies>
    <Assets>
		<Asset Type="Microsoft.VisualStudio.VsPackage" Path="GitExtensionsVSIX.pkgdef" />
    </Assets>
    <Prerequisites>
        <Prerequisite Id="Microsoft.VisualStudio.Component.CoreEditor" Version="[14.0,)" DisplayName="Visual Studio core editor" />
    </Prerequisites>
</PackageManifest>

@nul800sebastiaan
Copy link

nul800sebastiaan commented Jan 22, 2019

Aha! It was in the MSI file, just didn't offer an install from the MSI for some reason. If anyone else is looking for it, download and rename this file to GitExtensionsVSIX.vsix 👍

GitExtensionsVSIX.zip

Works! 🎉

image

@gerhardol
Copy link
Member

It was in the MSI file, just didn't offer an install from the MSI for some reason.

"Install" vsix feature just unpacks from the MSI, this as intended.
The file is distributed that way too, should be put on VS marketplace too by someone with privileges.
The .vsix will be removed from the msi, I want to deliver it as a separate Artifact too.

@vbjay vbjay mentioned this pull request Apr 3, 2019
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.

5 participants