Replace custom set library by Skylib's sets (#1799)

Author: evertedsphere

.gitignore

@@ -3,3 +3,5 @@
 *.swp
 *.swo
 __pycache__
+/.direnv
+/.envrc

haskell/c2hs.bzl

@@ -18,6 +18,7 @@ load(
     "target_unique_name",
 )
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 load(":private/version_macros.bzl", "version_macro_includes")
 
 def _c2hs_library_impl(ctx):
@@ -62,7 +63,7 @@ def _c2hs_library_impl(ctx):
     ]
     args.add_all(chi_includes)
 
-    version_macro_headers = set.empty()
+    version_macro_headers = sets.make()
     if ctx.attr.version:
         dep_info = gather_dep_info(ctx.attr.name, ctx.attr.deps)
         (version_macro_headers, version_macro_flags) = version_macro_includes(dep_info)
@@ -78,7 +79,7 @@ def _c2hs_library_impl(ctx):
             depset(cc.files),
             depset(hs.toolchain.bindir),
             depset(hs.toolchain.libdir),
-            set.to_depset(version_macro_headers),
+            depset(sets.to_list(version_macro_headers)),
             inputs,
         ]),
         input_manifests = input_manifests,

haskell/cabal.bzl

@@ -21,6 +21,7 @@ load(
     "truly_relativize",
 )
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 load(":private/validate_attrs.bzl", "typecheck_stackage_extradeps")
 load(":haddock.bzl", "generate_unified_haddock_info")
 load(
@@ -582,11 +583,11 @@ def _haskell_cabal_library_impl(ctx):
     hs_info = HaskellInfo(
         package_databases = depset([package_database], transitive = [dep_info.package_databases]),
         empty_lib_package_databases = dep_info.empty_lib_package_databases,
-        version_macros = set.empty(),
+        version_macros = sets.make(),
         source_files = depset(),
         boot_files = depset(),
         extra_source_files = depset(),
-        import_dirs = set.empty(),
+        import_dirs = sets.make(),
         hs_libraries = depset(
             direct = [lib for lib in [vanilla_library, dynamic_library, profiling_library] if lib],
             transitive = [dep_info.hs_libraries],
@@ -881,11 +882,11 @@ def _haskell_cabal_binary_impl(ctx):
     hs_info = HaskellInfo(
         package_databases = dep_info.package_databases,
         empty_lib_package_databases = dep_info.empty_lib_package_databases,
-        version_macros = set.empty(),
+        version_macros = sets.make(),
         source_files = depset(),
         boot_files = depset(),
         extra_source_files = depset(),
-        import_dirs = set.empty(),
+        import_dirs = sets.make(),
         hs_libraries = dep_info.hs_libraries,
         deps_hs_libraries = dep_info.deps_hs_libraries,
         empty_hs_libraries = dep_info.empty_hs_libraries,
@@ -1933,7 +1934,7 @@ def _stack_snapshot_impl(repository_ctx):
         else:
             visibility = sorted(
                 # use set to de-duplicate
-                set.to_list(set.from_list([
+                sets.to_list(sets.make([
                     str(vendored_packages[rdep].relative(":__pkg__"))
                     for rdep in reverse_deps[name]
                     if rdep in vendored_packages

haskell/doctest.bzl

@@ -4,6 +4,7 @@ load("@bazel_skylib//lib:dicts.bzl", "dicts")
 load(":cc.bzl", "cc_interop_info", "ghc_cc_program_args")
 load(":private/context.bzl", "haskell_context", "render_env")
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 load(
     "@rules_haskell//haskell:providers.bzl",
     "HaskellCcLibrariesInfo",
@@ -121,7 +122,7 @@ def _haskell_doctest_single(target, ctx):
 
     if ctx.attr.modules:
         inputs = ctx.attr.modules
-        args.add_all(set.to_list(hs_info.import_dirs), format_each = "-i%s")
+        args.add_all(sets.to_list(hs_info.import_dirs), format_each = "-i%s")
     else:
         inputs = [source.path for source in hs_info.source_files.to_list()]
 

haskell/experimental/private/module.bzl

@@ -1,4 +1,5 @@
 load("@bazel_skylib//lib:paths.bzl", "paths")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 load("//haskell:private/path_utils.bzl", "infer_main_module")
 load(
     "//haskell:private/dependencies.bzl",
@@ -840,7 +841,7 @@ def build_haskell_modules(
         repl_info = struct(
             source_files = depset(source_files),
             boot_files = depset(boot_files),
-            import_dirs = set.from_list(import_dirs),
+            import_dirs = sets.make(import_dirs),
             user_compile_flags = user_compile_flags,
         ),
     )

haskell/haddock.bzl

@@ -17,6 +17,7 @@ load(
 )
 load(":private/context.bzl", "haskell_context", "render_env")
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 
 def generate_unified_haddock_info(this_package_id, this_package_haddock, this_package_html, deps):
     """Collapse dependencies into a single `HaddockInfo`.

haskell/private/actions/compile.bzl

@@ -25,6 +25,7 @@ load(
     "link_libraries",
 )
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 load("//haskell/experimental:providers.bzl", "HaskellModuleInfo")
 load(
     ":private/actions/process_hsc_file.bzl",
@@ -161,7 +162,7 @@ def _compilation_defaults(
     # the two must both have the same root; i.e., both plain files,
     # both in bin_dir, or both in genfiles_dir.
 
-    import_dirs = set.from_list([
+    import_dirs = sets.make([
         hs.src_root,
         paths.join(hs.bin_dir.path, hs.src_root),
         paths.join(hs.genfiles_dir.path, hs.src_root),
@@ -176,15 +177,15 @@ def _compilation_defaults(
         elif s.extension == "hsc":
             s0, idir = process_hsc_file(hs, cc, hsc_flags, hsc_inputs, s)
             source_files.append(s0)
-            set.mutable_insert(import_dirs, idir)
+            sets.insert(import_dirs, idir)
         elif s.extension in ["hs-boot", "lhs-boot"]:
             boot_files.append(s)
         else:
             source_files.append(s)
 
         if s in import_dir_map:
             idir = import_dir_map[s]
-            set.mutable_insert(import_dirs, idir)
+            sets.insert(import_dirs, idir)
 
     # Write the -optP flags to a parameter file because they can be very long on Windows
     # e.g. 27Kb for grpc-haskell

haskell/private/actions/process_hsc_file.bzl

@@ -4,6 +4,7 @@ load("@bazel_skylib//lib:paths.bzl", "paths")
 load(":private/version_macros.bzl", "version_macro_includes")
 load(":private/path_utils.bzl", "declare_compiled")
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 
 def process_hsc_file(hs, cc, hsc_flags, hsc_inputs, hsc_file):
     """Process a single hsc file.
@@ -108,6 +109,6 @@ def preprocess_hsc_flags_and_inputs(dep_info, user_compile_flags, version):
     if version:
         (version_macro_headers, version_macro_flags) = version_macro_includes(dep_info)
         hsc_flags += ["--cflag=" + x for x in version_macro_flags]
-        hsc_inputs += set.to_list(version_macro_headers)
+        hsc_inputs += sets.to_list(version_macro_headers)
 
     return hsc_flags, hsc_inputs

haskell/private/actions/runghc.bzl

@@ -17,6 +17,7 @@ load(
     "link_libraries",
 )
 load("@bazel_skylib//lib:shell.bzl", "shell")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 
 def build_haskell_runghc(
         hs,
@@ -56,7 +57,7 @@ def build_haskell_runghc(
     )
 
     if lib_info != None:
-        for idir in set.to_list(hs_info.import_dirs):
+        for idir in sets.to_list(hs_info.import_dirs):
             args += ["-i{0}".format(idir)]
 
     link_libraries(

haskell/private/dependencies.bzl

@@ -4,6 +4,7 @@ load(
     "HaskellLibraryInfo",
 )
 load(":private/set.bzl", "set")
+load("@bazel_skylib//lib:sets.bzl", "sets")
 
 def gather_dep_info(name, deps):
     """Collapse dependencies into a single `HaskellInfo`.
@@ -64,7 +65,7 @@ def gather_dep_info(name, deps):
         if HaskellInfo in dep
     ])
 
-    import_dirs = set.empty()
+    import_dirs = sets.make()
     for dep in deps:
         if HaskellInfo in dep:
             import_dirs = set.mutable_union(import_dirs, dep[HaskellInfo].import_dirs)
@@ -83,7 +84,7 @@ def gather_dep_info(name, deps):
     acc = HaskellInfo(
         package_databases = package_databases,
         empty_lib_package_databases = empty_lib_package_databases,
-        version_macros = set.empty(),
+        version_macros = sets.make(),
         hs_libraries = hs_libraries,
         deps_hs_libraries = deps_hs_libraries,
         empty_hs_libraries = empty_hs_libraries,