Skip to content

Conversation

UjjawalPrabhat
Copy link

@UjjawalPrabhat UjjawalPrabhat commented Oct 5, 2025

Overview

Removes the redundant referenceapplication-demo directory from the OpenMRS distribution build by introducing a Maven Antrun plugin execution post-SDK build. This ensures the demo folder does not appear in the final artifacts, maintaining a clean and correct output.

Details

  • Adds a targeted Maven Antrun step in distro/pom.xml to safely delete the referenceapplication-demo folder after the build-distro phase.
  • Does not affect core metadata or demo content, which remains preserved via the referencedemodata module.

Linked Resource

Checklist

  • Build completes with no errors
  • referenceapplication-demo folder removed from output
  • Demo content is still functional (referencedemodata)
  • Scripts/documentation provide complete verification and reporting

…tro output

- Adds Maven Antrun plugin step to delete the redundant referenceapplication-demo folder after SDK build
- Ensures demo content is preserved via referencedemodata module
- Maintains integrity of build and application functionality
- Implements safe, robust, and maintainable solution as per verification and best practices
@dkayiwa
Copy link
Member

dkayiwa commented Oct 7, 2025

@UjjawalPrabhat what are the exact steps to reproduce the unwanted demo folder?

@UjjawalPrabhat
Copy link
Author

@UjjawalPrabhat what are the exact steps to reproduce the unwanted demo folder?

Thanks for reviewing! Here’s how to reproduce the unwanted referenceapplication-demo folder:

  1. Clone the repo and switch to the 3.4.0 release tag:
  1. Run the SDK build-distro goal:
  • mvn clean package openmrs-sdk:build-distro -P distro
  1. Inspect the generated files in:
  • target/sdk-distro/web/

You will see both:
referenceapplication/…
referenceapplication-demo/…

The referenceapplication-demo folder is the unwanted directory.

Let me know if you need any more details!

@dkayiwa
Copy link
Member

dkayiwa commented Oct 8, 2025

Are you able to reproduce it with the 3.6.0-SNAPSHOT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants