Skip to content

[PCL] Fix pcl#16792

Closed
larshg wants to merge 5 commits intomicrosoft:masterfrom
larshg:FixPCL
Closed

[PCL] Fix pcl#16792
larshg wants to merge 5 commits intomicrosoft:masterfrom
larshg:FixPCL

Conversation

@larshg
Copy link
Contributor

@larshg larshg commented Mar 19, 2021

Describe the pull request

@larshg larshg marked this pull request as ready for review March 19, 2021 22:44
@JackBoosY JackBoosY self-assigned this Mar 21, 2021
@JackBoosY JackBoosY added the category:port-bug The issue is with a library, which is something the port should already support label Mar 21, 2021
@JackBoosY
Copy link
Contributor

Does this PR depends on #16791 ?

@larshg
Copy link
Contributor Author

larshg commented Mar 22, 2021

Yes, for both to complete successfully - should I include #16791 in this one, to get all build?

@larshg larshg mentioned this pull request Mar 22, 2021
@JackBoosY
Copy link
Contributor

@larshg Yes if the changes of these two PRs depend on each other.

@larshg
Copy link
Contributor Author

larshg commented Mar 22, 2021

@JackBoosY #16724 makes PCL and OpenMVC not being build. I have created two PRs to fix those, as you wrote in #16724. Or should I include all changes in the original PR about FLANN, so all is green?

@JackBoosY
Copy link
Contributor

@larshg Yes please. We should make all the pipeline test pass.

@larshg
Copy link
Contributor Author

larshg commented Mar 23, 2021

Closing as it is included in #16724

@larshg larshg closed this Mar 23, 2021
@larshg larshg deleted the FixPCL branch May 5, 2021 15:02
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.

2 participants