Simplify Ruby builds by avoiding data files

Everything that we need can be obtained as srcs already

Author: p0deje

rb/BUILD.bazel

@@ -82,8 +82,7 @@ rb_library(
     srcs = glob([
         "lib/selenium/webdriver/devtools/**/*.rb",
         "lib/selenium/webdriver/devtools.rb",
-    ]),
-    data = [":mutation-listener"],
+    ]) + [":mutation-listener"],
     deps = [":common"],
 )
 
@@ -114,8 +113,7 @@ rb_library(
         "Gemfile",
         "CHANGES",
         "README.md",
-    ]),
-    data = [
+    ]) + [
         ":find-elements",
         ":get-attribute",
         ":is-displayed",
@@ -160,8 +158,7 @@ rb_library(
     srcs = glob([
         "lib/selenium/webdriver/remote/**/*.rb",
         "lib/selenium/webdriver/remote.rb",
-    ]),
-    data = [":selenium-server-deploy"],
+    ]) + [":selenium-server-deploy"],
     deps = [":common"],
 )
 
@@ -181,8 +178,7 @@ rb_library(
         "lib/selenium/devtools.rb",
         "lib/selenium/devtools/version.rb",
         "selenium-devtools.gemspec",
-    ]),
-    data = [":license"] + [":cdp-" + n for n in CDP_VERSIONS],
+    ]) + [":license"] + [":cdp-" + n for n in CDP_VERSIONS],
     deps = ["@bundle//:websocket"],
 )
 

rb/private/rb_gem.bzl

@@ -1,11 +1,6 @@
 def _rb_gem_impl(ctx):
     gem_builder = ctx.actions.declare_file("gem_builder.rb")
 
-    inputs = []
-    for dep in ctx.attr.deps:
-        inputs.extend(dep.files.to_list())
-        inputs.extend(dep[DefaultInfo].data_runfiles.files.to_list())
-
     ctx.actions.expand_template(
         template = ctx.file._gem_builder_template,
         output = gem_builder,
@@ -17,7 +12,7 @@ def _rb_gem_impl(ctx):
     )
 
     ctx.actions.run(
-        inputs = inputs,
+        inputs = ctx.files.deps,
         executable = gem_builder,
         outputs = [ctx.outputs.gem],
         execution_requirements = {
@@ -28,20 +23,13 @@ def _rb_gem_impl(ctx):
 rb_gem = rule(
     _rb_gem_impl,
     attrs = {
-        "srcs": attr.label_list(
-            allow_files = True,
-        ),
         "gemspec": attr.label(
             allow_single_file = True,
             mandatory = True,
         ),
         "deps": attr.label_list(
             allow_files = True,
         ),
-        "data": attr.label_list(
-            allow_empty = True,
-            allow_files = True,
-        ),
         "_gem_builder_template": attr.label(
             allow_single_file = True,
             default = Label("//rb/private:gem_builder.tpl"),