Skip to content

[vcpkg_find_acquire_program(NINJA)] Do not check version of system ninja on FreeBSD#46345

Merged
vicroms merged 1 commit intomicrosoft:masterfrom
SchaichAlonso:46344-use-system-ninja-on-freebsd
Jul 13, 2025
Merged

[vcpkg_find_acquire_program(NINJA)] Do not check version of system ninja on FreeBSD#46345
vicroms merged 1 commit intomicrosoft:masterfrom
SchaichAlonso:46344-use-system-ninja-on-freebsd

Conversation

@SchaichAlonso
Copy link
Copy Markdown
Contributor

@SchaichAlonso SchaichAlonso commented Jul 9, 2025

VCPKG upgraded it's ninja version to 1.12.1 in f576077 / #41980 . FreeBSD had ninja-1.12.1 in it's third party package repository, but reverted it to 1.11.0 (freebsd/freebsd-ports@1e1a1e9), as issues with ccache were discovered when using ninja-1.12.1.

vcpkg does not deploy ninja on FreeBSD, and does not accept the version found on the system, either.

This PR removes the version check on FreeBSD's branch, allowing vcpkg to use the pkg/ports version, even if it is "outdated".

Fixes #46344 .

VCPKG upgraded it's ninja version to 1.12.1 in microsoft/vcpkg@f576077 / microsoft#41980 .
FreeBSD had ninja-1.12.1 in it's third party package repository, but reverted it to 1.11.0 (freebsd/freebsd-ports@1e1a1e9), as issues with ccache were discovered when using ninja-1.12.1.

vcpkg does not deploy ninja on FreeBSD, and does not accept the version found on the system, either.

Remove the version check on FreeBSD's branch, to allow vcpkg to use the pkg/ports version even if it is "outdated".
@SchaichAlonso SchaichAlonso changed the title Do not check version of system ninja on FreeBSD [vcpkg_find_acquire_program(NINJA)] Do not check version of system ninja on FreeBSD Jul 9, 2025
@BillyONeal
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@vicroms vicroms self-assigned this Jul 10, 2025
@vicroms vicroms added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jul 10, 2025
@vicroms vicroms merged commit 9edc4f5 into microsoft:master Jul 13, 2025
18 checks passed
talregev pushed a commit to talregev/vcpkg that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ninja-1.12.1 not available on FreeBSD

3 participants