[proxygen]: fix dependency

[jiayuehua]

Describe the pull request

• What does your PR fix?

  Fixes #...

• Which triplets are supported/not supported? Have you updated the CI baseline?

  , <Yes/No>

• Does your PR follow the maintainer guide?

  yes

• If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

[github-actions]

You have modified or added at least one portfile where deprecated functions are used.

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

• ports/proxygen/portfile.cmake

You have modified or added at least one vcpkg.json where a "license" field is missing.
If you feel able to do so, please consider adding a "license" field to the following files:

• ports/proxygen/vcpkg.json

Valid values for the license field are listed at https://spdx.org/licenses/

[JackBoosY]

Why removed this dependency?

[jiayuehua]

this dependency is not used.

[JackBoosY]

Can you please confirm this with the upstream first?

Thanks.

[jiayuehua]

ok facebook/proxygen#396

[ras0219-msft]

This change 99% LGTM, however zstd::libzstd_static will only be defined if zstd was built statically.

Therefore, we either need to add logic to proxygen/lib/CMakeLists.txt to determine which target should be linked (zstd::libzstd_static vs zstd::libzstd_shared) or we should wait for a resolution to #22822 which will hopefully define both targets always.

[jiayuehua]

I see. I also think waiting for that patch resolution is better. not merging now is right decision.

[JackBoosY]

Since #22822 was merged, can we continue this PR?

[jiayuehua]

I will have a look soon. My daily job is a bit busy at every end of the seasons.

[github-actions]

You have modified or added at least one vcpkg.json where a "license" field is missing.

If you feel able to do so, please consider adding a "license" field to the following files:

• ports/proxygen/vcpkg.json

Valid values for the license field can be found in the documentation

[jiayuehua]

hi, @JackBoosY . I have updated my pr.

[strega-nil-ms]

Thanks!