Skip to content

Use tree artifacts in bootclasspath rule#15384

Closed
fmeum wants to merge 1 commit intobazelbuild:masterfrom
fmeum:bootloader-tree-artifacts
Closed

Use tree artifacts in bootclasspath rule#15384
fmeum wants to merge 1 commit intobazelbuild:masterfrom
fmeum:bootloader-tree-artifacts

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented May 2, 2022

Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.

@fmeum fmeum changed the title Use tree artifacts in bootclasspath rule WIP: Use tree artifacts in bootclasspath rule May 2, 2022
@ckolli5 ckolli5 added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label May 2, 2022
@ckolli5 ckolli5 requested a review from coeuvre May 2, 2022 14:52
@fmeum fmeum force-pushed the bootloader-tree-artifacts branch from 853e084 to c87938e Compare May 4, 2022 14:55
@fmeum fmeum changed the title WIP: Use tree artifacts in bootclasspath rule Use tree artifacts in bootclasspath rule May 4, 2022
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented May 4, 2022

@cushon This is a follow-up to #6393, which is close to being fixed. It is currently stacked on #15366 and marked as draft as long as that PR hasn't been merged, but if you have capacity, you could already review it.

The reason I'm interested in making this change is ongoing work on a structured path remapping approach to #6526 (comment) which requires forgoing .path, .dirname and .root in favor of add_all structured command-line features. With this change, bootclasspath would qualify.

@fmeum fmeum marked this pull request as ready for review May 4, 2022 15:01
@fmeum fmeum requested a review from a team as a code owner May 4, 2022 15:01
@fmeum fmeum marked this pull request as draft May 4, 2022 15:01
Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.
@fmeum fmeum force-pushed the bootloader-tree-artifacts branch from c87938e to 765ec3e Compare May 9, 2022 13:26
@fmeum fmeum marked this pull request as ready for review May 9, 2022 13:26
@fmeum
Copy link
Copy Markdown
Collaborator Author

fmeum commented May 9, 2022

@cushon The PRs this was based on have been merged, ready for review now.

@bazel-io bazel-io closed this in ae523f8 May 10, 2022
fmeum added a commit to fmeum/bazel that referenced this pull request May 19, 2022
Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.

Closes bazelbuild#15384.

PiperOrigin-RevId: 447759377
fmeum added a commit to fmeum/bazel that referenced this pull request May 19, 2022
Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.

Closes bazelbuild#15384.

PiperOrigin-RevId: 447759377
@fmeum fmeum deleted the bootloader-tree-artifacts branch July 6, 2022 06:47
fmeum added a commit to fmeum/bazel that referenced this pull request Jul 6, 2022
Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.

Closes bazelbuild#15384.

PiperOrigin-RevId: 447759377
ckolli5 added a commit that referenced this pull request Jul 7, 2022
Remote execution now creates the directories for output tree artifacts
prior to execution, so that the output of the Java compiler can be
modelled as a tree artifact even though not all versions create the
output directory if it doesn't exist.

Closes #15384.

PiperOrigin-RevId: 447759377

Co-authored-by: Chenchu K <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants