Skip to content

[openmvg] Update version to v2.0#21509

Closed
FrankXie05 wants to merge 16 commits intomicrosoft:masterfrom
FrankXie05:dev/Frank/openmvg
Closed

[openmvg] Update version to v2.0#21509
FrankXie05 wants to merge 16 commits intomicrosoft:masterfrom
FrankXie05:dev/Frank/openmvg

Conversation

@FrankXie05
Copy link
Contributor

Fix #21364

Update [openmvg] to the latest version 2.0

Note: No feature needs to test.

@JonLiu1993 JonLiu1993 added category:port-update The issue is with a library, which is requesting update new revision info:internal labels Nov 18, 2021
@cenit
Copy link
Contributor

cenit commented Nov 18, 2021

build_fixes.patch is still in the port folder even if not used anymore? why the patch rename, that was hiding the problem?

@FrankXie05
Copy link
Contributor Author

@cenit Thanks for your reminder, I will corrcet it right away. :)

@cenit
Copy link
Contributor

cenit commented Nov 25, 2021

now the situation is opposite. There is the new fix-build.patch that you introduced totally unused but still pushed in this PR. Please delete that file

@FrankXie05 FrankXie05 marked this pull request as ready for review November 26, 2021 02:06
@cenit
Copy link
Contributor

cenit commented Nov 26, 2021

now that graphviz has been merged, openmvg[tool] feature can depend on that, and you can remove the message info in the portfile (patch also the python scripts to use neato from vcpkg folder!)

#20564

@FrankXie05
Copy link
Contributor Author

This PR has been resolved by another PR #22020

@FrankXie05 FrankXie05 closed this Dec 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-update The issue is with a library, which is requesting update new revision

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[openmvg] update to 2.0

4 participants