title | description | author | ms.author | ms.date | ms.topic |
---|---|---|---|---|---|
vcpkg Maintainer Guide |
The Guide for maintainers contributing to vcpkg. |
vicroms |
viromer |
7/22/2024 |
concept-article |
This document lists a set of policies that you should apply when adding or updating a port recipe. It is intended to serve the role of Debian's Policy Manual, Homebrew's Maintainer Guidelines, and Homebrew's Formula Cookbook.
We wish to be able to show downstream users of libraries in the curated registry that the
combination of libraries in any given baseline we publish have been tested to work together in at
least some configurations. Allowing ports to exclude each other breaks the ability to test such
configurations, as the number of builds necessary for such tests would grow as
2^number_of_such_cases
. Moreover, installing additional dependencies is always considered "safe":
there is no way for a port or end user to assert that a dependency is not installed in their
requirements.
If you wish to represent such an alternative situation for users, consider describing how
someone can create an overlay port implementing the alternative
form with a comment in portfile.cmake
rather than trying to add additional ports never built in
the curated registry's continuous integration. For example, see
[email protected].
Before the introduction of registries, we accepted several
not tested ports-as-alternatives, such as boringssl
, that could make authoring overlay ports
easier. This is no longer accepted because registries allow publishing of these untested ports
without modifying the curated registry.
Whenever possible, separate changes into multiple PRs. This makes them significantly easier to review and prevents issues with one set of changes from holding up every other change.
For example, avoid reformatting or renaming variables in portfiles that otherwise have no reason to be modified for the issue at hand. However, if you need to modify the file for the primary purpose of the PR (updating the library), then obviously beneficial changes like fixing typos are appreciated!
Port names should attempt to be unambiguous about which package the port installs. Ideally, searching the port's name in a search engine should quickly lead you to the corresponding project. A good service to check many package names across multiple repositories at once is Repology.
Projects with short names or named after common words may require
disambiguation, specially when there are no projects with a strong association
to the given word. For example, a port with the name ip
is not acceptable
since it is likely that multiple projects would be named similarly.
Examples of good disambiguators are:
- The repository's owner username or organization:
google-cloud-cpp
. - The name of a suite of libraries the project is part of:
boost-dll
.
Common prefixes and suffixes used by C++ and open source projects are not valid disambiguators, some examples include but are not limited to:
cpp
,free
,lib
,open
,- numbers
For example, when comparing the following port names: ip-cpp
, libip
and
ip5
and removing the invalid disambiguators they all are reduced to the same
stem (ip
) and thus are considered to have the same name.
An exception to this guideline is made for names that are strongly associated
with a single project. For example: libpng
, openssl
and zlib
.
GitHub Draft PRs are a great way to get CI or human feedback on work that isn't yet ready to merge. Most new PRs should be opened as drafts and converted to normal PRs once the CI passes.
For more information about GitHub Draft PRs, see Introducing draft pull requests.
At this time, the following helpers are deprecated:
vcpkg_extract_source_archive_ex()
should be replaced by the supported overload ofvcpkg_extract_source_archive()
(withARCHIVE
)- The deprecated overload of
vcpkg_extract_source_archive()
withoutARCHIVE
should be replaced by the supported overload withARCHIVE
. vcpkg_apply_patches()
should be replaced by thePATCHES
arguments to the "extract" helpers (e.g.vcpkg_from_github()
)vcpkg_build_msbuild()
should be replaced byvcpkg_install_msbuild()
vcpkg_copy_tool_dependencies()
should be replaced byvcpkg_copy_tools()
vcpkg_configure_cmake
should be replaced byvcpkg_cmake_configure()
after removingPREFER_NINJA
vcpkg_build_cmake
should be replaced byvcpkg_cmake_build()
vcpkg_install_cmake
should be replaced byvcpkg_cmake_install()
vcpkg_fixup_cmake_targets
should be replaced byvcpkg_cmake_config_fixup
Some of the replacement helper functions are in "tools ports" to allow consumers to pin their
behavior at specific versions, to allow locking the behavior of the helpers at a particular
version. Tools ports need to be added to your port's "dependencies"
, like so:
{
"name": "vcpkg-cmake",
"host": true
},
{
"name": "vcpkg-cmake-config",
"host": true
}
Ideally, portfiles should be short, simple, and as declarative as possible.
Remove any boiler plate comments introduced by the create
command before submitting a PR.
Ports must not change their behavior based on which ports are already installed in a form that would change which contents that port installs. For example, given:
> vcpkg install a
> vcpkg install b
> vcpkg remove a
and
> vcpkg install b
the files installed by b
must be the same, regardless of influence by the previous installation of a
. This means that ports must not try to detect whether something is provided in the installed tree by another port before taking some action. A specific and common cause of such "path dependent" behavior is described below in "When defining features, explicitly control dependencies."
In the entire vcpkg system, no two ports a user is expected to use concurrently may provide the same file. If a port tries to install a file already provided by another file, installation will fail. If a port wants to use an extremely common name for a header, for example, it should place those headers in a subdirectory rather than in include
.
This property is checked regularly by continuous integration runs which try to install all ports in the registry, which will fail with FILE_CONFLICTS
if two ports provide the same file.
A core design ideal of vcpkg is to not create "lock-in" for users. In the build system, there should be no difference between depending on a library from the system, and depending on a library from vcpkg. To that end, we avoid adding CMake exports or targets to existing libraries with "the obvious name", to allow upstreams to add their own official CMake exports without conflicting with vcpkg.
To that end, any CMake configs that the port exports, which are not in the upstream library, should have unofficial-
as a prefix. Any additional targets should be in the unofficial::<port>::
namespace.
This means that the user should see:
find_package(unofficial-<port> CONFIG)
as the way to get at the unique-to-vcpkg packageunofficial::<port>::<target>
as an exported target from that port.
Examples:
brotli
creates theunofficial-brotli
package, producing targetunofficial::brotli::brotli
.
Each port has to provide a file named copyright
in the folder ${CURRENT_PACKAGES_DIR}/share/${PORT}
. If a package's license content is available within its source files, this file should be created by a call to vcpkg_install_copyright()
. vcpkg_install_copyright
also bundles multiple copyright files if necessary.
vcpkg_install_copyright(FILE_LIST "${SOURCE_PATH}/LICENSE")
An older method to manually create this file is with CMake's built in file
command. This is discouraged in favor of vcpkg_install_copyright
in new ports but is still allowed.
file(INSTALL "${SOURCE_PATH}/LICENSE" DESTINATION "${CURRENT_PACKAGES_DIR}/share/${PORT}" RENAME copyright)
If the license content in the upstream source files is not in text form (e.g. a PDF file), copyright
should contain an explanation as to how a user can find the license requirements. If possible, it should also include a link to the original source files indicating this, so users can check if it is up to date.
file(WRITE "${CURRENT_PACKAGES_DIR}/share/${PORT}/copyright" [[As of 2023-07-25, according to
https://github.com/GPUOpen-LibrariesAndSDKs/display-library/blob/master/Public-Documents/README.md#end-user-license-agreement
this software is bound by the "SOFTWARE DEVELOPMENT KIT LICENSE AGREEMENT" PDF located at
https://github.com/GPUOpen-LibrariesAndSDKs/display-library/blob/master/Public-Documents/ADL%20SDK%20EULA.pdf
]])
Version constraints within ports should generally be avoided, as they can hinder the independent evolution of projects. Adding such constraints is only permissible when there is a well-documented justification, such as proven incompatibility with specific earlier versions. These constraints should not be used merely to maintain parity with independent projects.
Features must be treated as additive functionality. If port[featureA]
installs and port[featureB]
installs, then port[featureA,featureB]
must install. Moreover, if a second port depends on [featureA]
and a third port depends on [featureB]
, installing both the second and third ports should have their dependencies satisfied.
Libraries in this situation must choose one of the available options as expressed in vcpkg, and users who want a different setting must use overlay ports at this time.
Existing examples we would not accept today retained for backwards compatibility:
libgit2
,libzip
,open62541
all have features for selecting a TLS or crypto backend.curl
has different crypto backend options but allows selecting between them at runtime, meaning the above tenet is maintained.darknet
hasopencv2
,opencv3
, features to control which version of opencv to use for its dependencies.
Notwithstanding the above, if there is a preview branch or similar where the preview functionality has a high probability of not disrupting the non-preview functionality (for example, no API removals), a feature is acceptable to model this setting.
Examples:
- The Azure SDKs (of the form
azure-Xxx
) have apublic-preview
feature. imgui
has anexperimental-docking
feature which engages their preview docking branch which uses a merge commit attached to each of their public numbered releases.
If a consumer is depending directly upon a library, they can list out any desired features easily (library[feature1,feature2]
). However, if a consumer does not know they are using a library, they cannot list out those features. If that hidden library is like libarchive
where features are adding additional compression algorithms (and thus behaviors) to an existing generic interface, default features offer a way to ensure a reasonably functional transitive library is built even if the final consumer doesn't name it directly.
If the feature adds additional APIs (or executables, or library binaries) and doesn't modify the behavior of existing APIs, it should be left off by default. This is because any consumer which might want to use those APIs can easily require it via their direct reference.
If in doubt, do not mark a feature as default.
If a consumer of a port depends on only the core functionality of that port, with high probability they must not be broken by turning on the feature. This is even more important when the alternative is not directly controlled by the consumer, but by compiler settings like /std:c++17
/ -std=c++17
.
Existing examples we would not accept today retained for backwards compatibility:
redis-plus-plus[cxx17]
controls a polyfill but does not bake the setting into the installed tree.ace[wchar]
changes all APIs to acceptconst wchar_t*
rather thanconst char*
.
A feature may replace polyfills with aliases provided that replacement is baked into the installed tree
Notwithstanding the above, ports may remove polyfills with a feature, as long as:
- Turning on the feature changes the polyfills to aliases of the polyfilled entity
- The state of the polyfill is baked into the installed headers, such that ABI mismatch "impossible" runtime errors are unlikely
- It is possible for a consumer of the port to write code which works in both modes, for example by using a typedef which is either polyfilled or not
Example:
abseil[cxx17]
changesabsl::string_view
to a replacement orstd::string_view
; the patch implements the baking requirement.
If it's critical to expose the underlying alternatives, we recommend providing messages at build time to instruct the user on how to copy the port into a private overlay:
set(USING_DOG 0)
message(STATUS "This version of LibContoso uses the Kittens backend. To use the Dog backend instead, create an overlay port of this with USING_DOG set to 1 and the `kittens` dependency replaced with `dog`.")
message(STATUS "This recipe is at ${CMAKE_CURRENT_LIST_DIR}")
message(STATUS "See the overlay ports documentation at https://github.com/microsoft/vcpkg/blob/master/docs/specifications/ports-overlay.md")
Do not use embedded copies of libraries. All dependencies should be split out and packaged separately so they can be updated and maintained.
When multiple buildsystems are available, prefer using CMake.
Additionally, when appropriate, it can be easier and more maintainable to rewrite alternative buildsystems into CMake using file(GLOB)
directives.
Examples: abseil
When building CMake libraries, vcpkg_cmake_configure()
will pass in the correct value for BUILD_SHARED_LIBS
based on the user's requested variant.
You can calculate alternative configure parameters by using string(COMPARE EQUAL "${VCPKG_LIBRARY_LINKAGE}" ...)
.
# portfile.cmake
string(COMPARE EQUAL "${VCPKG_LIBRARY_LINKAGE}" "static" KEYSTONE_BUILD_STATIC)
string(COMPARE EQUAL "${VCPKG_LIBRARY_LINKAGE}" "dynamic" KEYSTONE_BUILD_SHARED)
vcpkg_cmake_configure(
SOURCE_PATH ${SOURCE_PATH}
OPTIONS
-DKEYSTONE_BUILD_STATIC=${KEYSTONE_BUILD_STATIC}
-DKEYSTONE_BUILD_SHARED=${KEYSTONE_BUILD_SHARED}
)
If a library does not offer configure options to select the build variant, the build must be patched. When patching a build, you should always attempt to maximize the future maintainability of the port. Typically this means minimizing the number of lines that need to be touched to fix the issue at hand.
For example, when patching a CMake-based library, it may be sufficient to add EXCLUDE_FROM_ALL
to unwanted targets and wrap the install(TARGETS ...)
call in an if(BUILD_SHARED_LIBS)
. This will be shorter than wrapping or deleting every line that mentions the unwanted variant.
For a project CMakeLists.txt
with the following contents:
add_library(contoso SHARED contoso.c)
add_library(contoso_static STATIC contoso.c)
install(TARGETS contoso contoso_static EXPORT ContosoTargets)
install(EXPORT ContosoTargets
FILE ContosoTargets
NAMESPACE contoso::
DESTINATION share/contoso)
Only the install(TARGETS)
line needs to be patched.
add_library(contoso SHARED contoso.c)
add_library(contoso_static STATIC contoso.c)
if(BUILD_SHARED_LIBS)
set_target_properties(contoso_static PROPERTIES EXCLUDE_FROM_ALL 1)
install(TARGETS contoso EXPORT ContosoTargets)
else()
set_target_properties(contoso PROPERTIES EXCLUDE_FROM_ALL 1)
install(TARGETS contoso_static EXPORT ContosoTargets)
endif()
install(EXPORT ContosoTargets
FILE ContosoTargets
NAMESPACE contoso::
DESTINATION share/contoso)
When defining a feature that captures an optional dependency, ensure that the dependency will not be used accidentally when the feature is not explicitly enabled.
set(CMAKE_DISABLE_FIND_PACKAGE_ZLIB ON)
set(CMAKE_REQUIRE_FIND_PACKAGE_ZLIB OFF)
if ("zlib" IN_LIST FEATURES)
set(CMAKE_DISABLE_FIND_PACKAGE_ZLIB OFF)
set(CMAKE_REQUIRE_FIND_PACKAGE_ZLIB ON)
endif()
vcpkg_cmake_configure(
SOURCE_PATH ${SOURCE_PATH}
OPTIONS
-DCMAKE_DISABLE_FIND_PACKAGE_ZLIB=${CMAKE_DISABLE_FIND_PACKAGE_ZLIB}
-DCMAKE_REQUIRE_FIND_PACKAGE_ZLIB=${CMAKE_REQUIRE_FIND_PACKAGE_ZLIB}
)
The snippet below using vcpkg_check_features()
is equivalent.
vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
FEATURES
"zlib" CMAKE_REQUIRE_FIND_PACKAGE_ZLIB
INVERTED_FEATURES
"zlib" CMAKE_DISABLE_FIND_PACKAGE_ZLIB
)
vcpkg_cmake_configure(
SOURCE_PATH ${SOURCE_PATH}
OPTIONS
${FEATURE_OPTIONS}
)
ZLIB
in the snippet is case-sensitive. For more information, see the CMAKE_DISABLE_FIND_PACKAGE_<PackageName>
and CMAKE_REQUIRE_FIND_PACKAGE_<PackageName>
documentation.
A lib is considered conflicting if it does any of the following:
- Define
main
- Define malloc
- Define symbols that are also declared in other libraries
Conflicting libs are typically by design and not considered a defect. Because some build systems link against everything in the lib directory, these should be moved into a subdirectory named manual-link
.
When creating a new port, follow the versioning convention used by the package author. When updating the port, continue to use the same convention unless upstream says otherwise. For a full explanation of our conventions, see our versioning documentation.
If upstream has not published a release in a while, do not change the port's versioning scheme to version-date
in order to get the latest changes. These commits may include changes that are not production ready. Instead ask the upstream repository to publish a new release.
vcpkg uses this field to determine whether a given port is out-of-date and should be changed whenever the port's behavior changes.
Our convention is to use the "port-version"
field for changes to the port that don't change the upstream version, and to reset the "port-version"
back to zero when an update to the upstream version is made.
For Example:
- Zlib's package version is currently
1.2.1
, with no explicit"port-version"
(equivalent to a"port-version"
of0
). - You've discovered that the wrong copyright file has been deployed, and fixed that in the portfile.
- You should update the
"port-version"
field in the manifest file to1
.
See the versioning documentation for more information.
vcpkg uses a set of metadata files to power its versioning feature. These files are located in the following locations:
${VCPKG_ROOT}/versions/baseline.json
, (this file is common to all ports) and${VCPKG_ROOT}/versions/${first-letter-of-portname}-/${portname}.json
(one per port).
For example, for zlib
the relevant files are:
${VCPKG_ROOT}/versions/baseline.json
${VCPKG_ROOT}/versions/z-/zlib.json
We expect that each time you update a port, you also update its version files.
The recommended method to update these files is to run the x-add-version
command, e.g.:
vcpkg x-add-version zlib
If you're updating multiple ports at the same time, instead you can run:
vcpkg x-add-version --all
to update the files for all modified ports at once.
Note
These commands require you to have committed your changes to the ports before running them. The reason is that the Git SHA of the port directory is required in these version files. But don't worry, the x-add-version
command will warn you if you have local changes that haven't been committed.
For more information, see the Versioning reference and Creating registries.
vcpkg is a packaging solution, not the ultimate owners of the components that we deploy. We do need to apply patches in some cases to improve compatibility of components with platforms, or compatibility of components with each other.
- We want to avoid patches that:
- upstream would disagree with
- cause vulnerabilities or crashes
- we are incapable of maintaining across upstream version updates
- are large enough to cause license entanglement with the vcpkg repository itself
If a patch could possibly be useful by upstream, upstream must be notified of the patch's content. (Patches that apply vcpkg-specific behavior unrelated to upstream, such as devendoring a dependency, don't require notification.)
To avoid situations where upstream disagrees with the patch, we will wait at least 30 days to apply such patches.
We will skip this waiting period if we have high confidence that the change is correct. Examples of high confidence patches include, but are not limited to:
- Upstream's acceptance as a patch (for example, backporting a specific change from a pull request upstream has merged).
- Adding missing
#include
s. - Small and obvious product code fixes (for example, initializing an uninitialized variable).
- Disabling irrelevant-in-vcpkg components of the build such as tests or examples.
It is preferable to set options in a call to vcpkg_configure_xyz()
over patching the settings directly.
Common options that allow you to avoid patching:
- [MSBUILD]
<PropertyGroup>
settings inside the project file can be overridden via/p:
parameters - [CMAKE] Calls to
find_package(XYz)
in CMake scripts can be disabled via-DCMAKE_DISABLE_FIND_PACKAGE_XYz=ON
- [CMAKE] Cache variables (declared as
set(VAR "value" CACHE STRING "Documentation")
oroption(VAR "Documentation" "Default Value")
) can be overridden by just passing them in on the command line as-DVAR:STRING=Foo
. One notable exception is if theFORCE
parameter is passed toset()
. For more information, see the CMakeset
documentation
If an approved or merged patch file can be obtained from upstream, ports should try to download them and apply them instead of having them as part of the port files. This process is prefered because it:
- Confirms that upstream has accepted the patch changes
- Simplifies the reviewing process by shifting the onus upstream
- Reduces the vcpkg repository size for users that aren't using the patch
- Avoids license conflicts with the vcpkg repository
Patches should be downloaded from a stable endpoint to avoid SHA conflicts.
When downloading patch files from a pull request or commit from GitHub and
GitLab the ?full_index=1
parameter should be appended to the download URL.
Examples:
https://github.com/google/farmhash/pull/40.diff?full_index=1
https://github.com/linux-audit/audit-userspace/commit/f8e9bc5914d715cdacb2edc938ab339d5094d017.patch?full_index=1
https://gitlab.kitware.com/paraview/paraview/-/merge_requests/6375.diff?full_index=1
Some variables prefixed with VCPKG_<VARIABLE>
have an equivalent CMAKE_<VARIABLE>
.
However, not all of them are passed to the internal package build (see implementation: Windows toolchain).
Consider the following example:
set(VCPKG_C_FLAGS "-O2 ${VCPKG_C_FLAGS}")
set(VCPKG_CXX_FLAGS "-O2 ${VCPKG_CXX_FLAGS}")
Using vcpkg
's built-in toolchains this works, because the value of VCPKG_<LANG>_FLAGS
is forwarded to the appropriate CMAKE_LANG_FLAGS
variable. But, a custom toolchain that is not aware of vcpkg
's variables will not forward them.
Because of this, it is preferable to patch the buildsystem directly when setting CMAKE_<LANG>_FLAGS
.
When making changes to a library, strive to minimize the final diff. This means you should not reformat the upstream source code when making changes that affect a region. When disabling a conditional, it is better to add an AND FALSE
or && 0
to the condition than to delete every line of the conditional. If a large region needs to be disabled, it is shorter to add an if(0)
or #if 0
around the region instead of deleting every line in the patch.
Don't add patches if the port is outdated and updating the port to a newer released version would solve the same issue. vcpkg prefers updating ports over patching outdated versions.
This helps to keep the size of the vcpkg repository down as well as improves the likelihood that the patch will apply to future code versions.
The purpose of patching in vcpkg is to enable compatibility with compilers, libraries, and platforms. It is not to implement new features in lieu of following proper Open Source procedure (submitting an Issue/PR/etc).
When submitting a new port, check for any options like BUILD_TESTS
or WITH_TESTS
or POCO_ENABLE_SAMPLES
and ensure the additional binaries are disabled. This minimizes build times and dependencies for the average user.
Optionally, you can add a test
feature which enables building the tests, however this should not be in the Default-Features
list.
Unless the author of the library is already using it, we should not use this CMake functionality because it interacts poorly with C++ templates and breaks certain compiler features. Libraries that don't provide a .def file and do not use __declspec() declarations simply do not support shared builds for Windows and should be marked as such with vcpkg_check_linkage(ONLY_STATIC_LIBRARY)
.
This means that if the upstream library has different names in release and debug (libx versus libxd), then the debug library should not be renamed to libx
. Vice versa, if the upstream library has the same name in release and debug, we should not introduce a new name.
Important caveat:
- Static and shared variants often should be renamed to a common scheme. This enables consumers to use a common name and be ignorant of the downstream linkage. This is safe because we only make one at a time available.
If a library generates CMake integration files (foo-config.cmake
), renaming must be done through patching the CMake build itself instead of simply calling file(RENAME)
on the output archives/LIBs.
Finally, DLL files on Windows should never be renamed post-build because it breaks the generated LIBs.
We require that the manifest file be formatted. Use the following command to format all manifest files:
> vcpkg format-manifest --all
We are not accepting requests to add non-community triplets at this time. Promotion from community to full triplet status is primarily based on budget for the hardware to test such triplets and will be driven by metrics submitted by vcpkg to maximize the likelihood what people actually use is fully tested.
We will add community triplets if:
- It is demonstrated that people will actually use that community triplet; and,
- we don't know that such a triplet is broken.
For example, we did not add a triplet in microsoft/vcpkg#29034 because the author was just trying to "complete the set" rather than indicating they would actually use such a thing, and we did not add linux-dynamic until the patchelf solution to make the results relocatable was created.
While portfile.cmake
's and CMakeLists.txt
's share a common syntax and core CMake language constructs (aka "Scripting Commands"), portfiles run in "Script Mode", whereas CMakeLists.txt
files run in "Project Mode". The most important difference between these two modes is that "Script Mode" does not have the concepts of "Toolchain", "Language" and "Target". Any behaviors, including scripting commands, which depend on these constructs (e.g. CMAKE_CXX_COMPILER
, CMAKE_EXECUTABLE_SUFFIX
, CMAKE_SYSTEM_NAME
) will not be correct.
Portfiles have direct access to variables set in the triplet file, but CMakeLists.txt
s do not (though there is often a translation that happens -- VCPKG_LIBRARY_LINKAGE
versus BUILD_SHARED_LIBS
).
Portfiles and Project builds invoked by portfiles are run in different processes. Conceptually:
+----------------------------+ +------------------------------------+
| CMake.exe | | CMake.exe |
+----------------------------+ +------------------------------------+
| Triplet file | ====> | Toolchain file |
| (x64-windows.cmake) | | (scripts/buildsystems/vcpkg.cmake) |
+----------------------------+ +------------------------------------+
| Portfile | ====> | CMakeLists.txt |
| (ports/foo/portfile.cmake) | | (buildtrees/../CMakeLists.txt) |
+----------------------------+ +------------------------------------+
To determine the host in a portfile, the standard CMake variables are fine (CMAKE_HOST_WIN32
).
To determine the target in a portfile, the vcpkg triplet variables should be used (VCPKG_CMAKE_SYSTEM_NAME
).
See also our triplet documentation for a full enumeration of possible settings.