Skip to content

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Jan 30, 2025

Before this commit, the generated name for the inner class had the wrong format <package>.<parent>.<child> (canonical name). GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate an appropriate format for a class name. Specifically, an inner class should be separated by a dollar sign, not a dot.

See gh-44016

@nosan nosan changed the base branch from main to 3.4.x January 30, 2025 09:22
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 30, 2025
@nosan nosan force-pushed the 44016 branch 2 times, most recently from 2518f9f to 9935ac3 Compare January 30, 2025 09:25
Before this commit, the generated name for the inner class had the
wrong format <package>.<parent>.<child> (canonical name).
GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate
an appropriate format for a class name. Specifically, an inner class
should be separated by a dollar sign, not a dot.

See spring-projectsgh-44016

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
@wilkinsona wilkinsona changed the title Update SpringBootJoranConfigurator to use proper GraalVM format Logback configuration that relies on inner-classes does not work in a native image Jan 30, 2025
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 30, 2025
@wilkinsona wilkinsona added this to the 3.3.x milestone Jan 30, 2025
@wilkinsona wilkinsona self-assigned this Jan 30, 2025
@nosan
Copy link
Contributor Author

nosan commented Jan 30, 2025

Oh, I apologize for the mistake. The target branch should be 3.3.x. :(

@wilkinsona
Copy link
Member

wilkinsona commented Jan 30, 2025

No problem at all! You've done more than enough already. I'll take care of it as part of merging the fix.

@wilkinsona wilkinsona modified the milestones: 3.3.x, 3.3.9 Jan 30, 2025
wilkinsona pushed a commit that referenced this pull request Jan 30, 2025
Before this commit, the generated name for the inner class had the
wrong format <package>.<parent>.<child> (canonical name).
GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate
an appropriate format for a class name. Specifically, an inner class
should be separated by a dollar sign, not a dot.

See gh-44021

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants