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

Acknowledge TASTY as known ClassFile attribute in ijar #24165

Closed

Conversation

WojciechMazur
Copy link
Contributor

Currently producing ijar using Scala 3 produces the following warnings:

ijar: skipping unknown attribute: "TASTY".

TASTY is a Scala 3 specific ClassFile attribute and informs that for given .class file there is also emmited as .tasty file (these were handled in #12529). It can be consumed in the same way as currently handled Scala and ScalaSig attributes.
Handling that should remove redundant warnings

Here's example on how this attribute is emmited:
https://github.com/scala/scala3/blob/91ef92159c628eaeab8311dc82bed7ed4fe03c63/compiler/src/dotty/tools/backend/jvm/CodeGen.scala#L92-L110

@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Oct 31, 2024
@iancha1992 iancha1992 added the team-Rules-Server Issues for serverside rules included with Bazel label Oct 31, 2024
@fmeum
Copy link
Collaborator

fmeum commented Oct 31, 2024

@cushon

@fmeum fmeum added team-Rules-Java Issues for Java rules and removed team-Rules-Server Issues for serverside rules included with Bazel labels Oct 31, 2024
@WojciechMazur
Copy link
Contributor Author

PR was approved during the review. Is there anything left before merging it?
@fmeum @cushon

@fmeum
Copy link
Collaborator

fmeum commented Nov 6, 2024

@cushon Is this missing the awaiting-merge label or are you working on the import?

@cushon cushon added the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Nov 6, 2024
@copybara-service copybara-service bot closed this in 78afd0b Nov 6, 2024
@github-actions github-actions bot removed awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally awaiting-review PR is awaiting review from an assigned reviewer labels Nov 6, 2024
@WojciechMazur WojciechMazur deleted the ijar/accept-tasty branch November 6, 2024 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-Java Issues for Java rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants