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

🐛 Test cases for #390 #427

Merged
merged 5 commits into from
Nov 21, 2023
Merged

🐛 Test cases for #390 #427

merged 5 commits into from
Nov 21, 2023

Conversation

ascheman
Copy link
Contributor

@ascheman ascheman commented Nov 15, 2023

Adding test cases for #390.

The Java example was refactored to make it run (i.e., mvn compile at least), and extended by a submodule.
The submodule contains a property based dependency version which cannot be immediately resolved, as there is no properties definition in the submodule itself.

The hotfix provided by #425 prevents the binary from failing.
Nevertheless, the version property should be resolved from the parent pom which is not yet addressed.

As the bug report only was about the SEGV caused by the problem, #390 may be closed anyway.

@jmle
Copy link
Contributor

jmle commented Nov 17, 2023

@ascheman could you sign all the commits?

@ascheman ascheman force-pushed the bugfix/390-workaround branch 3 times, most recently from 5438186 to 19b0ac8 Compare November 20, 2023 12:48
Signed-off-by: Gerd Aschemann <gerd@aschemann.net>
Signed-off-by: Gerd Aschemann <gerd@aschemann.net>
Signed-off-by: Gerd Aschemann <gerd@aschemann.net>
…eyor#420).

Signed-off-by: Gerd Aschemann <gerd@aschemann.net>
Signed-off-by: Gerd Aschemann <gerd@aschemann.net>
@ascheman ascheman force-pushed the bugfix/390-workaround branch from 19b0ac8 to 38090d6 Compare November 20, 2023 14:43
@ascheman ascheman marked this pull request as ready for review November 20, 2023 14:48
@jmle jmle self-requested a review November 21, 2023 11:14
Copy link
Contributor

@jmle jmle left a comment

Choose a reason for hiding this comment

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

Looks 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.

2 participants