Skip to content

Conversation

@dtolnay
Copy link
Contributor

@dtolnay dtolnay commented Apr 20, 2022

Previously running bazel build ... in examples/cargo_manfest_dir/external_crate would leave the following untracked files:

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        examples/cargo_manifest_dir/external_crate/bazel-bin
        examples/cargo_manifest_dir/external_crate/bazel-external_crate
        examples/cargo_manifest_dir/external_crate/bazel-out
        examples/cargo_manifest_dir/external_crate/bazel-testlogs

I noticed that of the 5 workspaces found by find . -name WORKSPACE.bazel three of them were ignoring bazel-* via the top-level .gitignore while one had its own .gitignore containing only bazel-*. I consolidated that one into the top-level .gitignore and added the missing 5th.

Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dtolnay dtolnay force-pushed the ignore branch 3 times, most recently from 5e26162 to e03d67d Compare April 29, 2022 19:44
@UebelAndre
Copy link
Collaborator

@dtolnay Hey, if you rebase this I'd be happy to merge it 😄

…nore

Previously running `bazel build ...` in examples/cargo_manfest_dir/external_crate
would leave the following untracked files:

    Untracked files:
      (use "git add <file>..." to include in what will be committed)
            examples/cargo_manifest_dir/external_crate/bazel-bin
            examples/cargo_manifest_dir/external_crate/bazel-external_crate
            examples/cargo_manifest_dir/external_crate/bazel-out
            examples/cargo_manifest_dir/external_crate/bazel-testlogs

I noticed that of the 5 workspaces found by `find . -name WORKSPACE.bazel`
three of them were ignoring "bazel-*" via the top-level .gitignore while
one had its own .gitignore containing only "bazel-*". I consolidated
that one into the top-level .gitignore and added the 5th.
@dtolnay
Copy link
Contributor Author

dtolnay commented May 4, 2022

Rebased.

@UebelAndre UebelAndre merged commit 61eee54 into bazelbuild:main May 4, 2022
@dtolnay dtolnay deleted the ignore branch May 4, 2022 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants