Skip to content

Add switch to disable/enable javax-annotation processing for E4-Injector#820

Merged
HannesWell merged 1 commit intoeclipse-platform:masterfrom
HannesWell:javax-processing-switch
Nov 5, 2023
Merged

Add switch to disable/enable javax-annotation processing for E4-Injector#820
HannesWell merged 1 commit intoeclipse-platform:masterfrom
HannesWell:javax-processing-switch

Conversation

@HannesWell
Copy link
Copy Markdown
Member

The switch can be set via the VM property eclipse.e4.inject.javax.disabled.
For now the default value is 'false', which enables processing of 'javax' annotations by default.

Users can specify the VM argument -Declipse.e4.inject.javax.disabled=true in order to test an application without processing of 'javax' annotations now,

The switch can be set via the VM property
'eclipse.e4.inject.javax.disabled'.
For now the default value is 'false', which enables processing of
'javax' annotations by default.

Users can specify the VM argument
'-Declipse.e4.inject.javax.disabled=true'
in order to test an application without processing of 'javax'
annotations now,
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 4, 2023

Test Results

     591 files  ±0       591 suites  ±0   1h 7m 16s ⏱️ - 5m 20s
  3 799 tests ±0    3 794 ✔️ ±0    5 💤 ±0  0 ±0 
12 003 runs  ±0  11 967 ✔️ ±0  36 💤 ±0  0 ±0 

Results for commit c61b9ee. ± Comparison against base commit a1c46cd.

@akurtakov
Copy link
Copy Markdown
Member

I havent' tested the patch but from reading it - it looks good to me.

@HannesWell HannesWell merged commit 312651a into eclipse-platform:master Nov 5, 2023
@HannesWell HannesWell deleted the javax-processing-switch branch November 5, 2023 07:54
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