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

Update eigen3's portfile to use new gitlab repo #9821

Merged
merged 4 commits into from Jan 30, 2020

Conversation

traversaro
Copy link
Contributor

@traversaro traversaro commented Jan 26, 2020

As part of the migration from BitBucket to GitLab (see http://eigen.tuxfamily.org/index.php?title=News:Migration_to_GitLab.com_scheduled_on_the_December_4th) the official GitHub mirror at https://github.com/eigenteam/eigen-git-mirror is deprecated and soon will be removed, so to avoid problems it is better to already migrate the eigen3's port to the new repo.

@cenit
Copy link
Contributor

cenit commented Jan 27, 2020

somehow the git repository has been rebuilt?!?
In any case the SHA has been modified for the same reference, so you have to update it:

4cc3717b9cbe78335e05f724919497214edd482d4812aeb1a9fd6da5b3f6d1b194bb93ed0dab9e734b4334a5b88e8f8c339c43c1b2044332286ef5e758f9ecf4

@traversaro
Copy link
Contributor Author

somehow the git repository has been rebuilt?!?
In any case the SHA has been modified for the same reference, so you have to update it:

4cc3717b9cbe78335e05f724919497214edd482d4812aeb1a9fd6da5b3f6d1b194bb93ed0dab9e734b4334a5b88e8f8c339c43c1b2044332286ef5e758f9ecf4

Thanks, I prepared the PR in an environment with reduced network, so I could not check that, thank for the hash.

@vicroms vicroms added the info:reviewed Pull Request changes follow basic guidelines label Jan 30, 2020
@vicroms
Copy link
Member

vicroms commented Jan 30, 2020

/azp run

@vicroms
Copy link
Member

vicroms commented Jan 30, 2020

Thanks for the PR!

@vicroms vicroms merged commit 1046c29 into microsoft:master Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants