vcpkg icon indicating copy to clipboard operation
vcpkg copied to clipboard

[curl] Use sources from GitHub Releases to fix `LIBCURL_VERSION` and `LIBCURL_TIMESTAMP`

Open osipxd opened this issue 1 month ago • 6 comments

Is your feature request related to a problem? Please describe.

Currently when vcpkg checks out sources from GitHub, LIBCURL_VERSION has suffix "-DEV" and LIBCURL_TIMESTAMP is "[undefined]" Sources archives provided in GitHub Releases are produced using maketgz script which sets correct version and timestamp.

Proposed solution

Use sources from GitHub Releases instead of checking out a git repository.

Describe alternatives you've considered

No response

Additional context

For example, Conan uses sources from releases to build curl.

osipxd avatar Nov 27 '25 09:11 osipxd

I could try to contribute a fix if you approve the solution

osipxd avatar Nov 27 '25 09:11 osipxd

We can change the display strings, but I'm not convinced of switching away from vcpkg_from_github.

dg0yt avatar Nov 30 '25 09:11 dg0yt

We can change the display strings

Do you mean run some parts of the maketgz script? I suppose LIBCURL_TIMESTAMP will be equal to the date when the binary has been built then, not the curl release date?

I'm not convinced of switching away from vcpkg_from_github.

Are there any downsides of switching from it?

osipxd avatar Dec 03 '25 15:12 osipxd

Do you mean run some parts of the maketgz script? I suppose LIBCURL_TIMESTAMP will be equal to the date when the binary has been built then, not the curl release date?

I mean to outright remove the -DEV fragment and to set LIBCURL_TIMESTAMP string to either [vcpkg] or something more specific for that version. I acknowledge that this is not the releases timestamp, but it isn't pristine source or official binaries either.

Are there any downsides of switching from it?

Nothing dramatic.

  • Ignoring the dominant pattern.
  • Decoupling from pristine git sources. (git sources isolated the liblzma port from tarball manipulations.)
  • Disabling the availability of --head builds.
  • Making it a little bit more difficult to prepare port updates ahead of releases. (such as #48563).

OTOH I don't consider the gain from switching dramatic.

dg0yt avatar Dec 04 '25 06:12 dg0yt

I don't consider the gain from switching dramatic.

I agree that it doesn't worth it

osipxd avatar Dec 04 '25 17:12 osipxd

I will try to apply my suggested modifications to the next release. This issue might help as a reminder.

dg0yt avatar Dec 04 '25 21:12 dg0yt