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

Weaver returns null instead of original bytes for unwoven classes #280

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

kriegaex
Copy link
Contributor

@kriegaex kriegaex commented Feb 7, 2024

This change makes sense independently of #277, but also enables using

-cp "my.jar;aspectjweaver.jar" -XX:+AllowArchivingWithJavaAgent -javaagent:aspectjweaver.jar

while creating a CDS archive. Afterward, the application can be run in its woven state from the CDS archive even without -javaagent, because the byte code was archived in its woven state ("poor man's AJC"). See #277 (comment) for details.

Fixes #277.

@kriegaex kriegaex self-assigned this Feb 7, 2024
@kriegaex kriegaex added the enhancement New feature or request label Feb 7, 2024
@kriegaex kriegaex added this to the 1.9.21.1 milestone Feb 7, 2024
@kriegaex kriegaex force-pushed the gh-277 branch 2 times, most recently from d934674 to 2fc1139 Compare February 7, 2024 04:30
This change makes sense independently of #277, but also enables using

  - cp "my.jar;aspectjweaver.jar"
  -XX:+AllowArchivingWithJavaAgent
  -javaagent:aspectjweaver.jar

while creating a CDS archive. Afterward, the application can be run in
its woven state from the CDS archive even without '-javaagent', because
the byte code was archived in its woven state ("poor man's AJC"). See
#277 (comment)
for details.

Fixes #277.

Signed-off-by: Alexander Kriegisch <[email protected]>
@kriegaex kriegaex merged commit 3e3c837 into master Feb 7, 2024
3 checks passed
@kriegaex kriegaex deleted the gh-277 branch February 7, 2024 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weaver should return null instead of the original bytes for unwoven classes
2 participants