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

Enable file version info in build #745

Merged
merged 2 commits into from
May 23, 2019
Merged

Enable file version info in build #745

merged 2 commits into from
May 23, 2019

Conversation

kmahone
Copy link
Member

@kmahone kmahone commented May 23, 2019

When we switched over from Package ES builds to regular Azure DevOps builds, we lost some functionality in Package ES that was automatically setting the version info in the built dll.

This version info is used to determine what version of WinUI a given app is using.

This PR updates the build to set the version information.

When building locally, the version information is taken from version.props (currently defined to be 2.2.0.0). When building in Azure Pipelines, the third and forth component of this version is overridden based on the date (YY, MM, DD) and the pipeline's revision number (RRR) so that we get version 2.2.YYMM.DDRRR
This closely matches the nupkg which is versioned as:
2.2.YYMMDDRRR

@kmahone kmahone requested a review from jevansaks May 23, 2019 00:44
@kmahone kmahone requested a review from a team as a code owner May 23, 2019 00:44
dev/dll/Microsoft.UI.Xaml.rc Show resolved Hide resolved
dev/dll/version.h Show resolved Hide resolved
version.props Outdated Show resolved Hide resolved
version.props Outdated Show resolved Hide resolved
@kmahone kmahone requested a review from llongley May 23, 2019 01:08
@jevansaks jevansaks added needs-cherrypicktorelease PR tagged for cherry-pick to the current release branch (but not yet picked) release note PR that we want to call out in the next release summary labels May 23, 2019
@kmahone
Copy link
Member Author

kmahone commented May 23, 2019

/azp run WinUI-Public-Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kmahone kmahone merged commit 03d39f9 into master May 23, 2019
@kmahone kmahone deleted the user/kmahone/filever branch May 23, 2019 20:06
@jevansaks jevansaks removed the needs-cherrypicktorelease PR tagged for cherry-pick to the current release branch (but not yet picked) label May 30, 2019
jevansaks pushed a commit that referenced this pull request May 30, 2019
When we switched over from Package ES builds to regular Azure DevOps builds, we lost some functionality in Package ES that was automatically setting the version info in the built dll.

This version info is used to determine what version of WinUI a given app is using.

This PR updates the build to set the version information.

When building locally, the version information is taken from version.props (currently defined to be 2.2.0.0). When building in Azure Pipelines, the third and forth component of this version is overridden based on the date (YY, MM, DD) and the pipeline's revision number (RRR) so that we get version 2.2.YYMM.DDRRR
This closely matches the nupkg which is versioned as:
2.2.YYMMDDRRR
@jevansaks jevansaks removed the needs-cherrypicktorelease PR tagged for cherry-pick to the current release branch (but not yet picked) label May 30, 2019
@msft-github-bot
Copy link
Collaborator

🎉Microsoft.UI.Xaml v2.1.190606001 has been released which incorporates this pull request.:tada:

Handy links:

@msft-github-bot
Copy link
Collaborator

🎉Microsoft.UI.Xaml v2.2.190611001 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release note PR that we want to call out in the next release summary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants