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 boost-multi library #21179

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

Conversation

correaa
Copy link

@correaa correaa commented Nov 17, 2023

Specify library name and version: b-multi/0.80.1

This is an attempt to the add the Multi library to Conan.
https://gitlab.com/correaa/boost-multi"


@CLAassistant
Copy link

CLAassistant commented Nov 17, 2023

CLA assistant check
All committers have signed the CLA.

@conan-center-bot

This comment has been minimized.

@uilianries
Copy link
Member

@correaa Thank you for your first contribution!

Please consider the follow points:

Please, read the contribution guide to get more information: https://github.com/conan-io/conan-center-index/tree/master/docs/adding_packages

@correaa
Copy link
Author

correaa commented Nov 17, 2023

done.

Regarding exporting local sources, I don’t understand what it means.
i believe my submission follows the header only template.
Am I missing something?

@correaa
Copy link
Author

correaa commented Nov 17, 2023

got it. i had a typo in the filename and i had to remove version number.

anything else?

@conan-center-bot

This comment has been minimized.

1 similar comment
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@correaa
Copy link
Author

correaa commented Nov 18, 2023

I see "403 Forbidden" when I try to look at the detail of the failed steps, such as ci3/conan-v2/pr-merge.

@uilianries
Copy link
Member

@correaa The output log is only available by the Bot comment above: #21179 (comment)

Those links listed on Gthub Check Status are not available.

The bot indicated that you were not in the authorized list yet, but you should be listed there now. I'll restart the CI build, it should run good now.

@conan-center-bot

This comment has been minimized.

@correaa
Copy link
Author

correaa commented Nov 27, 2023

assuming this will work, do you recommend having these configuration files in the main repository, e.g. in https://gitlab.com/correaa/boost-multi/-/blob/master/conan ?

I see other projects doing that.

@conan-center-bot

This comment has been minimized.

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

This comment has been minimized.

@uilianries
Copy link
Member

@correaa Hello again, after talking about this PR, we agreed that you can use boost-multi as package name. In case you have intention to submit to Boost, and in case accepted in the future, we just mark this recipe as deprecated, so people will use Boost instead.

The thing is, we had problems with boost-xxx packages, but when was an official package distribute, resulting in a collision when someone install boost and the boost-xxx. It's not your case, because your project is not official.

@correaa
Copy link
Author

correaa commented Nov 28, 2023

Thank you for the special considerations.

That's right, this is not an official Boost package.
I, however, got a preliminary endorsement to review it.

I will proceed to move the files and change the names to boost-multi.

Am I in the right track? I am talking about the failing test.

@uilianries
Copy link
Member

I will proceed to move the files and change the names to boost-multi.

Please, move to boost-multi, both the package folder name and the attribute name.

@uilianries
Copy link
Member

Am I in the right track? I am talking about the failing test.

No problem, let's rename, then I'll take a look in case of failure. So far, the recipe is looking good.

@correaa correaa changed the title Add bmulti library Add boost-multi library Dec 4, 2023
@conan-center-bot

This comment has been minimized.

correaa and others added 2 commits December 7, 2023 07:33
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Failure in build 23 (ef57107f90930ded9023d8a2529a071c2e5e40a4):

  • boost-multi/0.80.1:
    CI failed to create some packages (All logs)

    Logs for packageID 5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=gcc
    compiler.libcxx=libstdc++11
    compiler.version=9
    os=Linux
    
    [...]
    [build_requires]
    [env]
    [conf]
    tools.system.package_manager:mode=install
    tools.system.package_manager:sudo=True
    
    boost-multi/0.80.1: Forced build from source
    Installing package: boost-multi/0.80.1
    Requirements
        boost-multi/0.80.1 from local cache - Cache
    Packages
        boost-multi/0.80.1:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Build
    
    Installing (downloading, building) binaries...
    [HOOK - conan-center.py] pre_source(): [IMMUTABLE SOURCES (KB-H010)] OK
    boost-multi/0.80.1: Configuring sources in /home/conan/w/prod-v1/bsr/69052/cffda/.conan/data/boost-multi/0.80.1/_/_/source/src
    boost-multi/0.80.1: 
    [HOOK - conan-center.py] post_source(): [LIBCXX MANAGEMENT (KB-H011)] OK
    [HOOK - conan-center.py] post_source(): [CPPSTD MANAGEMENT (KB-H022)] OK
    [HOOK - conan-center.py] post_source(): [SHORT_PATHS USAGE (KB-H066)] OK
    boost-multi/0.80.1: Building your package in /home/conan/w/prod-v1/bsr/69052/cffda/.conan/data/boost-multi/0.80.1/_/_/build/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9
    boost-multi/0.80.1: Generator txt created conanbuildinfo.txt
    boost-multi/0.80.1: Aggregating env generators
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] 'fPIC' option not found
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] OK
    boost-multi/0.80.1: Calling build()
    boost-multi/0.80.1: Package '5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9' built
    boost-multi/0.80.1: Build folder /home/conan/w/prod-v1/bsr/69052/cffda/.conan/data/boost-multi/0.80.1/_/_/build/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/build-release
    boost-multi/0.80.1: Generated conaninfo.txt
    boost-multi/0.80.1: Generated conanbuildinfo.txt
    boost-multi/0.80.1: Generating the package
    boost-multi/0.80.1: Package folder /home/conan/w/prod-v1/bsr/69052/cffda/.conan/data/boost-multi/0.80.1/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9
    boost-multi/0.80.1: Calling package()
    boost-multi/0.80.1: Copied 110 '.hpp' files
    [HOOK - conan-center.py] post_package(): [DEFAULT PACKAGE LAYOUT (KB-H013)] OK
    [HOOK - conan-center.py] post_package(): [MATCHING CONFIGURATION (KB-H014)] OK
    [HOOK - conan-center.py] post_package(): [SHARED ARTIFACTS (KB-H015)] OK
    [HOOK - conan-center.py] post_package(): [STATIC ARTIFACTS (KB-H074)] OK
    [HOOK - conan-center.py] post_package(): [EITHER STATIC OR SHARED OF EACH LIB (KB-H076)] OK
    [HOOK - conan-center.py] post_package(): [PC-FILES (KB-H020)] OK
    [HOOK - conan-center.py] post_package(): [CMAKE-MODULES-CONFIG-FILES (KB-H016)] OK
    [HOOK - conan-center.py] post_package(): [PDB FILES NOT ALLOWED (KB-H017)] OK
    [HOOK - conan-center.py] post_package(): [LIBTOOL FILES PRESENCE (KB-H018)] OK
    [HOOK - conan-center.py] post_package(): [MS RUNTIME FILES (KB-H021)] OK
    [HOOK - conan-center.py] post_package(): [SHORT_PATHS USAGE (KB-H066)] OK
    [HOOK - conan-center.py] post_package(): [MISSING SYSTEM LIBS (KB-H043)] OK
    [HOOK - conan-center.py] post_package(): [APPLE RELOCATABLE SHARED LIBS (KB-H077)] OK
    boost-multi/0.80.1: WARN: This conanfile has no build step
    [HOOK - conan-center.py] post_package(): ERROR: [PACKAGE LICENSE (KB-H012)] No 'licenses' folder found in package: /home/conan/w/prod-v1/bsr/69052/cffda/.conan/data/boost-multi/0.80.1/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9  (https://github.com/conan-io/conan-center-index/blob/master/docs/error_knowledge_base.md#KB-H012-PACKAGE-LICENSE) 
    ERROR: [HOOK - conan-center.py] post_package(): Some checks failed running the hook, check the output
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

Comment on lines +2 to +4
"0.80.1":
url: https://gitlab.com/correaa/boost-multi/-/archive/v0.80.1/boost-multi-v0.80.1.tar.gz
sha256: 7d1ebee123783e0fadfb58a0f55309148d63295fcb4b9c5442d851e3578cad1e
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
"0.80.1":
url: https://gitlab.com/correaa/boost-multi/-/archive/v0.80.1/boost-multi-v0.80.1.tar.gz
sha256: 7d1ebee123783e0fadfb58a0f55309148d63295fcb4b9c5442d851e3578cad1e
"0.83.0":
url: "https://gitlab.com/correaa/boost-multi/-/archive/v0.83.0/boost-multi-v0.83.0.tar.gz"
sha256: "52324a01e25fd66a32c8bf88f41940ba6364fd218cfab53b7ec5c63115dea1bf"

I would recommend updating to 0.83.0, so the LICENSE file will be available.

@@ -0,0 +1,3 @@
versions:
"0.80.1":
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
"0.80.1":
"0.83.0":

@correaa
Copy link
Author

correaa commented Dec 11, 2023

Is it possible to have latest version of the library? is it not recommended? Ideally, I think the best is to have at least one named (versioned) version and latest (from master branch).

Is that ok?

@uilianries
Copy link
Member

@correaa You are the author, tell me you 😉

There is recommendation about version in CCI: https://github.com/conan-io/conan-center-index/blob/master/docs/adding_packages/conanfile_attributes.md#version

The recommendation is adding only the latest version, then add old versions in case someone needs and open an issue asking. Each version could generate +100 packages, so is important save some resource from CI when possible.

Consuming directly from master is allowed for those cases when the project is quiet for a long period, or, the project does not provide tags/releases.

In your case, I would recommend providing the latest version. In case you have something more to be added from master, you could generate a patch version in your project, then use it.

@correaa
Copy link
Author

correaa commented Dec 14, 2023

So a way to do this would be to always have a version (branch) called latest that is quasi in sync with master (e.g. manually maintained in github/gitlab) and add only that one to Conan?

After that what would remain to be done?

@uilianries
Copy link
Member

So a way to do this would be to always have a version (branch) called latest that is quasi in sync with master (e.g. manually maintained in github/gitlab) and add only that one to Conan?

Not mandatory, but preferable.

After that what would remain to be done?

Just update config.yml and conandata.yml with the new release. The CI should re-build the PR automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants