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 environment variable CONAN_DEFAULT_PROFILE to define path to default profile #3615

Merged
merged 9 commits into from Oct 1, 2018

Conversation

Projects
None yet
4 participants
@jgsogo
Copy link
Member

commented Sep 25, 2018

close #3193

Changelog: Feature: The environment variable CONAN_DEFAULT_PROFILE_PATH allows the user to define the path (absolute and existing) to the default profile that will be used by Conan.

@jgsogo

This comment has been minimized.

Copy link
Owner Author

commented on conans/client/migrations.py in c58d5ab Aug 30, 2018

This is no longer a problem as this code has been deleted (it was not used)

@jgsogo jgsogo added this to the 1.9 milestone Sep 25, 2018

@jgsogo jgsogo requested a review from lasote Sep 25, 2018

@ghost ghost assigned jgsogo Sep 25, 2018

@ghost ghost added the stage: review label Sep 25, 2018

@lasote lasote modified the milestones: 1.9, 1.8 Sep 25, 2018

@lasote
Copy link
Contributor

left a comment

I think CONAN_DEFAULT_PROFILE_PATH would be better.

@lasote

This comment has been minimized.

Copy link
Contributor

commented Sep 25, 2018

1.8
Add the docs.

@jgsogo

This comment has been minimized.

Copy link
Member Author

commented Sep 25, 2018

@lasote, is this expected behaviour? profile2 is not listed but it is shown:

(python3) MacBook-Pro-de-Javier:conan jgsogo$ CONAN_DEFAULT_PROFILE_PATH=/Users/jgsogo/dev/conan/conan/default2 conan profile list
default
default.old
gcc49

(python3) MacBook-Pro-de-Javier:conan jgsogo$ CONAN_DEFAULT_PROFILE_PATH=/Users/jgsogo/dev/conan/conan/default2 conan profile show default2
Configuration for profile default2:

[settings]
os=Macos
os_build=Macos
arch=x86_64
arch_build=x86_64
compiler=apple-clang
compiler.version=9.1
compiler.libcxx=libc++
build_type=Release
[options]
[build_requires]
[env]

@danimtb

This comment has been minimized.

Copy link
Member

commented Sep 25, 2018

@jgsogo Conan help output seems OK with the behavior of conan profile list:

λ conan profile --help
usage: conan profile [-h] {list,show,new,update,get,remove} ...

Lists profiles in the '.conan/profiles' folder, or shows profile details. The
'list' subcommand will always use the default user 'conan/profiles' folder.
But the 'show' subcommand is able to resolve absolute and relative paths, as
well as to map names to '.conan/profiles' folder, in the same way as the '--
profile' install argument.

positional arguments:
  {list,show,new,update,get,remove}
    list                List current profiles
    show                Show the values defined for a profile
    new                 Creates a new empty profile
    update              Update a profile with desired value
    get                 Get a profile key
    remove              Remove a profile key

optional arguments:
  -h, --help            show this help message and exit

However the second output seems weird to me. Are you sure your default2 profile is not in the current working dir?

@jgsogo

This comment has been minimized.

Copy link
Member Author

commented Sep 25, 2018

You are completely right, @danimtb. default2 profile was in my working directory.

@lasote

lasote approved these changes Sep 26, 2018

@lasote lasote requested a review from memsharded Sep 26, 2018

@danimtb danimtb assigned memsharded and unassigned jgsogo Sep 27, 2018

@memsharded memsharded merged commit 81306cc into conan-io:develop Oct 1, 2018

2 checks passed

continuous-integration/jenkins/pr-head This commit looks good
Details
license/cla Contributor License Agreement is signed.
Details

@ghost ghost removed the stage: review label Oct 1, 2018

@jgsogo jgsogo deleted the jgsogo:issue/3193 branch Oct 2, 2018

grisumbras pushed a commit to grisumbras/conan that referenced this pull request Dec 27, 2018

Merge pull request conan-io#3615 from jgsogo/issue/3193
Add environment variable CONAN_DEFAULT_PROFILE to define path to default profile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.