Skip to content

Remove warning cmake_minimum_required#91

Closed
hannesa2 wants to merge 1 commit intomasterfrom
RemoveWarning
Closed

Remove warning cmake_minimum_required#91
hannesa2 wants to merge 1 commit intomasterfrom
RemoveWarning

Conversation

@hannesa2
Copy link

@hannesa2 hannesa2 commented Mar 1, 2026

Since libgit2's CMakeLists.txt already calls cmake_minimum_required(VERSION 3.5.1) as its very first statement, the variable was always redundant. Removing it eliminates the warning.

This is the causing warning:
CMake suite maintained and supported by Kitware (kitware.com/cmake). CMake Warning (dev) at CMakeLists.txt:14 (PROJECT): cmake_minimum_required() should be called prior to this top-level project() call.

Line 690 in https://github.com/gitx/gitx/actions/runs/22541819447/job/65298081457

Since libgit2's CMakeLists.txt already calls cmake_minimum_required(VERSION 3.5.1) as its very first statement, the variable was always redundant. Removing it eliminates the warning.

This is the causing warning:
CMake suite maintained and supported by Kitware (kitware.com/cmake).
CMake Warning (dev) at CMakeLists.txt:14 (PROJECT):
cmake_minimum_required() should be called prior to this top-level project() call.

Line 690 in https://github.com/gitx/gitx/actions/runs/22541819447/job/65298081457
@hannesa2 hannesa2 closed this Mar 1, 2026
@hannesa2 hannesa2 deleted the RemoveWarning branch March 1, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant