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

Added basic appveyor CI support #41

Closed
wants to merge 1 commit into from
Closed

Added basic appveyor CI support #41

wants to merge 1 commit into from

Conversation

Telokis
Copy link
Contributor

@Telokis Telokis commented Sep 20, 2016

Will simply build vcpkg and, then, install each of the libraries available in ports.
Output can be seen here
Fails because cocos2d doesn't have a CONTROL file.

@msftclas
Copy link

Hi @Ninetainedo, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

@ras0219-msft
Copy link
Contributor

I think we should parse the output of ".\vcpkg search" to get the list instead of hardcoding it. Additionally, we should remove cocos-2d entirely (for now) since it does not work.

@Telokis
Copy link
Contributor Author

Telokis commented Sep 20, 2016

We could parse the output but wouldn't it be better to try installing everything located in ports?
This way we can easily spot ill-registered libraries. (As cocos2d, for instance)

@Telokis
Copy link
Contributor Author

Telokis commented Sep 20, 2016

Earlier I tried to ignore failing libraries and I got about one full hour to install every library.
This obviously won't scale well with hundreds of libraries.
The ideal would be to only try installing libraries that have changed. If the portfile.cmake is different, the library is installed. Otherwise, we skip it.

However, I have no idea how to do so efficiently for now. This could probably be done by a PowerShell script based on portfile's MD5 but the script would need a way to update said MD5.

@ras0219-msft
Copy link
Contributor

Hmm, yeah. We could run regex against each portfile to select the version number and compare it against either the output of vcpkg list or packages\<xyz>\CONTROL.

I definitely agree that as the libraries number increases, we need to have a better strategy here. I'm quite impressed it finished in an hour, honestly.

@Telokis
Copy link
Contributor Author

Telokis commented Sep 22, 2016

I'm closing this PR and I'll open an issue about testing.

@Telokis Telokis closed this Sep 22, 2016
@Telokis Telokis mentioned this pull request Sep 22, 2016
strega-nil added a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
strega-nil added a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
The following PRs are included:

* hopefully fix crash in constraints (microsoft#60)
* [vcpkg] allow --version to check the version (microsoft#50)
* Remove baseline warning (microsoft#27)
* [git] always pass autocrlf=false (microsoft#58)
* ignore QtCreator CMake project files (microsoft#54)
* ignore .DS_store files (microsoft#53)
* [vcpkg] x-add-version now also checks if the manifest file is properly formatted (microsoft#43)
* hopefully fix ci issue microsoft#16773 (microsoft#34)
* Add docs to set VCPKG_ROOT to run tests (microsoft#45)
* [vcpkg] x-add-version improve speed by calling get_builtin_baseline only once (microsoft#44)
* add clang-format version to format-cxxcode (microsoft#41)
* [vcpkg] Introduce experimental workaround X_VCPKG_NUGET_ID_PREFIX (microsoft#40)
* [supports] Add `native` identifier expression and x-check-support command (microsoft#29)
* [metrics] Split reporting of installs into name:triplet (microsoft#39)
* [vcpkg] Improve error when accessing missing feature (microsoft#38)
* [vcpkg] Allow shallow git registries (microsoft#37)
* Disable git autocrlf when archiving tree (microsoft#36)
* Use only named packages from extra registries (microsoft#35)
* [registries] add metrics (microsoft#30)
* Add vcpkg policy cmake helper port support (microsoft#17)
* [osx] add support for rosetta (microsoft#23)
* don't build tls12-download unless it's needed (microsoft#33)
* Add new telemetry points for versioning (microsoft#21)
* add cmake_minimum_required to vcpkg_tags (microsoft#25)
* [x-add-versions] Perform atomic replacement of versioning files (microsoft#28)
* [tools] support gsutil (microsoft#19)
* add CUDA 11.1 and 11.2 to KEEP_ENV_VARS defaults (microsoft#26)
* Add finite timeout on CURL metrics endpoint. (microsoft#22)
* fix UB in make_error_code(utf8_errc) (microsoft#18)
JoergAtGithub pushed a commit to JoergAtGithub/vcpkg that referenced this pull request Jul 2, 2022
dempo93 pushed a commit to dempo93/vcpkg that referenced this pull request Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants