Move the build-mylib test into the regular test suite (and get rid of the separate build-mylib CI job)

Author: DilumAluthge

.github/workflows/ci.yml

@@ -130,33 +130,3 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # If authenticating with GitHub Actions token
           DOCUMENTER_KEY: ${{ secrets.DOCUMENTER_KEY }} # If authenticating with SSH deploy key
         run: julia --project=docs/ -e 'using Pkg; Pkg.instantiate(); include("docs/make.jl")'
-  build-mylib:
-    runs-on: ubuntu-latest
-    timeout-minutes: 60
-    strategy:
-      # Only run 1 of the `build-mylib` job at a time, so that this job doesn't take over
-      # too many CI resources, and also to leave space for other runs in the JuliaLang org.
-      max-parallel: 1
-      fail-fast: false
-      matrix:
-        julia-version:
-          - '1.10' # current LTS
-          - '1.11' # current stable
-    steps:
-      - uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
-      - uses: julia-actions/setup-julia@9b79636afcfb07ab02c256cede01fe2db6ba808c # v2.6.0
-        with:
-          version: ${{ matrix.julia-version }}
-      - uses: julia-actions/cache@824243901fb567ccb490b0d0e2483ccecde46834 # v2.0.5
-      - uses: julia-actions/julia-buildpkg@90dd6f23eb49626e4e6612cb9d64d456f86e6a1c # v1.6.0
-        with:
-          project: 'examples/MyLib'
-      - uses: julia-actions/julia-buildpkg@90dd6f23eb49626e4e6612cb9d64d456f86e6a1c # v1.6.0
-        with:
-          project: 'examples/MyLib/build'
-      - run: |
-          cd examples/MyLib
-          make
-      - run: ./examples/MyLib/my_application.out
-        env:
-          LD_LIBRARY_PATH: 'examples/MyLib/MyLibCompiled/lib'

Project.toml

@@ -28,8 +28,9 @@ julia = "1.6"
 
 [extras]
 Example = "7876af07-990d-54b4-ab0e-23690620f79a"
+GNUMake_jll = "6c1a3432-6b93-5c89-98b5-f3caf0c092ba"
 TOML = "fa267f1f-6049-4f14-aa54-33bafae1ed76"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 
 [targets]
-test = ["Test", "Example", "TOML"]
+test = ["Test", "Example", "GNUMake_jll", "TOML"]

examples/MyLib/Makefile

@@ -2,27 +2,31 @@
 
 .DEFAULT_GOAL := all
 
+.PHONY: all
+all: build-library build-executable
+
 JULIA ?= julia
-DLEXT := $(shell $(JULIA) --startup-file=no -e 'using Libdl; print(Libdl.dlext)')
+GCC ?= gcc
+DLEXT := $(shell $(JULIA) --startup-file=no -e 'import Libdl; print(Libdl.dlext)')
 
 TARGET="MyLibCompiled"
 
 MYLIB_INCLUDES = $(TARGET)/include/julia_init.h $(TARGET)/include/mylib.h
 MYLIB_PATH := $(TARGET)/lib/libmylib.$(DLEXT)
 
+.PHONY: build-library
 build-library: build/build.jl src/MyLib.jl
-	$(JULIA) --startup-file=no --project=. -e 'using Pkg; Pkg.instantiate()'
-	$(JULIA) --startup-file=no --project=build -e 'using Pkg; Pkg.instantiate(); include("build/build.jl")'
+	$(JULIA) --startup-file=no --project=. -e 'import Pkg; Pkg.instantiate(); Pkg.precompile()'
+	$(JULIA) --startup-file=no --project=build -e 'import Pkg; Pkg.instantiate(); Pkg.precompile()'
+	$(JULIA) --startup-file=no --project=build -e 'include("build/build.jl")'
 
 INCLUDE_DIR = $(TARGET)/include
 
+.PHONY: build-executable
 build-executable:
-	gcc my_application.c -o my_application.out -I$(INCLUDE_DIR) -L$(TARGET)/lib -ljulia -lmylib
-
-all: build-library build-executable
+	$(gcc) my_application.c -o my_application.out -I$(INCLUDE_DIR) -L$(TARGET)/lib -ljulia -lmylib
 
+.PHONY: clean
 clean:
 	$(RM) *~ *.o *.$(DLEXT)
 	$(RM) -Rf $(TARGET)
-
-.PHONY: build-library build-executable clean all

test/runtests.jl

@@ -3,6 +3,7 @@ using Test
 using Libdl
 using Pkg
 
+import GNUMake_jll
 import TOML
 
 ENV["JULIA_DEBUG"] = "PackageCompiler"
@@ -242,4 +243,52 @@ end
         hello = read(`$(Base.julia_cmd()) -J $(sysimage_path) -e 'print("hello, world")'`, String)
         @test hello == "hello, world"
     end
+
+    @testset "examples/MyLib" begin
+        # This testset makes sure that the `examples/MyLib` example does not bitrot.
+
+        if Sys.iswindows()
+            @info "Skipping the examples/MyLib test on Windows"
+            @test_skip false
+        else
+            if Sys.isapply()
+                @info "Skipping the examples/MyLib test on macOS"
+                @test_skip false
+        else
+            rootdir_testdir = @__DIR__
+            rootdir = dirname(rootdir_testdir)
+            rootdir_examples = joinpath(rootdir, "examples")
+            rootdir_examples_MyLib = joinpath(rootdir_examples, "MyLib")
+
+            my_run = (cmd::Cmd) -> begin
+                env2 = copy(ENV)
+
+                # The `JULIA` environment variable is used by our `MyLib/Makefile`.
+                # The `GCC` environment variable is used by our `MyLib/Makefile`.
+                env2["JULIA"] = Base.julia_cmd()[1]
+                env2["GCC"] = PackageCompiler.get_compiler_cmd()[1]
+
+                return run(cmd)
+            end
+
+            cd(rootdir_examples_MyLib) do
+                # We don't want to assume that the machine running the tests has `make` installed
+                # and available in the PATH. Therefore, we use the `GNUMake_jll.jl`` package.
+                my_run(`$(GNUMake_jll.make())`)
+
+                env2 = copy(ENV)
+                env2["LD_LIBRARY_PATH"] = "examples/MyLib/MyLibCompiled/lib"
+                cmd = `./examples/MyLib/my_application.out`
+                run(setenv(cmd, env2))
+
+                # Finally, exercise a few more `Makefile` targets, to make sure that they
+                # don't bitrot.
+                my_run(`$(GNUMake_jll.make()) all`)
+                my_run(`$(GNUMake_jll.make()) build-library`)
+                my_run(`$(GNUMake_jll.make()) build-executable`)
+                my_run(`$(GNUMake_jll.make()) clean`)
+
+            end
+        end
+    end # testset
 end