Skip to content

[mlpack] Explicitly depend on stb#11785

Merged
ras0219-msft merged 1 commit intomicrosoft:masterfrom
ras0219:dev/roschuma/mlpack
Jun 9, 2020
Merged

[mlpack] Explicitly depend on stb#11785
ras0219-msft merged 1 commit intomicrosoft:masterfrom
ras0219:dev/roschuma/mlpack

Conversation

@ras0219
Copy link
Copy Markdown
Contributor

@ras0219 ras0219 commented Jun 5, 2020

mlpack installs /include/stb_image.h when built outside the presence of stb. This PR adds stb as an explicit dependency, ensuring the package does not conflict.

As a drive-by fix, this also disables git-based version detection which generates files into the source directory.

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    Yes

  • Does your PR follow the maintainer guide?
    Yes

@NancyLi1013 NancyLi1013 self-assigned this Jun 5, 2020
@NancyLi1013 NancyLi1013 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Jun 8, 2020
@ras0219-msft
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ras0219-msft ras0219-msft merged commit 09abe70 into microsoft:master Jun 9, 2020
cenit pushed a commit to cenit/vcpkg that referenced this pull request Jun 11, 2020
Co-authored-by: Robert Schumacher <roschuma@microsoft.com>
JangBoo pushed a commit to JangBoo/vcpkg that referenced this pull request Jun 18, 2020
Co-authored-by: Robert Schumacher <roschuma@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants