Upgrade testing platforms (#71)

* Upgrade testing platforms * Remove complex_deps IO::Tty is problematic on Mac. There should be an easier example to replace it with. * Comment out fcgi_import_test I will create a todo to figure out what is going on with this test and ask the original author to take a look. * Update presubmit.yml * Comment out program_test The script does not run on Mac. Need to revisit its purpose and work out how to get it to run on all environments. * buildifier examples/external_module/BUILD * Remove unused symbol --------- Co-authored-by: Erik Tank <[email protected]>
bazel-contrib · Sep 23, 2024 · a9e83d5 · a9e83d5
1 parent 427babc
commit a9e83d5
Show file tree

Hide file tree

Showing 11 changed files with 15 additions and 2,997 deletions.
diff --git a/.bazelci/presubmit.yml b/.bazelci/presubmit.yml
@@ -1,21 +1,15 @@
 ---
 platforms:
-  ubuntu1804:
+  ubuntu2204:
     build_targets:
     - "//..."
     test_targets:
     - "//..."
-  macos:
+  macos_arm64:
     build_targets:
     - "//..."
-    - "-//examples/cpan:IOTty"
-    - "-//examples/cpan:IOTtyXS"
-    - "-//examples/cpan_remote:fcgi_import_test"
-    - "-//examples/external_module:complex_deps"
     test_targets:
     - "//..."
-    - "-//examples/cpan_remote:fcgi_import_test"
-    - "-//examples/external_module:complex_deps"
 buildifier:
   version: latest
   warnings: "all"
diff --git a/examples/cpan/BUILD b/examples/cpan/BUILD
@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-load("//perl:perl.bzl", "perl_library", "perl_xs")
+load("//perl:perl.bzl", "perl_library")
 
 package(default_visibility = ["//visibility:public"])
 
@@ -22,48 +22,3 @@ perl_library(
     srcs = ["Test-Mock-Simple-0.10/lib/Test/Mock/Simple.pm"],
     includes = ["Test-Mock-Simple-0.10/lib"],
 )
-
-# Better example of module placement with dependence
-perl_library(
-    name = "IOPty",
-    srcs = ["IO/Pty.pm"],
-)
-
-perl_library(
-    name = "IOTty",
-    srcs = [
-        "IO/Tty.pm",
-        ":IOTtyXS",
-    ],
-    deps = [
-        "IOPty",
-        "IOTtyConstant",
-    ],
-)
-
-perl_library(
-    name = "IOTtyConstant",
-    srcs = ["IO/Tty/Constant.pm"],
-)
-
-perl_xs(
-    name = "IOTtyXS",
-    srcs = ["auto/IO/Tty/Tty.xs"],
-    defines = [
-        # "HAVE_DEV_PTMX",
-        "HAVE_GETPT",
-        "HAVE_GRANTPT",
-        "HAVE_POSIX_OPENPT",
-        "HAVE_PTSNAME",
-        "HAVE_PTSNAME_R",
-        "HAVE_PTY_H",
-        "HAVE_SIGACTION",
-        "HAVE_SYS_STROPTS_H",
-        "HAVE_TERMIOS_H",
-        "HAVE_TERMIO_H",
-        "HAVE_TTYNAME",
-        "HAVE_UNLOCKP",
-    ],
-    output_loc = "auto/IO/Tty/Tty.so",
-    textual_hdrs = ["auto/IO/Tty/xssubs.c"],
-)