Skip to content

Instrument IPhoneBuilder to break down build time statistics #4313

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.

Sign up for GitHub

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

Merged
shai-almog merged 1 commit into from
Dec 19, 2025
Merged

Instrument IPhoneBuilder to break down build time statistics #4313

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
50 changes: 48 additions & 2 deletions maven/codenameone-maven-plugin/src/main/java/com/codename1/builders/IPhoneBuilder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -213,6 +213,7 @@ private int getDeploymentTargetInt(BuildRequest request) {

@Override
public boolean build(File sourceZip, BuildRequest request) throws BuildException {
Stopwatch stopwatch = new Stopwatch();
addMinDeploymentTarget(DEFAULT_MIN_DEPLOYMENT_VERSION);
detectJailbreak = request.getArg("ios.detectJailbreak", "false").equals("true");
defaultEnvironment.put("LANG", "en_US.UTF-8");
Expand Down Expand Up @@ -346,7 +347,7 @@ public boolean build(File sourceZip, BuildRequest request) throws BuildException
} catch (IOException ex) {
throw new BuildException("Failed to unzip source Zip file.", ex);
}

stopwatch.split("Setup & Unzip");


// We allow devs to add local podspecs inside a folder called "podspecs". This will
Expand Down Expand Up @@ -479,6 +480,7 @@ public boolean build(File sourceZip, BuildRequest request) throws BuildException

}
}
stopwatch.split("Extract Extensions");

File podSpecs = new File(tmpFile, "podspecs");
podSpecs.mkdirs();
Expand Down Expand Up @@ -563,6 +565,7 @@ public boolean build(File sourceZip, BuildRequest request) throws BuildException


}
stopwatch.split("Google Services Setup");

if (googleClientId == null && useGoogleSignIn) {
log("GoogleService-Info.plist file specifies that GoogleSignIn should be used but it doesn't provide a client ID. Likely the GoogleService-Info.plist file is not valid. See "+GOOGLE_SIGNIN_TUTORIAL_URL+" for instructions on setting up GoogleSignIn");
Expand Down Expand Up @@ -603,7 +606,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failed to scan project classes for permissions.", ex);
}

stopwatch.split("Scan Classes");

debug("Local Notifications "+(usesLocalNotifications?"enabled":"disabled"));
try {
Expand Down Expand Up @@ -646,6 +649,7 @@ public void usesClassMethod(String cls, String method) {
} catch (IOException ex) {
throw new BuildException("Failed to extract nativeios.jar",ex);
}
stopwatch.split("Extract Libs");

if(request.getArg("noExtraResources", "false").equals("true")) {
new File(buildinRes, "CN1Resource.res").delete();
Expand Down Expand Up @@ -910,6 +914,7 @@ public void usesClassMethod(String cls, String method) {
adPadding = " Display.getInstance().setProperty(\"adPaddingBottom\", \"" + adPadding + "\");\n";
}
}
stopwatch.split("Inject Build Hints");

File stubSource = new File(tmpFile, "stub");
stubSource.mkdirs();
Expand All @@ -918,6 +923,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failed to generate Unit Test Files", ex);
}
stopwatch.split("Generate Unit Tests");

String newStorage = "";
if(request.getArg("ios.newStorageLocation", "true").equals("true")) {
Expand Down Expand Up @@ -1005,6 +1011,7 @@ public void usesClassMethod(String cls, String method) {
} catch (IOException ex) {
throw new BuildException("Failed to write stub source", ex);
}
stopwatch.split("Generate Stubs");

Class[] nativeInterfaces = getNativeInterfaces();
if(nativeInterfaces != null && nativeInterfaces.length > 0) {
Expand Down Expand Up @@ -1173,6 +1180,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failure occurred while compiling native interface stubs", ex);
}
stopwatch.split("Compile Stubs");


try {
Expand All @@ -1190,6 +1198,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failed to generate launch screen");
}
stopwatch.split("Generate Icons");

resultDir = new File(tmpFile, "result");
resultDir.mkdirs();
Expand Down Expand Up @@ -1526,6 +1535,7 @@ public void usesClassMethod(String cls, String method) {
debug("iosDeploymentTargetMajorVersionInt="+getDeploymentTargetInt(request));

debug("Building using addLibs="+addLibs);
stopwatch.split("Prepare ParparVM");
try {
if (!exec(userDir, env, 420000, "java", "-DsaveUnitTests=" + isUnitTestMode(), "-DfieldNullChecks=" + fieldNullChecks, "-DINCLUDE_NPE_CHECKS=" + includeNullChecks, "-DbundleVersionNumber=" + bundleVersionNumber, "-Xmx384m",
"-jar", parparVMCompilerJar, "ios",
Expand All @@ -1543,6 +1553,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failure while trying to run ByteCodeTranslator of ParparVM", ex);
}
stopwatch.split("ParparVM Execution");
try {
String orientations = request.getArg("ios.interface_orientation", null);
if (orientations != null && orientations.split(":").length < 4) {
Expand Down Expand Up @@ -1617,6 +1628,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failed to update infoplist file", ex);
}
stopwatch.split("Post-VM Setup");
if (runPods) {
try {
List<File> podSpecFileList = new ArrayList<File>();
Expand Down Expand Up @@ -1932,6 +1944,7 @@ public void usesClassMethod(String cls, String method) {
} catch (Exception ex) {
throw new BuildException("Failed to generate PodFile", ex);
}
stopwatch.split("CocoaPods");
}

try {
Expand Down Expand Up @@ -1979,6 +1992,9 @@ public void usesClassMethod(String cls, String method) {

}

stopwatch.split("Finalize");
stopwatch.stop();

if ("xcode".equals(getBuildTarget()) || getBuildTarget() == null) {
xcodeProjectDir = new File(tmpFile, "dist");
return true;
Expand Down Expand Up @@ -2151,6 +2167,36 @@ private class GoogleServicePlist {

}

private class Stopwatch {
private long startTime;
private long lastSplitTime;
private final StringBuilder summary = new StringBuilder();

public Stopwatch() {
startTime = System.currentTimeMillis();
lastSplitTime = startTime;
summary.append("Build Time Statistics:\n");
summary.append("----------------------\n");
}

public void split(String stepName) {
long now = System.currentTimeMillis();
long duration = now - lastSplitTime;
String message = String.format("%-40s : %d ms", stepName, duration);
log(message);
summary.append(message).append("\n");
lastSplitTime = now;
}

public void stop() {
long now = System.currentTimeMillis();
long totalDuration = now - startTime;
summary.append("----------------------\n");
summary.append(String.format("%-40s : %d ms", "Total Time", totalDuration));
log(summary.toString());
}
}

private File[] extractAppExtensions(File sourceDirectory, File targetDirectory) throws IOException {
if (sourceDirectory == null || !sourceDirectory.isDirectory()) {
throw new IllegalArgumentException("extractAppExtensions sourceDirectory must be an existing directory but received "+sourceDirectory);
Expand Down
Loading