Skip to content

Commit

Permalink
Update besom scripts (#396)
Browse files Browse the repository at this point in the history
- make azure-native compile in finite time
  • Loading branch information
pawelprazak authored Feb 22, 2024
1 parent 88e87b4 commit eb4eb25
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,9 @@ export GITHUB_TOKEN=$(gh auth token)
just clean-out cli packages maven-all
```

Cation: publishing to Maven Central is irreversible.
Tip: it's safer to publish the packages on-by-one or in batches due to how Maven Central behaves.
Note: `azure-native` publishing takes a long time (1-2 hours) it is recommended to handle it separately.

In case of any issues, you can try to resolve the issues manually at https://oss.sonatype.org/index.html#stagingRepositories.

Expand Down
4 changes: 2 additions & 2 deletions scripts/Packages.scala
Original file line number Diff line number Diff line change
Expand Up @@ -73,10 +73,12 @@ object Packages:
val publishedMavenFile = publishMavenDir / "published.json"

def compileOpts(heapMaxGb: Int = 32): Vector[os.Shellable] =
println(s"Compiling with max heap size: ${heapMaxGb}G")
Vector(
"--server=false",
"--javac-opt=-verbose",
s"--javac-opt=-J-XX:MaxHeapSize=${heapMaxGb}G",
"--javac-opt=-J-XX:NewRatio=1", // increase young vs old gen size, default is 2
"--javac-opt=-J-XX:+UseParallelGC"
)

Expand Down Expand Up @@ -128,8 +130,6 @@ object Packages:
private val codegenProblemPackages = blockedPackages ++ Vector()

private val compileProblemPackages = blockedPackages ++ Vector(
"azure-native", // does not compile in finite time
"azure-quickstart-acr-geo-replication", // depends on azure-native
"aws-iam", // id parameter, schema error - components should make this viable
"nuage" // id parameter, schema error - components should make this viable
)
Expand Down

0 comments on commit eb4eb25

Please sign in to comment.