-
Notifications
You must be signed in to change notification settings - Fork 314
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
Copy aspect files to workspace #7202
Conversation
c65289c
to
78d12ee
Compare
9f748b1
to
012d8a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @LeFrosch! I left some comments. I am not familiar with the template related parts if you would like someone else to review those.
30482fb
to
84ca923
Compare
rebased master |
The AspectStorageService is responsible for writing the aspect files into a directory contained in the workspace root. The AspectWriter extension allows to register writers that write individual parts of the aspect to this directory.
Bazel 5 seems to have problems with relative labels used in the aspect. This also means that the plugin is no longer compatible with Bazel 5. # Conflicts: # MODULE.bazel.lock
To ensure compatibility when executing the jars during the aspect execution, they have to be built with java 8.
e34f8f7
to
c9a34d1
Compare
c9a34d1
to
a92cc8e
Compare
@LeFrosch I think there might be a small bug somewhere in this. In our internal repo, this ends up passing |
@srdo-humio Thanks for reporting this, but as far as I understand this should be legal label since in the |
@LeFrosch Bazel 7.4.1 at least seems to not like it. I'm getting this in the console:
which then causes the sync to fail entirely Here's what's in the directory, just to rule out issues with that:
But you're right, the target syntax appears correct. Edit: And I checked, it is definitely the |
Sorry, I see the problem. We had |
Ah perfect, thanks for reporting this anyway. Might be good to know in future. |
Copy all aspects into the workspace instead of using a
--override_repository
or--inject_repository
flag to point to the bundled aspects. This provides some benefits:--override_repository
--inject_repository
is also not requiredSince aspects need to be copied to the workspace anyway, we can now store them inside the plugin jar. This makes accessing the aspects easier, since they are not in different locations for tests and production.
Also adds aspect writing as a dedicated sync step: