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.

Sign up for GitHub

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

Jump to bottom

Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarkus has InlineBeforeAnalysisBuildItem #2416

Closed
JiriOndrusek opened this issue Mar 30, 2021 · 2 comments · Fixed by #2722
Closed

Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarkus has InlineBeforeAnalysisBuildItem #2416

JiriOndrusek opened this issue Mar 30, 2021 · 2 comments · Fixed by #2722
Assignees
@ppalaga
Labels
housekeeping
Milestone
2.0.0-M2

Comments

@JiriOndrusek
Copy link
Contributor

There is a requirement to provide a compiler flag -H:+InlineBeforeAnalysis for native mode of spring-rabbitmq component.
Flag should be replaced with buildItem once following feature (quarkusio/quarkus#16117) is implemented in quarkus.

(Another solution is to remove spring-rabbitmq, if there is a new rabbitmq component, which is not based on the spring framework)
@JiriOndrusek JiriOndrusek changed the title Spring-rabbitmq remove required flag once Quarkus contains a proper buildItem Spring-rabbitmq - remove required flag once Quarkus contains a proper buildItem Mar 30, 2021
@JiriOndrusek JiriOndrusek self-assigned this Mar 30, 2021
@JiriOndrusek JiriOndrusek mentioned this issue Mar 30, 2021
Merged
@ppalaga
Copy link
Contributor

ppalaga commented May 26, 2021

InlineBeforeAnalysisBuildItem is now available.

@ppalaga ppalaga added this to the 2.0.0 milestone May 26, 2021
@ppalaga ppalaga modified the milestones: 2.0.0, 2.0.0-M2 Jun 4, 2021
@ppalaga ppalaga changed the title Spring-rabbitmq - remove required flag once Quarkus contains a proper buildItem Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarkus has InlineBeforeAnalysisBuildItem Jun 4, 2021
@ppalaga
Copy link
Contributor

ppalaga commented Jun 4, 2021

A fix is underway

@ppalaga ppalaga assigned ppalaga and unassigned JiriOndrusek Jun 4, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 4, 2021
@ppalaga
Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarku…
a3617e9 
…s has InlineBeforeAnalysisBuildItem apache#2416
@ppalaga ppalaga mentioned this issue Jun 4, 2021
Merged
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 4, 2021
@ppalaga
Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarku…
04451ae 
…s has InlineBeforeAnalysisBuildItem apache#2416
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 4, 2021
@ppalaga
Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarku…
1d89d9d 
…s has InlineBeforeAnalysisBuildItem apache#2416
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 6, 2021
@ppalaga
Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarku…
8b5229e 
…s has InlineBeforeAnalysisBuildItem apache#2416
ppalaga added a commit that referenced this issue Jun 7, 2021
@ppalaga
Remove -H:+InlineBeforeAnalysis from Spring RabbitMQ test once Quarku…
639e4ad 
…s has InlineBeforeAnalysisBuildItem #2416
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ppalaga ppalaga

Labels
housekeeping
Projects
None yet
Milestone
2.0.0-M2
Development

Successfully merging a pull request may close this issue.

Upgrade to Quarkus 2.0.0.CR3 ppalaga/camel-quarkus
2 participants
@ppalaga @JiriOndrusek