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.

Sign up for GitHub

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

Jump to bottom

[question] python_requires and build infos #73

Open
1 task
dazze1812 opened this issue Oct 6, 2023 · 4 comments · May be fixed by #75
Open
1 task

[question] python_requires and build infos #73

dazze1812 opened this issue Oct 6, 2023 · 4 comments · May be fixed by #75
Assignees
@memsharded

Comments

@dazze1812
Copy link

What is your question?

Hi!
the dependency graph of a python_require package does not include any modules. There is only node "0".

{
    "graph": {
        "nodes": {
            "0": {
                "ref": "conanfile",
                "id": "0",
                "recipe": "Cli",
                ...
            }
        },
        "root": {
            "0": "None"
        },
        "overrides": {},
        "resolved_ranges": {}
    }
}

This results in a build info (created with "conan art:build-info create") not containing any modules.

When uploading the build info to Artifactory, the build is not linked to the package files in the repository and it is not possible to promote such a build.

How can I promote builds of python_require packages in Artifactory?

Have you read the CONTRIBUTING guide?

  • I've read the CONTRIBUTING guide
@memsharded memsharded self-assigned this Oct 6, 2023
@memsharded
Copy link
Member

Hi @dazze1812

Thanks for your question.

The observation is correct. The python_requires are not regular packages, they do not have binaries, and they do not constitute a "node" in the dependency graph (it is a C/C++ packages dependency graph).

We have recently added full information about python_requires in the serialized graph.json. So I suspect that what you are seeing might be a gap in the actual conan art:build-info command, not in Conan itself, lets move this ticket to the conan extensions repo where this belongs.

@memsharded memsharded transferred this issue from conan-io/conan Oct 6, 2023
@memsharded
Copy link
Member

I am having a look, indeed the extension is not modeling python_requires.
There is a gap in the serialization of python_requires, that will need to be added in next 2.0.X, I'll manage that

@memsharded
Copy link
Member

Implemented conan-io/conan#14886 for next Conan 2.0.14

@memsharded memsharded mentioned this issue Oct 9, 2023
Closed
@memsharded
Copy link
Member

The fix for Conan client was merged, started to fix the buildInfo command in #74

@memsharded memsharded linked a pull request Oct 9, 2023 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@memsharded memsharded

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@memsharded @dazze1812