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

Correct jpa sunappclientxml #1582

Merged
merged 21 commits into from
Nov 7, 2024

Conversation

scottmarlow
Copy link
Contributor

Applied change via:

find -name *Stateless3Test*.java | while read -r file; do
    tac $file | sed "/application-client/ {s//sun-application-client/; :loop; n; b loop}" | tac > $file.out;
    cp $file.out $file;
    rm $file.out
done

find -name *Stateful3Test*.java | while read -r file; do
    tac $file | sed "/application-client/ {s//sun-application-client/; :loop; n; b loop}" | tac > $file.out;
    cp $file.out $file;
    rm $file.out
done

find -name *Appmanaged*.java | while read -r file; do
    tac $file | sed "/application-client/ {s//sun-application-client/; :loop; n; b loop}" | tac > $file.out;
    cp $file.out $file;
    rm $file.out
done

Need to test...

@scottmarlow scottmarlow force-pushed the correct_jpa_sunappclientxml branch from c5dca4c to 2f1c2a1 Compare October 14, 2024 20:24
@scottmarlow scottmarlow force-pushed the correct_jpa_sunappclientxml branch from 59c3ad5 to b00b843 Compare November 5, 2024 14:39
…arate

commits for the same

This reverts commit 4476976.
…ess3/stateless3_vehicle_ejb.xml) to instead use /com/sun/ts/tests/common/vehicle/stateless3/stateless3_vehicle_client.xml
…ateful3/stateful3_vehicle_ejb.xml) to instead use URL ejbResURL1 = Client.class.getResource(/com/sun/ts/tests/common/vehicle/stateful3/stateful3_vehicle_client.xml)
…d use //com/sun/ts/tests/common/vehicle/appmanaged/appmanaged_vehicle_client.xml
…o instead use //com/sun/ts/tests/common/vehicle/appmanagedNoTx/appmanagedNoTx_vehicle_client.xml
…ar.xml to //com/sun/ts/tests/common/vehicle/stateless3/stateless3_vehicle_ejb.jar.sun-ejb-jar.xml
….xml to /com/sun/ts/tests/common/vehicle/stateful3/stateful3_vehicle_ejb.jar.sun-ejb-jar.xml
…ar.xm to //com/sun/ts/tests/common/vehicle/appmanaged/appmanaged_vehicle_ejb.jar.sun-ejb-jar.xml
…un-ejb-jar.xml to //com/sun/ts/tests/common/vehicle/appmanagedNoTx/appmanagedNoTx_vehicle_ejb.jar.sun-ejb-jar.xml
….persistence.common.schema30.Spouse should use consistence get/setter/fieldNames"

This reverts commit be72476.
Signed-off-by: Scott Marlow <smarlow@redhat.com>
@scottmarlow scottmarlow force-pushed the correct_jpa_sunappclientxml branch from b00b843 to f3091a0 Compare November 6, 2024 17:35
@scottmarlow scottmarlow marked this pull request as ready for review November 7, 2024 14:05
@scottmarlow scottmarlow merged commit c175f38 into jakartaee:main Nov 7, 2024
2 checks passed
@scottmarlow scottmarlow deleted the correct_jpa_sunappclientxml branch November 7, 2024 14:05
@scottmarlow scottmarlow restored the correct_jpa_sunappclientxml branch November 8, 2024 01:35
@scottmarlow
Copy link
Contributor Author

scottmarlow commented Nov 8, 2024

Some of these changes are causing failures after trying David's workaround. Will need to look further into which one.

f3091a0 seems to be good.

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.

1 participant