Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FISH-8510 Upgrade Patched Project to 2.7.15 #38

Open
wants to merge 18 commits into
base: eclipselink-2.7.15.payara-maintenance
Choose a base branch
from

Conversation

Viii3
Copy link
Member

@Viii3 Viii3 commented Oct 16, 2024

Tested and built successfully on Ubuntu VM with ClusterJSP deployment.

Notes:

  • Commit 59ac429: Dropped due to upstream having upgraded to ASM 9.7 instead.

David Weaver and others added 18 commits October 15, 2024 09:38
# Conflicts:
#	jpa/org.eclipse.persistence.jpa/src/org/eclipse/persistence/internal/jpa/metadata/accessors/mappings/RelationshipAccessor.java
# Conflicts:
#	foundation/org.eclipse.persistence.core/src/org/eclipse/persistence/platform/database/InformixPlatform.java
Work with the argumentIndices indices on a local copy, avoiding
concurrency issues, which caused NullPointerException. The other
threads were running copyTo() and assigned null to argumentIndices.
- some fixed parameters
- rename asm files before install (build names it differently than
required by install)
- add prepare functionality to create required libs in user's home
(todo for next time -- move them to ./target)
@breakponchito breakponchito self-requested a review October 19, 2024 00:24
Copy link

@breakponchito breakponchito left a comment

Choose a reason for hiding this comment

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

LGTM

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.

8 participants