Fix windows --watchfs

As discussed in #1931 --watchfs was made a no-op on windows in d04d7baa1605c8c3a637255bd61c00dba73ad800. The reason for this is that it will hold locks for the (parent-directories of) directories it is watching. Since we recursively register the watcher this leads to locks on folders we do not want to be locked.

The issue is known and tracked at https://bugs.openjdk.java.net/browse/JDK-6972833 but is marked as a Won't Fix. The suggested fix is to instead use ExtendedWatchEventModifier.FILE_TREE which allows a single WatchKey to recursively watch a folder on windows.

This pull request implements the FILE_TREE flag for the root folder that is being watched, and disables the registration of the recursive folders. I have tried to keep the change as small as possible, but I can imagine that we wish to separate this logic to dedicated windows class similar to the OSX variant.

Fixes #1931

Closes #11334.

PiperOrigin-RevId: 312285715
2 files changed
tree: 68df7ca68129d7a280a6a8b5c5d515019d1a2490
  1. .bazelci/
  2. examples/
  3. scripts/
  4. site/
  5. src/
  6. third_party/
  7. tools/
  8. .bazelrc
  9. .gitattributes
  10. .gitignore
  11. AUTHORS
  12. BUILD
  13. CHANGELOG.md
  14. CODEOWNERS
  15. combine_distfiles.py
  16. combine_distfiles_to_tar.sh
  17. compile.sh
  18. CONTRIBUTING.md
  19. CONTRIBUTORS
  20. distdir.bzl
  21. ISSUE_TEMPLATE.md
  22. LICENSE
  23. README.md
  24. WORKSPACE
README.md

Bazel

{Fast, Correct} - Choose two

Build and test software of any size, quickly and reliably.

  • Speed up your builds and tests: Bazel rebuilds only what is necessary. With advanced local and distributed caching, optimized dependency analysis and parallel execution, you get fast and incremental builds.

  • One tool, multiple languages: Build and test Java, C++, Android, iOS, Go, and a wide variety of other language platforms. Bazel runs on Windows, macOS, and Linux.

  • Scalable: Bazel helps you scale your organization, codebase, and continuous integration solution. It handles codebases of any size, in multiple repositories or a huge monorepo.

  • Extensible to your needs: Easily add support for new languages and platforms with Bazel's familiar extension language. Share and re-use language rules written by the growing Bazel community.

Getting Started

Documentation

Contributing to Bazel

See CONTRIBUTING.md

Build status