Offline tests and adaparse compilation without installed Git

[mymedia2]

What is the problem this feature will solve?

In Debian, we would like to run automated tests in controlled environments without Internet access. Our testing tool installs each required dependency before configuring tests.

What is the feature you are proposing to solve the problem?

Please implement dependency search trough CMake packages with find_package(). Fallback to CPM is possible if the package is not present in the system.

find_package(GTest 1.14.0)
if (NOT GTest_FOUND)
  CPMAddPackage(
    NAME GTest
    GITHUB_REPOSITORY google/googletest
    VERSION 1.14.0
    OPTIONS  "BUILD_GMOCK OFF" "INSTALL_GTEST OFF"
  )
endif()

... and so on for all other dependencies.

These considerations also apply to compiling adaparse.

What alternatives have you considered?

No response

[anonrig]

Which version are you using? This issue should have been fixed on main branch where we bundle CPM with Ada rather than downloading it.

[lemire]

@mymedia2

If you want CMake to first look locally, you can do so by passing the CPM_USE_LOCAL_PACKAGES option. If you want require the use of local packages (at the expensive of a failure when they are not found), you can do so by passing CPM_LOCAL_PACKAGES_ONLY option.

E.g.,

cmake -B build -D CPM_USE_LOCAL_PACKAGES=ON -D ADA_TESTING=ON

This should do exactly what you ask that we do. That is, the functionality is already present.

[mymedia2]

Great! I used tagged version 2.9.2 and did not know the CPM script has been bundled. Though, checks if (Git_FOUND) block tests and adaparse if Git is not installed. I thought Git was required for downloading. But when the dependencies are already available offline. the checks look unnecessary.

-- The tests are disabled because git was not found.
-- The tools are disabled because git was not found.

ada/CMakeLists.txt

Line 60 in 390b23b

if(Git_FOUND)

ada/CMakeLists.txt

Line 109 in 390b23b

if(Git_FOUND)

So I can only request to rearrange the checks and make the CPM_USE_LOCAL_PACKAGES variable working even when Git is missing or disabled.

[anonrig]

We should definitely add a CI job to test this if Debian depends on it. Thanks for raising the issue @mymedia2. Is there anything else blocking your or your colleagues work for Debian?

[mymedia2]

@anonrig

Is there anything else blocking your or your colleagues work for Debian?

I think, no. We implement our features gradually, step-by-step.
And thank you for quick response!