-
Notifications
You must be signed in to change notification settings - Fork 339
Copy aspect files to workspace #7202
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
Conversation
c65289c to
78d12ee
Compare
9f748b1 to
012d8a1
Compare
mai93
left a comment
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_repositoryor--inject_repositoryflag to point to the bundled aspects. This provides some benefits:--override_repository--inject_repositoryis 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:
