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

Update getting_started.rst #1310

Closed
wants to merge 1 commit into from
Closed

Conversation

Titonus
Copy link
Contributor

@Titonus Titonus commented May 31, 2019

-settings arch=x86 v -settings arch=x86_64 when trying 32 bit binaries

@@ -378,11 +378,11 @@ For example, if we have a profile with a 32-bit GCC configuration in a profile c
We strongly recommend using :ref:`profiles` and managing them with :ref:`conan_config_install`.

However, the user can always override the default profile settings in the :command:`conan install` command using the :command:`--settings`
parameter. As an exercise, try building the Encrypter project 32-bit version:
parameter. As an exercise, try building the Encrypter project 32-bit version, after creating a new profile (gcc_x86):
Copy link
Contributor

Choose a reason for hiding this comment

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

I agree, this part is very confusing, but I don't know if we fix it only with that change.

Copy link
Member

Choose a reason for hiding this comment

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

I think the confusion comes from the name of the profile. Will try to push some changes here. Thanks for noticing @Titonus!

@danimtb
Copy link
Member

danimtb commented May 31, 2019

Suggested changes in Titonus#1

@danimtb
Copy link
Member

danimtb commented Jun 24, 2019

As this seems to be inactive, I will collect the changes proposed in Titonus#1 and open a new PR. Thanks anyway!

@danimtb
Copy link
Member

danimtb commented Jun 25, 2019

closing this in favor of #1338. It was already merged! Thanks again @Titonus

@danimtb danimtb closed this Jun 25, 2019
@Titonus
Copy link
Contributor Author

Titonus commented Jun 25, 2019

De nada ;-)

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