Skip to content

tests: allow Eigen repo and version to be overridden#3324

Merged
rwgk merged 2 commits intopybind:masterfrom
rwgk:bugfix_std_free_eigen_off
Oct 4, 2021
Merged

tests: allow Eigen repo and version to be overridden#3324
rwgk merged 2 commits intopybind:masterfrom
rwgk:bugfix_std_free_eigen_off

Conversation

@rwgk
Copy link
Copy Markdown
Collaborator

@rwgk rwgk commented Oct 3, 2021

To work around gitlab eigen outage.

The eigen repo is still unavailable (seems to be 2+ days now):

Screen Shot 2021-10-04 at 06 11 00

@rwgk rwgk force-pushed the bugfix_std_free_eigen_off branch from daa6f41 to f32ffec Compare October 3, 2021 22:54
@rwgk rwgk changed the title Clone of PR #3321 with all -DDOWNLOAD_EIGEN=OFF All -DDOWNLOAD_EIGEN=OFF (temporarily). Oct 3, 2021
@rwgk rwgk marked this pull request as ready for review October 4, 2021 13:12
@rwgk rwgk requested a review from henryiii as a code owner October 4, 2021 13:12
@rwgk rwgk requested a review from Skylion007 October 4, 2021 13:12
@rwgk
Copy link
Copy Markdown
Collaborator Author

rwgk commented Oct 4, 2021

It's very easy to roll back. Better than driving blind while the eigen problem is being worked on.

@henryiii henryiii force-pushed the bugfix_std_free_eigen_off branch from f32ffec to 489b81d Compare October 4, 2021 16:32
@henryiii henryiii changed the title All -DDOWNLOAD_EIGEN=OFF (temporarily). tests: allow Eigen repo and version to be overridden Oct 4, 2021
@henryiii
Copy link
Copy Markdown
Collaborator

henryiii commented Oct 4, 2021

Will need a rebase and merge, since we want to roll back the second commit after they have the original repo restored.

set(PYBIND11_EIGEN_REPO
"https://gitlab.com/libeigen/eigen.git"
CACHE STRING "Eigen repository to use for tests")
# This hash is for 3.3.8, using a hash for security reasons
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically, this is also faster on re-configure, too, since it doesn't have to reach out again.

@rwgk rwgk merged commit 9f146a5 into pybind:master Oct 4, 2021
@rwgk rwgk deleted the bugfix_std_free_eigen_off branch October 4, 2021 17:06
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Oct 4, 2021
@rwgk rwgk removed the needs changelog Possibly needs a changelog entry label Oct 4, 2021
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