-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-23698 Switch central repo to https #1053
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
Conversation
b68df72 to
65f2556
Compare
|
💔 -1 overall
This message was automatically generated. |
65f2556 to
fbe6943
Compare
|
💔 -1 overall
This message was automatically generated. |
| <snapshots> | ||
| <enabled>false</enabled> | ||
| </snapshots> | ||
| </repository> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked the parent apache pom (v22) to see if it had this repo in it. It doesn't. Just has apache repo.
This looks good. When you clean your local repo, are you downloading from https?
Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, verified that.
|
Its for branch-1 only? |
bharathv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its for branch-1 only?
Right now it looks like only branch-1 is affected, probably because branch-1 jobs use an older version of maven (we don't log the version anywhere to confirm that). Like I noted in the jira, mvn versions >= 3.2.3 automatically use https. So folks building locally with latest mvn versions won't hit this issue either.
| <snapshots> | ||
| <enabled>false</enabled> | ||
| </snapshots> | ||
| </repository> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, verified that.
|
I ran into this and thought the build failure was related to HBASE-23675. |
petersomogyi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I verified the build using Maven 3.0.5.
Signed-off-by: Peter Somogyi <[email protected]>
Signed-off-by: Peter Somogyi <[email protected]>
Signed-off-by: Peter Somogyi <[email protected]> (cherry picked from commit 8cb278e)
No description provided.