Skip to content

Clean up modules and add 2.8.5 distribution with LFS support - #7

Open
pmanko wants to merge 2 commits into
mainfrom
cleanup-modules-and-add-lfs-init
Open

Clean up modules and add 2.8.5 distribution with LFS support#7
pmanko wants to merge 2 commits into
mainfrom
cleanup-modules-and-add-lfs-init

Conversation

@pmanko

@pmanko pmanko commented Dec 17, 2025

Copy link
Copy Markdown
Member

Summary

Cleans up legacy module directories and adds OpenMRS 2.8.5 module distribution with proper Git LFS tracking.

Changes

  • Removed old module directories:
    • openmrs_modules
    • openmrs_modules-2.8.4
    • openmrs_modules-legacy
  • Added openmrs_modules-2.8.5/ with 47 modules from 2.8.5 distribution
  • Removed broken LFS pointer files (5 files)
  • Added init-lfs.sh helper script for Git LFS initialization
  • Updated README.md with Git LFS setup instructions

All .omod files are tracked via Git LFS as configured in .gitattributes.

Testing

After merging, test from a clean clone:

  1. Clone repository
  2. Run ./init-lfs.sh to initialize Git LFS
  3. Verify modules are properly tracked

- Remove old module directories (openmrs_modules, openmrs_modules-2.8.4, openmrs_modules-legacy)
- Add openmrs_modules-2.8.5 with modules from 2.8.5-20251217T181405Z-1-001.zip
- Remove broken LFS pointer files
- Add init-lfs.sh helper script for Git LFS setup
- Update README.md with Git LFS initialization instructions

All .omod files are tracked via Git LFS as configured in .gitattributes
@ibacher

ibacher commented Dec 17, 2025

Copy link
Copy Markdown
Contributor

I'm just going to use this to add some notes:

One of the main things we did as part of this clean-up is to play with the Tomcat parameters. For Ubuntu, this turned into having a setenv.sh file like this:

JAVA_OPTS="-Djava.awt.headless=true -XX:+UseG1GC"
CATALINA_OPTS="-Xms8192m"

The current OpenMRS Docker images are built on the Tomcat Docker image which doesn't have the same defaults as Ubuntu some of which we needed to work-around.

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