Skip to content

[boost-modular-build-helper] Specify msvc version in user-config.jam#15137

Merged
ras0219-msft merged 1 commit intomicrosoft:masterfrom
ras0219-msft:dev/roschuma/boost-build-custom-msvc
Dec 16, 2020
Merged

[boost-modular-build-helper] Specify msvc version in user-config.jam#15137
ras0219-msft merged 1 commit intomicrosoft:masterfrom
ras0219-msft:dev/roschuma/boost-build-custom-msvc

Conversation

@ras0219-msft
Copy link
Copy Markdown
Contributor

This PR fixes an issue with generated user-config.jam files for non-system-wide copies of MSVC. If the version of MSVC is not specified, boost-build assumes an extremely old version and fails to build.

Source: boost-modular-build-helper
Version: 1.74.0
Port-Version: 1
Port-Version: 2
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.

Does generate-ports.ps1 need to be updated?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think we need to update in generate-ports.ps1 since when we update the boost to new version, the Port-version would be removed or set to 0.

@PhoebeHui PhoebeHui self-assigned this Dec 16, 2020
@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Dec 16, 2020
@PhoebeHui PhoebeHui added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Dec 16, 2020
@ras0219-msft ras0219-msft merged commit 981e65c into microsoft:master Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants