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

Feature/#2 conan.io based implementation #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Jun 12, 2018

@paul-michalik

vcpkgio

@ghost ghost requested a review from paul-michalik June 13, 2018 14:49
@ghost
Copy link
Author

ghost commented Jun 13, 2018

@paul-michalik
Please have a look on this implementation.
Currently it has following features:

  1. it support uploading/download vcpkg binaries to/from bintray through conan
  2. it provides setting file which can be customized based on specific requirement.
  3. it is also not specific to conan , anybody can use different remote client e.g nuget for uploading and downloading binaries.
  4. it supports bundle based artifacts which means a bundle can comprise of many packages.
  5. so user can upload/download a single bundle which will provide all the packages for his requirement..
  6. it also support merging among downloaded bundles but it should particular unique_id.
  7. unique_id should be define to make sure of bundle compatibility. currently commit_id is provided as unique_id
  8. vcpkg.io has 3 main tools: git_tools, vcpkg_tools and remote_tools which can be used independently.
  9. git_tools.py is responsible for downloading vcpkg source , pulling vcpkg source and get the specific commit of the vcpkg source.
  10. vcpkg_tools.py is responsible for handle all vcpkg operation.
  11. remote_tools.py has a factory class which support other remote clients too. currently conan is used for downloading and uploading vcpkg binaries.

@ghost ghost force-pushed the feature/#2-conan.io-based-implementation branch from 75157e8 to c1fb9f0 Compare July 4, 2018 07:46
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.

0 participants