Remove UWP target and references; mark UWP as historical#4624
Merged
shai-almog merged 1 commit intomasterfrom Mar 19, 2026
Merged
Remove UWP target and references; mark UWP as historical#4624shai-almog merged 1 commit intomasterfrom
shai-almog merged 1 commit intomasterfrom
Conversation
…mplates ### Motivation - Remove the UWP (`windows-device`/`uwp`) build target from the Maven plugin, generated POM templates and helper scripts so generated projects and IDE shortcuts no longer advertise or attempt UWP builds. - Remove IDE/run-configuration shortcuts and CLI wrapper commands that referenced UWP so tooling/documentation matches supported targets. - Prevent the initializr `win` module from defaulting to a UWP build target so new projects do not try to submit UWP builds by default. ### Description - Deleted the UWP build wrapper class and mojo (`maven/codenameone-maven-plugin/src/main/java/com/codename1/maven/buildWrappers/BuildWindowsDeviceMojo.java`). - Removed the `windows-device` Ant target from the plugin template (`maven/codenameone-maven-plugin/src/main/resources/com/codename1/maven/buildxml-template.xml`) and removed `windows-device/uwp` entries from the IntelliJ `workspace.xml` template (`maven/codenameone-maven-plugin/src/main/resources/com/codename1/maven/intellij/workspace.xml`). - Removed `uwp`/`windows-device` profiles and mentions from initializr templates and generated POM fragments (`scripts/initializr/common/pom.xml` and the `common/src/main/resources/*-pom.xml` templates) so starter POMs no longer include a UWP profile. - Updated initializr helper scripts to drop `windows_device`/`uwp` commands from the Bash/Batch wrappers (`scripts/initializr/build.sh`, `scripts/initializr/build.bat`) and removed the UWP command/mentions from the CLI wrapper source and docs (`maven/cli/src/main/batsh/build.batsh`, `maven/cli/README.adoc`). - Changed the `initializr-win` module default build target to `none` (`scripts/initializr/win/pom.xml`) so activating that module does not implicitly select a UWP build. - Adjusted the CN1 build mojo documentation/comment to reflect the removed target from the documented list of supported build targets (`maven/codenameone-maven-plugin/src/main/java/com/codename1/maven/CN1BuildMojo.java`). ### Testing - Ran a repository search to validate remaining references: `rg -n "UWP|uwp|windows-device|buildWindowsUWP" maven/codenameone-maven-plugin maven/cli scripts/initializr -g '!**/target/**'` and confirmed the targeted UWP entries in the changed modules were removed. - Attempted to build the Maven plugin module with Java 8 disabled in this environment by running `mvn -f maven/pom.xml -pl codenameone-maven-plugin -am -DskipTests package`; the build started and progressed to module compilation in this environment (no plugin errors surfaced while packaging reached the plugin module stage). - Attempted to build the initializr module with `mvn -f scripts/initializr/pom.xml -DskipTests package`, which failed in this environment due to an unrelated network issue while downloading `UpdateCodenameOne.jar` (`java.net.SocketException: Network is unreachable`), so the full module package step could not be completed here.
Cloudflare Preview
|
|
Developer Guide build artifacts are available for download from this workflow run:
Developer Guide quality checks:
Unused image preview:
|
✅ Continuous Quality ReportTest & Coverage
Static Analysis
Generated automatically by the PR CI workflow. |
Collaborator
Author
|
Compared 32 screenshots: 32 matched. Native Android coverage
✅ Native Android screenshot tests passed. Native Android coverage
|
Collaborator
Author
iOS screenshot updatesCompared 32 screenshots: 30 matched, 2 updated.
Benchmark Results
Detailed Performance Metrics
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Description
7.0.229and removed UWP-specific instructions where appropriate.BuildWindowsDeviceMojo, removing thewindows-deviceAnt target and related entries inbuildxml-template.xml, and eliminatingwin/uwpprofiles and modules from initializr/pom templates.windows_device/uwpcommands, removing theinitializr-winmodule and its C# native files, and updatingmaven/clihelp/output to no longer list UWP build commands.CN1BuildMojoto stop advertisingwindows-devicein itscodename1.buildTargetdocumentation and removed related IntelliJ workspace/runner entries and project templates that targeted the UWP/device build.Testing
mvn -DskipTests packageto ensure updated pom/templates and plugin changes do not break packaging, and the build completed successfully.win/windows-deviceprofiles and that the CLIhelpoutput and scripts no longer list the removed UWP build commands by running the localbuild.*script variants.Codex Task