Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Update vxl master #686

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

dstoup
Copy link
Collaborator

@dstoup dstoup commented May 13, 2021

This patch updates VXL to today's master, hash 51558266cc28ffb522ae72eebc883a88124a4b6c.

I've updated all patches to reflect upstream changes, including removing the openjpeg patch which isn't needed any more since the issue has been resolved. I've also patched the vnl/io CMakeLists.txt to remove builds of vnl_io_real_polynomial and vnl_io_real_npolynomial on WIN32 platforms. It has a difficult link error but no packages, VXL included, use the io of those modules.

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@dstoup dstoup requested a review from mleotta May 14, 2021 19:26
@dstoup dstoup changed the title Update vxl master WIP: Update vxl master May 14, 2021
@dstoup
Copy link
Collaborator Author

dstoup commented May 14, 2021

FYI, I've marked this WIP for now. I have it building by itself and kwiver should build with almost all options on (some I can't build on Windows regardless) but I still need to make changes so vidtk will build. Plus, I'm assuming there is a fair amount of testing we should do to make sure the VXL master still behaves the same as our previous hash.

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

@kwcvrobot
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants