Skip to content

[eigen3] Fix include path#13739

Merged
BillyONeal merged 5 commits intomicrosoft:masterfrom
LilyWangL:dev/Lily/issue13397
Nov 6, 2020
Merged

[eigen3] Fix include path#13739
BillyONeal merged 5 commits intomicrosoft:masterfrom
LilyWangL:dev/Lily/issue13397

Conversation

@LilyWangL
Copy link
Contributor

Describe the pull request

@LilyWangL LilyWangL added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Sep 25, 2020
@LilyWangL LilyWangL marked this pull request as ready for review October 21, 2020 03:11
@petrasvestartas
Copy link

petrasvestartas commented Oct 21, 2020

@LilyWangL

I can confirm that this issue persists for other users.
I tried to follow the other issues you reference, but I do not understand the workaround.

Could explain how to reference Eigen for PCL?

image

@PhoebeHui PhoebeHui marked this pull request as draft October 22, 2020 07:31
@LilyWangL LilyWangL marked this pull request as ready for review November 4, 2020 08:13
@LilyWangL
Copy link
Contributor Author

@LilyWangL

I can confirm that this issue persists for other users.
I tried to follow the other issues you reference, but I do not understand the workaround.

Could explain how to reference Eigen for PCL?

image

Sorry I am late. You can modify your [VCPKG_path]/ports/eigen3/portfile.cmake according to my change and rebuild eigen3. Then I think you can rebuild your project successfully.

@LilyWangL
Copy link
Contributor Author

Baseline error will be fixed in PR #14400

@BillyONeal BillyONeal merged commit f754a36 into microsoft:master Nov 6, 2020
@BillyONeal
Copy link
Member

Thanks for your help!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[eigen3] installs headers in the wrong place since recent merge

4 participants