Skip to content

Commit 43bccaf

Browse files
committed
Fix overly-nested-depset warnings
1 parent 3180f8f commit 43bccaf

File tree

1 file changed

+18
-19
lines changed

1 file changed

+18
-19
lines changed

haskell/repl.bzl

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -143,10 +143,10 @@ def _data_runfiles(ctx, rule, attr):
143143
)
144144

145145
def _merge_HaskellReplLoadInfo(load_infos):
146-
source_files = depset()
147-
boot_files = depset()
148-
module_names = depset()
149-
import_dirs = depset()
146+
source_files = depset(transitive = [load_info.source_files for load_info in load_infos])
147+
boot_files = depset(transitive = [load_info.boot_files for load_info in load_infos])
148+
module_names = depset(transitive = [load_info.module_names for load_info in load_infos])
149+
import_dirs = depset(transitive = [load_info.import_dirs for load_info in load_infos])
150150
cc_libraries_infos = []
151151
cc_infos = []
152152
cc_shared_library_infos = []
@@ -156,10 +156,6 @@ def _merge_HaskellReplLoadInfo(load_infos):
156156
java_deps = []
157157

158158
for load_info in load_infos:
159-
source_files = depset(transitive = [source_files, load_info.source_files])
160-
boot_files = depset(transitive = [boot_files, load_info.boot_files])
161-
module_names = depset(transitive = [module_names, load_info.module_names])
162-
import_dirs = depset(transitive = [import_dirs, load_info.import_dirs])
163159
cc_libraries_infos.append(load_info.cc_libraries_info)
164160
cc_infos.append(load_info.cc_info)
165161
cc_shared_library_infos.extend(load_info.cc_shared_library_infos)
@@ -213,26 +209,26 @@ def _merge_HaskellReplLoadInfoMulti(root_info, load_infos):
213209
)
214210

215211
def _merge_HaskellReplDepInfo(dep_infos, dep_infos_for_package_dbs = []):
216-
package_ids = depset()
217-
package_databases = depset()
218-
interface_dirs = depset()
212+
package_ids = depset(transitive = [dep_info.package_ids for dep_info in dep_infos])
213+
package_databases = depset(transitive = [
214+
dep_info.package_databases
215+
for dep_info in dep_infos
216+
] + [
217+
dep_info.package_databases
218+
for dep_info in dep_infos_for_package_dbs
219+
])
220+
interface_dirs = depset(transitive = [dep_info.interface_dirs for dep_info in dep_infos])
219221
cc_libraries_infos = []
220222
cc_infos = []
221223
cc_shared_library_infos = []
222224
runfiles = []
223225

224226
for dep_info in dep_infos:
225-
package_ids = depset(transitive = [package_ids, dep_info.package_ids])
226-
package_databases = depset(transitive = [package_databases, dep_info.package_databases])
227-
interface_dirs = depset(transitive = [interface_dirs, dep_info.interface_dirs])
228227
cc_libraries_infos.append(dep_info.cc_libraries_info)
229228
cc_infos.append(dep_info.cc_info)
230229
cc_shared_library_infos.extend(dep_info.cc_shared_library_infos)
231230
runfiles.append(dep_info.runfiles)
232231

233-
for dep_info in dep_infos_for_package_dbs:
234-
package_databases = depset(transitive = [package_databases, dep_info.package_databases])
235-
236232
return HaskellReplDepInfo(
237233
direct_package_ids = [],
238234
package_ids = package_ids,
@@ -329,11 +325,10 @@ def _create_HaskellReplCollectInfo(target, dep_labels, dep_package_ids, dep_pack
329325
def _merge_HaskellReplCollectInfo(root_args, dep_args):
330326
load_infos = {}
331327
dep_infos = {}
332-
haskell_targets_root = depset()
328+
haskell_targets_root = depset(transitive = [arg.haskell_targets_postorder for arg in root_args])
333329
for arg in root_args:
334330
load_infos.update(arg.load_infos)
335331
dep_infos.update(arg.dep_infos)
336-
haskell_targets_root = depset(transitive = [haskell_targets_root, arg.haskell_targets_postorder])
337332

338333
transitive_targets = []
339334
for arg in dep_args:
@@ -457,6 +452,8 @@ def _create_HaskellMultiReplInfo(from_source, from_binary, collect_info):
457452
load_info = merged_load_info,
458453
dep_info = merged_dep_info,
459454
)
455+
456+
# buildifier: disable=overly-nested-depset
460457
label_order = depset(direct = [label], transitive = [label_order])
461458
repl_infos[label] = repl_info
462459

@@ -868,6 +865,8 @@ def _hie_bios_impl_multi(ctx):
868865
inputs,
869866
],
870867
)
868+
869+
# buildifier: disable=overly-nested-depset
871870
global_runfiles_depset = depset(direct = [unit_file_fragment], transitive = [global_runfiles_depset, runfiles_depset])
872871
cc_path = _rlocation(ctx, hs.toolchain.cc_wrapper.executable)
873872
ld_path = "$(rlocation {}{})".format(

0 commit comments

Comments
 (0)