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

Unify all maven urls #968

Merged
merged 1 commit into from
Jan 28, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 7 additions & 22 deletions WORKSPACE
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ load("//scala:scala.bzl", "scala_repositories")

scala_repositories()

load("//scala:scala_cross_version.bzl", "default_maven_server_urls")
load("//scala:scala_maven_import_external.bzl", "scala_maven_import_external")
load("//twitter_scrooge:twitter_scrooge.bzl", "scrooge_scala_library", "twitter_scrooge")

Expand Down Expand Up @@ -48,10 +49,7 @@ scalafmt_repositories()

load("//scala:scala_cross_version.bzl", "default_scala_major_version", "scala_mvn_artifact")

MAVEN_SERVER_URLS = [
"https://jcenter.bintray.com",
"https://repo1.maven.org/maven2",
]
MAVEN_SERVER_URLS = default_maven_server_urls()

# test adding a scala jar:
jvm_maven_import_external(
Expand Down Expand Up @@ -106,9 +104,7 @@ scala_maven_import_external(
artifact_sha256 = "4eb582bc99d96c8df92fc6f0f608fd123d278223982555ba16219bf8be9f75a9",
fetch_sources = True,
licenses = ["notice"],
server_urls = [
"https://repo.maven.apache.org/maven2/",
],
server_urls = MAVEN_SERVER_URLS,
srcjar_sha256 = "5e586357a289f5fe896f7b48759e1c16d9fa419333156b496696887e613d7a19",
)

Expand Down Expand Up @@ -151,10 +147,7 @@ scala_maven_import_external(
artifact_sha256 = "972139718abc8a4893fa78cba8cf7b2c903f35c97aaf44fa3031b0669948b480",
fetch_sources = True,
licenses = ["notice"], # Apache 2.0
server_urls = [
"https://repo1.maven.org/maven2/",
"https://mirror.bazel.build/repo1.maven.org/maven2",
],
server_urls = MAVEN_SERVER_URLS,
srcjar_sha256 = "b186965c9af0a714632fe49b33378c9670f8f074797ab466f49a67e918e116ea",
)

Expand Down Expand Up @@ -224,21 +217,15 @@ scala_maven_import_external(
artifact = "org.springframework:spring-core:5.1.5.RELEASE",
artifact_sha256 = "f771b605019eb9d2cf8f60c25c050233e39487ff54d74c93d687ea8de8b7285a",
licenses = ["notice"], # Apache 2.0
server_urls = [
"https://repo1.maven.org/maven2/",
"https://mirror.bazel.build/repo1.maven.org/maven2",
],
server_urls = MAVEN_SERVER_URLS,
)

scala_maven_import_external(
name = "org_springframework_spring_tx",
artifact = "org.springframework:spring-tx:5.1.5.RELEASE",
artifact_sha256 = "666f72b73c7e6b34e5bb92a0d77a14cdeef491c00fcb07a1e89eb62b08500135",
licenses = ["notice"], # Apache 2.0
server_urls = [
"https://repo1.maven.org/maven2/",
"https://mirror.bazel.build/repo1.maven.org/maven2",
],
server_urls = MAVEN_SERVER_URLS,
deps = [
"@org_springframework_spring_core",
],
Expand All @@ -254,7 +241,5 @@ scala_maven_import_external(
artifact_sha256 = "897460d4488b7dd6ac9198937d6417b36cc6ec8ab3693fdf2c532652f26c4373",
fetch_sources = False,
licenses = ["notice"],
server_urls = [
"https://repo.maven.apache.org/maven2/",
],
server_urls = MAVEN_SERVER_URLS,
)
6 changes: 5 additions & 1 deletion jmh/jmh.bzl
Original file line number Diff line number Diff line change
@@ -1,10 +1,14 @@
load("//scala:scala.bzl", "scala_binary", "scala_library")
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
)
load(
"@io_bazel_rules_scala//scala:scala_maven_import_external.bzl",
_scala_maven_import_external = "scala_maven_import_external",
)

def jmh_repositories(maven_servers = ["https://repo.maven.apache.org/maven2"]):
def jmh_repositories(maven_servers = _default_maven_server_urls()):
_scala_maven_import_external(
name = "io_bazel_rules_scala_org_openjdk_jmh_jmh_core",
artifact = "org.openjdk.jmh:jmh-core:1.20",
Expand Down
6 changes: 5 additions & 1 deletion junit/junit.bzl
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
)
load(
"@io_bazel_rules_scala//scala:scala_maven_import_external.bzl",
_scala_maven_import_external = "scala_maven_import_external",
)

def junit_repositories(maven_servers = ["https://repo.maven.apache.org/maven2"]):
def junit_repositories(maven_servers = _default_maven_server_urls()):
_scala_maven_import_external(
name = "io_bazel_rules_scala_junit_junit",
artifact = "junit:junit:4.12",
Expand Down
6 changes: 2 additions & 4 deletions scala/private/macros/scala_repositories.bzl
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
load(
"@io_bazel_rules_scala//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
_default_scala_version = "default_scala_version",
_default_scala_version_jar_shas = "default_scala_version_jar_shas",
_extract_major_version = "extract_major_version",
Expand Down Expand Up @@ -56,10 +57,7 @@ def scala_repositories(
_default_scala_version(),
_default_scala_version_jar_shas(),
),
maven_servers = [
"https://repo.maven.apache.org/maven2",
"https://maven-central.storage-download.googleapis.com/maven2",
],
maven_servers = _default_maven_server_urls(),
scala_extra_jars = _default_scala_extra_jars()):
(scala_version, scala_version_jar_shas) = scala_version_shas
major_version = _extract_major_version(scala_version)
Expand Down
8 changes: 8 additions & 0 deletions scala/scala_cross_version.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,14 @@ def default_scala_version_jar_shas():
"scala_reflect": "6ba385b450a6311a15c918cf8688b9af9327c6104f0ecbd35933cfcd3095fe04",
}

def default_maven_server_urls():
return [
"https://repo.maven.apache.org/maven2",
"https://maven-central.storage-download.googleapis.com/maven2",
"https://mirror.bazel.build/repo1.maven.org/maven2",
"https://jcenter.bintray.com",
]

def extract_major_version(scala_version):
"""Return major Scala version given a full version, e.g. "2.11.11" -> "2.11" """
return scala_version[:scala_version.find(".", 2)]
Expand Down
10 changes: 5 additions & 5 deletions scala/scalafmt/scalafmt_repositories.bzl
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
)
load(
"@io_bazel_rules_scala//scala:scala_maven_import_external.bzl",
_scala_maven_import_external = "scala_maven_import_external",
Expand All @@ -12,11 +16,7 @@ def scalafmt_default_config(path = ".scalafmt.conf"):
build.append(")")
native.new_local_repository(name = "scalafmt_default", build_file_content = "\n".join(build), path = "")

def scalafmt_repositories(
maven_servers = [
"https://repo.maven.apache.org/maven2",
"https://maven-central.storage-download.googleapis.com/maven2",
]):
def scalafmt_repositories(maven_servers = _default_maven_server_urls()):
_scala_maven_import_external(
name = "com_geirsson_metaconfig_core_2_11",
artifact = "com.geirsson:metaconfig-core_2.11:0.8.3",
Expand Down
3 changes: 2 additions & 1 deletion scala_proto/private/scala_proto_default_repositories.bzl
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
_default_scala_version = "default_scala_version",
_extract_major_version = "extract_major_version",
_scala_mvn_artifact = "scala_mvn_artifact",
Expand All @@ -11,7 +12,7 @@ load(

def scala_proto_default_repositories(
scala_version = _default_scala_version(),
maven_servers = ["https://repo.maven.apache.org/maven2"]):
maven_servers = _default_maven_server_urls()):
major_version = _extract_major_version(scala_version)

scala_jar_shas = {
Expand Down
3 changes: 2 additions & 1 deletion specs2/specs2.bzl
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
_default_scala_version = "default_scala_version",
_extract_major_version = "extract_major_version",
_scala_mvn_artifact = "scala_mvn_artifact",
Expand All @@ -14,7 +15,7 @@ def specs2_version():

def specs2_repositories(
scala_version = _default_scala_version(),
maven_servers = ["https://repo.maven.apache.org/maven2"]):
maven_servers = _default_maven_server_urls()):
major_version = _extract_major_version(scala_version)

scala_jar_shas = {
Expand Down
3 changes: 2 additions & 1 deletion specs2/specs2_junit.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ load(
load("//junit:junit.bzl", "junit_repositories")
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
_default_scala_version = "default_scala_version",
_extract_major_version = "extract_major_version",
_scala_mvn_artifact = "scala_mvn_artifact",
Expand All @@ -18,7 +19,7 @@ load(

def specs2_junit_repositories(
scala_version = _default_scala_version(),
maven_servers = ["https://repo.maven.apache.org/maven2"]):
maven_servers = _default_maven_server_urls()):
major_version = _extract_major_version(scala_version)

specs2_repositories(scala_version, maven_servers)
Expand Down
3 changes: 2 additions & 1 deletion twitter_scrooge/twitter_scrooge.bzl
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
load(
"//scala:scala_cross_version.bzl",
_default_maven_server_urls = "default_maven_server_urls",
_default_scala_version = "default_scala_version",
_extract_major_version = "extract_major_version",
_scala_mvn_artifact = "scala_mvn_artifact",
Expand All @@ -23,7 +24,7 @@ _jar_extension = ".jar"

def twitter_scrooge(
scala_version = _default_scala_version(),
maven_servers = ["https://repo.maven.apache.org/maven2"]):
maven_servers = _default_maven_server_urls()):
major_version = _extract_major_version(scala_version)

_scala_maven_import_external(
Expand Down