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

add platform.delegates/0.1.3 #6063

Merged
merged 54 commits into from
Jul 12, 2021
Merged

Conversation

uselessgoddess
Copy link
Contributor

@uselessgoddess uselessgoddess commented Jun 25, 2021

Specify library name and version: platform.delegates/0.1.3

platform.delegates is one of the libraries of the LinksPlatform modular framework, which uses innovations from the C++17 standard, for easier use delegates/events in csharp style.


  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

uselessgoddess and others added 30 commits June 16, 2021 16:56
[for update pull request]
- Fixed source-function: used shorter one syntax for get sources
- Added required conan version [>=1.33.0]
Co-authored-by: Javier G. Sogo <jgsogo@gmail.com>
Co-authored-by: Javier G. Sogo <jgsogo@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
@uilianries
Copy link
Member

package_info is missing.

@@ -65,3 +65,6 @@ def package(self):

def package_id(self):
self.info.header_only()

def package_info(self):
pass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pass
self.cpp_info.names["cmake_find_package"] = "Platform.Delegates"
self.cpp_info.names["cmake_find_package_multi"] = "Platform.Delegates"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😅😅😅
#6063 (comment)

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@Konard
Copy link

Konard commented Jul 9, 2021

Hello everyone :)
So can we start the review process to add this package to the Conan index? :)
We would like to make each library separate. That is what we mean by modular framework. Each library is independent and can be used by its own using the least possible dependencies besides STL.
We also would like to use platform. prefix as long no one else using it.
As we do with merged platform.interfaces and with all our packages in NuGet: https://www.nuget.org/profiles/konard

@conan-center-bot

This comment has been minimized.

Copy link
Contributor

@prince-chrismc prince-chrismc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

recipes/platform.delegates/all/conanfile.py Outdated Show resolved Hide resolved
recipes/platform.delegates/all/conanfile.py Outdated Show resolved Hide resolved
uselessgoddess and others added 2 commits July 10, 2021 09:08
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

prince-chrismc
prince-chrismc previously approved these changes Jul 10, 2021
@conan-center-bot

This comment has been minimized.

@uselessgoddess
Copy link
Contributor Author

Oh... Funny situation

@conan-center-bot
Copy link
Collaborator

All green in build 18 (3741165df5caf41efe89c39f516974c086b8a7e9):

  • platform.delegates/0.1.3@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit 1d4acd1 into conan-io:master Jul 12, 2021
AndreyMlashkin pushed a commit to AndreyMlashkin/conan-center-index that referenced this pull request Jul 19, 2021
* Added platform.interfaces package

* Small rework architecture

* Move platform-interfaces to platfrom.interfaces

* Delete recipes/platform-interfaces directory

* Other small fixes

* Added empty line

* Fix register error

* Fixed version for new release

* Fixed version for new release

* Update to [0.1.0]

* [0.1.1] - is main version

* Added  platform.interfaces test_package

* Update to 0.1.2 version

* Update to 0.1.2 version

* fixed errors in the test_package.cpp 

[for update pull request]

* Removing empty line in the CMakeListst.txt

[partial commits]

* Removing empty line in the config.yml

[partial commits]

* Removing empty line problems

* Fixed validating c++ standard

* Added apple clang validation and fixes VS version

* I believe in Python

* conanfile reviewing

- Fixed source-function: used shorter one syntax for get sources
- Added required conan version [>=1.33.0]

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Javier G. Sogo <jgsogo@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Javier G. Sogo <jgsogo@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Update recipes/platform.interfaces/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Cancel change name

* Add platform.delegates 0.1.2

* Add platform.delegates 0.1.2
update conanfile.py

* Resolving merge

* Update CMakeLists.txt

* Update test_package.cpp

* Update config.yml

* Oh no. since C++20

* Add platform.delegates 0.1.3

* Update config.yml

* Added unknown compiler warning

* Update recipes/platform.delegates/all/test_package/test_package.cpp

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Internal cpp sources

* Update recipes/platform.delegates/all/conanfile.py

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Update conanfile.py

* Update recipes/platform.delegates/all/conanfile.py

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Update recipes/platform.delegates/all/conanfile.py

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Remove package_info

* Add only-developers packages versions

* Revert "Add only-developers packages versions"

This reverts commit 9c61391

Co-authored-by: Javier G. Sogo <jgsogo@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
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.

7 participants