Fabian Meumertzheim
Fabian Meumertzheim
Running `bazel test //src/test/shell/bazel:bazel_workspaces_test --test_output=streamed --test_filter=test_sparse_tar` on this PR with the patch below applied still fails for me with `Truncated TAR archive`: ```patch git diff | cat diff --git a/src/test/shell/bazel/bazel_workspaces_test.sh...
But this works, so it looks like we need to do more to pick up the fix: ```sh #!/usr/bin/env bash set -o errexit -o nounset echo "Downloading commons-compress" wget https://repo1.maven.org/maven2/org/apache/commons/commons-compress/1.26.1/commons-compress-1.26.1.jar...
@bazel-io fork 7.2.0
@lberki I implemented the solution you mentioned as your favorite one in https://github.com/bazelbuild/bazel/issues/19208#issuecomment-1689609485.
As a follow-up, I would also use this new logic in `HeaderDiscovery`. CC @sluongng
@lberki Gentle ping
@comius I have mostly given up on this for now. There are too many callers of `PathFragment` methods that make properly dealing with invalid paths almost impossible. I will wait...
@iancha1992 Could you add `awaiting-merge`?
As the author of https://github.com/bazelbuild/bazel/pull/14738, I would strongly suggest fixing this in Bazelisk. Many parts of Bazel use the architecture of the Bazel binary to reason about the host OS,...
@crorvick If this is still an issue, could you update your reproducer to demonstrate the problem you are seeing when using the runfiles library? A few issues related to missing...