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

Tutorial for local flow #2968

Merged
merged 10 commits into from
Feb 16, 2023
Merged

Conversation

czoido
Copy link
Contributor

@czoido czoido commented Feb 15, 2023

And also placeholders for some missing commands

Examples: conan-io/examples2#72

Download (without install) a single conan package from a remote server.

It downloads just the package, but not its transitive dependencies, and it will not call
any generate, generators or deployers.
Copy link
Contributor

Choose a reason for hiding this comment

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

generate and generators? a typo, perhaps?

-h, --help show this help message and exit
-v [V] Level of detail of the output. Valid options from less verbose to more verbose: -vquiet, -verror,
-vwarning, -vnotice, -vstatus, -v or -vverbose, -vv or -vdebug, -vvv or -vtrace
--logger Show the output with log format, with time, type and message.
Copy link
Contributor

Choose a reason for hiding this comment

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

it outputs in json format, is it expected?

conan download --logger -r conancenter  zlib/1.2.11
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:56.822022+00:00", "data": "Downloading recipe 'zlib/1.2.11#ffa77daf83a57094149707928bdce823'"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:59.592272+00:00", "data": "Downloading conan_export.tgz"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:59.602051+00:00", "data": "Downloading conanmanifest.txt"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:59.604367+00:00", "data": "Downloading conanfile.py"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:59.984767+00:00", "data": "Decompressing conan_export.tgz"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:53:59.988140+00:00", "data": "Downloading 'zlib/1.2.11' sources"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:54:00.474447+00:00", "data": "Downloading conan_sources.tgz"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:54:00.475827+00:00", "data": "Decompressing conan_sources.tgz"}}
{"json": {"level": "STATUS", "time": "2023-02-15T15:54:00.476446+00:00", "data": "Sources downloaded from 'conancenter'"}}

also, it could be better to specify what is time (is it local, UTC, what's the format)

[-s:b SETTINGS_BUILD] [-s:h SETTINGS_HOST] [-c CONF_HOST] [-c:b CONF_BUILD] [-c:h CONF_HOST]
path

Create a package directly from pre-compiled binaries
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Create a package directly from pre-compiled binaries
Creates a package directly from pre-compiled binaries

$ conan download --help
usage: conan download [-h] [-v [V]] [--logger] [--only-recipe] [-p PACKAGE_QUERY] -r REMOTE reference

Download (without install) a single conan package from a remote server.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Download (without install) a single conan package from a remote server.
Downloads (without install) a single conan package from a remote server.


positional arguments:
{add,list,remove} sub-command help
add Define the given <path> location as the package <reference>, so when this package is required, it is
Copy link
Contributor

Choose a reason for hiding this comment

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

it's a bit unclear how and where do I specify path and reference

--lockfile-packages Lock package-id and package-revision information
--lockfile-clean remove unused
-o OPTIONS_HOST, --options OPTIONS_HOST
Define options values (host machine), e.g.: -o Pkg:with_qt=true
Copy link
Contributor

Choose a reason for hiding this comment

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

it doesn't work, as options format has changed in v2:

conan export-pkg . --version 1.2.11 -o zlib:shared=True  
Using lockfile: '/home/sse4/projects/conan-center-index/recipes/zlib/all/conan.lock'
ERROR: Error while initializing options. The usage of package names `zlib:shared` in options is deprecated, use a pattern like `zlib/*` or `zlib*` instead

-o:h OPTIONS_HOST, --options:host OPTIONS_HOST
Define options values (host machine), e.g.: -o:h Pkg:with_qt=true
-pr PROFILE_HOST, --profile PROFILE_HOST
Apply the specified profile to the host machine
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it specifies both host and build at the same time per conan-io/tribe#23


positional arguments:
path Path to a folder containing a recipe (conanfile.py or conanfile.txt) or to a recipe file. e.g.,
./my_project/conanfile.txt.
Copy link
Contributor

Choose a reason for hiding this comment

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

what is it relative to?


positional arguments:
path Path to a folder containing a recipe (conanfile.py or conanfile.txt) or to a recipe file. e.g.,
./my_project/conanfile.txt.
Copy link
Contributor

Choose a reason for hiding this comment

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

also, what's the effect of calling conan source on conanfile.txt? it doesn't have a source method, right?

Copy link
Contributor Author

@czoido czoido Feb 15, 2023

Choose a reason for hiding this comment

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

This is and the rest are just copy&paste from the --help of the commands. We will check all of them in the conan repo in case something needs changes but they can't be addressed in this one.

@czoido czoido merged commit 4d8b82e into conan-io:release/2.0.0-beta Feb 16, 2023
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