[serf] add new port

[rinrab]

• Changes comply with the maintainer guide.
• The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
• Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
• The versioning scheme in vcpkg.json matches what upstream says.
• The license declaration in vcpkg.json matches what upstream says.
• The installed as the "copyright" file matches what upstream says.
• The source code of the component installed comes from an authoritative source.
• The generated "usage text" is accurate. See adding-usage for context.
• The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
• Only one version is in the new port's versions file.
• Only one version is added to each modified port's versions file.

[dg0yt]

Upstream has a cmake buildsystem. Is there a good reason to add a vendored msbuild?

[rinrab]

The CMake build system is currently only available in the development version (trunk) and has not been officially released yet. In the stable version, SCons is the only available build system.

[MonicaLiu0311]

The above are some modifications to deprecated functions and redundant code. In addition, there is another problem. I found that this port provides neither .pc file nor *config.cmake. Please provide at least one usage.

[rinrab]

The above are some modifications to deprecated functions and redundant code. In addition, there is another problem. I found that this port provides neither .pc file nor *config.cmake. Please provide at least one usage.

I added package-config, usage, and fixed that modifications about deprecated functions and redundant code.

[rinrab]

I've implemented the corrections you pointed out and it's now ready for review. Additionally, I've made a minor adjustment to the target name: I've appended the major version name to it, changing it from 'serf' to 'serf-1'.

[MonicaLiu0311]

usage:

serf provides CMake targets:

    find_package(SERF REQUIRED)
    target_link_libraries(main PRIVATE SERF:serf)

When testing usage, the following error occurs:

>------ Build All started: Project: CMakeUsage, Configuration: x64-Debug ------
  MSBuild version 17.9.5+33de0b227 for .NET Framework
  
    1>Checking Build System
    Building Custom Rule C:/Users/monica/source/repos/CMakeUsage/CMakeUsage/CMakeLists.txt
    CMakeUsage.cpp
C:\Users\monica\source\repos\CMakeUsage\CMakeUsage\CMakeUsage.cpp(2,10): error C1083: Cannot open include file: 'serf.h': No such file or directory 

Build All failed.

CMakeLists.txt

cmake_minimum_required (VERSION 3.8)
set(CMAKE_TOOLCHAIN_FILE "G:/serf/scripts/buildsystems/vcpkg.cmake")
project ("CMakeUsage")
add_library (main "CMakeUsage.cpp" "CMakeUsage.h")
find_package(SERF REQUIRED)
target_link_libraries(main PRIVATE SERF:serf)

CMakeUsage.cpp

#include <iostream>
#include "serf.h"
using namespace std;
int main()

{

cout << "Hello CMake." << endl;

return 0;

}

[rinrab]

usage:

serf provides CMake targets:

    find_package(SERF REQUIRED)
    target_link_libraries(main PRIVATE SERF:serf)

When testing usage, the following error occurs:

>------ Build All started: Project: CMakeUsage, Configuration: x64-Debug ------
  MSBuild version 17.9.5+33de0b227 for .NET Framework
  
    1>Checking Build System
    Building Custom Rule C:/Users/monica/source/repos/CMakeUsage/CMakeUsage/CMakeLists.txt
    CMakeUsage.cpp
C:\Users\monica\source\repos\CMakeUsage\CMakeUsage\CMakeUsage.cpp(2,10): error C1083: Cannot open include file: 'serf.h': No such file or directory 

Build All failed.

CMakeLists.txt

CMakeUsage.cpp

Thank you! I found and fixed this problem; I forgot to specify the INTERFACE_INCLUDE_DIRECTORIES property.

[MonicaLiu0311]

The usage test passed on x64-windows (header files found):

serf provides CMake targets:

    find_package(SERF REQUIRED)
    target_link_libraries(main PRIVATE SERF:serf)

[dg0yt]

Do CMake config and target name match the future upstream name, or are they unofficial?
Does it work properly in single-config mode?
Doesn't it need find_dependency and INTERFACE_LINK_LIBRARIES for static linkage?

[MonicaLiu0311]

Note: I will be converting your PR to draft status. When you're ready, please revert to "ready for review".

[rinrab]

Thank you! Today I'm going to fix these problems.

[dg0yt]

This is not entirely correct if users override linkage per port.
The correct way is to use target expressions as in

$<IF:$<TARGET_EXISTS:zstd::libzstd_shared>,zstd::libzstd_shared,zstd::libzstd_static>

(TBH I would prefer the unofficial config of apr to provide uniform target names...)

[MonicaLiu0311]

@rinrab Is there any new progress?

Please resolve the conflict. Finally, if you have finished editing, click “Ready for review”.