launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #32839
Re: [Merge] ~ruinedyourlife/launchpad:maven-explanation-and-logs into launchpad:master
Diff comments:
> diff --git a/lib/lp/crafts/model/craftrecipebuildjob.py b/lib/lp/crafts/model/craftrecipebuildjob.py
> index 0afc460..f3d939e 100644
> --- a/lib/lp/crafts/model/craftrecipebuildjob.py
> +++ b/lib/lp/crafts/model/craftrecipebuildjob.py
> @@ -462,6 +462,12 @@ class CraftPublishingJob(CraftRecipeBuildJobDerived):
> f.write(settings_xml)
>
> # Run mvn deploy using the pom file
> + env = os.environ.copy()
> + env["MAVEN_OPTS"] = (
> + "-Xmx128m -Xms128m "
> + "-XX:CompressedClassSpaceSize=32m "
s/avoid/afford/g
> + "-XX:MaxMetaspaceSize=64m"
> + )
> result = subprocess.run(
> [
> "mvn",
--
https://code.launchpad.net/~ruinedyourlife/launchpad/+git/launchpad/+merge/490254
Your team Launchpad code reviewers is requested to review the proposed merge of ~ruinedyourlife/launchpad:maven-explanation-and-logs into launchpad:master.
References