Skip to content

KFSPTS-37856 remove un-needed server references and convert to use central maven repo and github artifacts#1914

Open
jhulslander wants to merge 7 commits into
developfrom
KFSPTS-37856
Open

KFSPTS-37856 remove un-needed server references and convert to use central maven repo and github artifacts#1914
jhulslander wants to merge 7 commits into
developfrom
KFSPTS-37856

Conversation

@jhulslander

Copy link
Copy Markdown
Contributor

some new exclusions were added as they were libraries we don't need and there were problems being able to get them from Maven.
Please double check that the dependencies aren't needed.

@jhulslander

Copy link
Copy Markdown
Contributor Author

retest this please

@jhulslander jhulslander marked this pull request as ready for review June 12, 2026 23:09
@jhulslander

Copy link
Copy Markdown
Contributor Author

This is ready for review but please do NOT merge this yet. We need to cooridinate this change.
Each developer much update their Maven settings file. Jenkins needs to use the new settings file. The Develop branch, CEMI branch, and master branch will need the POM file changes to build

Please also review https://confluence.cornell.edu/x/WulgH paying attention to the update Maven instructions.

@cah292 cah292 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed that I was able to get KFS to build locally with this branch and the settings.xml file changes in place. (I ran out of time today to check whether KFS would launch, though.) I also updated the Confluence instructions to include Windows setup steps.

However, there were a few items I wanted to get clarity on, and it would be helpful to have another Mac developer test out this setup.

Comment thread pom.xml
Comment thread pom.xml
@cah292

cah292 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

My concerns have been addressed, so this can be merged once our team determines an appropriate time to move things forward. If another Mac developer wants to perform some local testing of this, we could also wait for that testing to be completed.

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