Skip to content

Conversation

tjwatson
Copy link
Member

@tjwatson tjwatson commented Sep 6, 2025

@tjwatson
Copy link
Member Author

tjwatson commented Sep 6, 2025

!build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link
Member

LibbyBot commented Sep 6, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 8 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 4 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@tjwatson
Copy link
Member Author

tjwatson commented Sep 8, 2025

@volosied The commit in dba98a6 is pretty straight forward. But I am unsure what test to add. Do you know what tests exercise this code that I can add to?

@tjwatson
Copy link
Member Author

tjwatson commented Sep 8, 2025

!build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link
Member

LibbyBot commented Sep 8, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 9 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 4 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

Copy link
Member

@volosied volosied left a comment

Choose a reason for hiding this comment

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

Looks simple enough for JSP -- overrideLibraryRef will add additional jars the classpath to allow overridding as you described.

@tjwatson tjwatson merged commit a6f18af into OpenLiberty:integration Sep 9, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants