Skip to content

Ensure kotlin compiler includes method param names#12641

Closed
ruifigueira wants to merge 4 commits intospring-projects:masterfrom
ruifigueira:bug/kotlinc-java-parameters
Closed

Ensure kotlin compiler includes method param names#12641
ruifigueira wants to merge 4 commits intospring-projects:masterfrom
ruifigueira:bug/kotlinc-java-parameters

Conversation

@ruifigueira
Copy link
Copy Markdown
Contributor

@ruifigueira ruifigueira commented Mar 26, 2018

Configures kotlin-maven-plugin to include the -java-parameters
argument, so that method parameter names are included in bytecode.

See gh-9323
Fixes gh-12640

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 26, 2018
@philwebb philwebb added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 27, 2018
@philwebb philwebb added this to the 2.0.x milestone Mar 27, 2018
@philwebb philwebb mentioned this pull request Mar 27, 2018
Closed
@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Mar 27, 2018

Thanks for the PR, @ruifigueira. I suspect we need to do the same in Gradle too. Would you like to confirm that and make the necessary changes as part of this pull request?

@wilkinsona wilkinsona added the status: waiting-for-feedback We need additional information before we can continue label Mar 27, 2018
@ruifigueira
Copy link
Copy Markdown
Contributor Author

ruifigueira commented Mar 27, 2018

Sure, @wilkinsona. I found a similar PR (#9839) which is a great help, so I'll change KotlinPluginAction accordingly.

@snicoll snicoll added status: on-hold We can't start working on this issue yet and removed status: waiting-for-feedback We need additional information before we can continue labels Mar 27, 2018
@ruifigueira
Copy link
Copy Markdown
Contributor Author

ruifigueira commented Mar 27, 2018

Gradle plugin now enables -java-parameters by default.
I also included some documentation.

@ruifigueira
Ensure kotlin compiler includes method param names
e1181d0 
Configures kotlin-maven-plugin to include the `-java-parameters`
argument, so that method parameter names are included in bytecode.

See gh-9323
Fixes gh-12640
@ruifigueira
Enable javaParameters in kotlin plugin for gradle by default
a5a1a28 
This way, java parameters names are included in bytecode and can be
retrieved using reflection.

Fixes gh-12640
@ruifigueira
Add documentation about kotlin -java-parameters
0562479 
This commit documents the need of `-java-parameters` flag when compiling
kotlin to ensure `Endpoint` method parameters names are available, as
well as informing that `spring-boot-gradle-plugin` / maven with
`spring-boot-starter-parent` do exactly that by default when
activated.

Fixes gh-12640
@ruifigueira
Replace kotlin-maven-plugin args with javaParameters
8f44b1f 
Using `javaParameters` configuration parameter in `kotlin-maven-plugin`
instead of args: it is cleaner and users can add `args` to the plugin
configuration without having to worry about adding `-java-parameters`

Fixes gl-12640
@snicoll snicoll removed the status: on-hold We can't start working on this issue yet label Mar 28, 2018
@wilkinsona wilkinsona modified the milestones: 2.0.x, 2.0.1 Mar 28, 2018
@wilkinsona wilkinsona self-assigned this Mar 28, 2018
@ruifigueira
Copy link
Copy Markdown
Contributor Author

ruifigueira commented Mar 28, 2018

Concourse CI doesn't seem to like me :) I was looking into those build errors and they are from Spring Boot Server Tests:

[ERROR] nestedMetaInfResourceIsAvailableViaServletContext[Undertow: exploded jar](org.springframework.boot.context.embedded.EmbeddedServletContainerJarPackagingIntegrationTests)  Time elapsed: 4.007 s  <<< ERROR!
java.lang.NoClassDefFoundError: org/springframework/web/client/RestTemplate$AcceptHeaderRequestCallback

and in another job, it is failing due to a Github Pull Request under build-deployment-tests > git-pull-request:

/usr/lib/ruby/gems/2.4.0/gems/httpclient-2.8.3/lib/httpclient/session.rb:625:in `rescue in create_socket': getaddrinfo: Try again (api.github.com:443) (Faraday::ConnectionFailed)

I noticed that the first one is also occuring in the official Spring Boot master branch, so I'm assuming it has nothing to do with my changes.

wilkinsona pushed a commit that referenced this pull request Mar 28, 2018
@ruifigueira @wilkinsona
Configure Kotlin compiler to use -java-parameters by default
5e2726f 
See gh-12641
wilkinsona added a commit that referenced this pull request Mar 28, 2018
@wilkinsona
Merge pull request #12641 from Rui Figueira
bf4c572 
* gh-12641:
  Polish “Configure Kotlin compiler to use -java-parameters by default”
  Configure Kotlin compiler to use -java-parameters by default
@wilkinsona
Copy link
Copy Markdown
Member

wilkinsona commented Mar 28, 2018

Thanks very much for making your first contribution to Spring Boot, @ruifigueira. Especially one that was so complete. Sorry about the CI failures. I believe they were caused by a corrupted jar file from one of our snapshot dependencies.

I've merged your changes into master along with a little bit of polish. Thanks again.

@snicoll snicoll mentioned this pull request Mar 4, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@wilkinsona wilkinsona

Labels

type: enhancement A general enhancement

Projects

None yet

Milestone

2.0.1

Development

Successfully merging this pull request may close these issues.

Kotlin method parameter names are not included in bytecode

5 participants

@ruifigueira @wilkinsona @snicoll @philwebb @spring-projects-issues
⚡