Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tinyfiledialogs] Fix for good #19338

Closed
wants to merge 4 commits into from

Conversation

strega-nil
Copy link
Contributor

@strega-nil strega-nil commented Aug 3, 2021

This stabilizes the source access for tinyfiledialogs, so we no longer have to worry about it being inaccessible or changed.

It adds a new parameter to vcpkg_from_git, FETCH_REF, which tells vcpkg to instead fetch that branch in a non-shallow way. In almost all cases, unless the upstream rebases their entire history, this will allow us to avoid the issue of downloading non-advertised objects, which many git servers do not support.

This was originally part of #19207, but I didn't want to have to rebuild the world on that PR.

@strega-nil-ms strega-nil-ms added category:port-bug The issue is with a library, which is something the port should already support info:next-rollup labels Aug 3, 2021
scripts/cmake/vcpkg_from_git.cmake Show resolved Hide resolved
scripts/cmake/vcpkg_from_git.cmake Outdated Show resolved Hide resolved
scripts/cmake/vcpkg_from_git.cmake Show resolved Hide resolved
@JackBoosY JackBoosY added the info:internal This PR or Issue was filed by the vcpkg team. label Aug 4, 2021
@strega-nil-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@strega-nil-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@strega-nil-ms
Copy link
Contributor

Closed for rollup #19469.

strega-nil-ms pushed a commit to strega-nil/vcpkg that referenced this pull request Aug 11, 2021
[tinyfiledialogs] Fix for good
strega-nil-ms added a commit that referenced this pull request Aug 16, 2021
* [rollup:2021-08-09] PR #16706 (@JackBoosY)

[vcpkg_fixup_cmake_targets] Fix up OSX system development path

* [rollup:2021-08-09] PR #19238 (@strega-nil)

[scripts-audit] vcpkg_download_distfile

* [rollup:2021-08-09] PR #19239 (@strega-nil)

[scripts-audit] vcpkg_find_fortran

* [rollup:2021-08-09] PR #19338 (@strega-nil)

[tinyfiledialogs] Fix for good

* [rollup:2021-08-09] PR #19348 (@strega-nil)

[scripts-audit] vcpkg_fixup_pkgconfig

* fix ports.cmake with newer vcpkg_download_distfile

* fix vcpkg create

* move vcpkg_common_definitions down so that it's not incorrect

* fix vcpkg_internal_get_cmake_vars

Co-authored-by: nicole mazzuca <mazzucan@outlook.com>
Co-authored-by: Jack·Boos·Yu <47264268+JackBoosY@users.noreply.github.com>
Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com>
mahge added a commit to mahge/vcpkg that referenced this pull request Nov 3, 2021
  - The directory specified no longer matches the directory used for
    fetching.

  - This was overlooked by PR microsoft#19338.
BillyONeal added a commit that referenced this pull request Nov 26, 2021
- The directory specified no longer matches the directory used for
    fetching.

  - This was overlooked by PR #19338.

Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants