Skip to content
This repository has been archived by the owner on May 24, 2019. It is now read-only.

Improvements to QtCreator docs #13

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

Conversation

timlinux
Copy link
Member

@timlinux timlinux commented Jan 8, 2017

  • Added cache
  • Switched to using CodeBlocs - Ninja (fixed by having /usr/local/bin in path)

Other small docs tweaks.

* Added cache
* Switched to using CodeBlocs - Ninja (fixed by having /usr/local/bin in path)

Other small docs tweaks.
@timlinux
Copy link
Member Author

timlinux commented Jan 8, 2017

@dakcarto thanks for applying my PR - sorry for not replying to all of the various comments from you and @m-kuhn. I'll also continue making tweaks as we go. I still miss a make install section so that it moves the .app bundles into ~/Applications (some of the instructions for PyCharm). I will be getting @ismailsunni and @lucernae to run through these notes over the next week or two so we can validate the whole process top to bottom and fix any holes.

@m-kuhn Thanks for your notes about build dir - I agree with @dakcarto in this case though because having an easy build path makes things a lot easier if you are trying to use that path from other tools / scripts. Worst of all QtCreator puts spaces in the path which is :-( ...

screen shot 2017-01-07 at 7 47 17 pm

### Making a kit
### Set up a ccache compile profile in QtCreator

CCache will cache previously compiled objects and greatly speed up the compilation process. We will make an alias for compiling c++ sources. Make sure you did ``brew install ccache`` when you were installing your dependencies further up in this document:
Copy link
Member

Choose a reason for hiding this comment

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

I'd strip the "when you were..." part. After all, if you haven't done it back then, you should do it now.

@@ -315,7 +335,7 @@ Press clone to make a copy and in the copy set it up with the following options:
* **Device type:** desktop
* **Device:** Local PC
* **Sys root:** Leave blank
* **Compiler:** Clang (x86_64bin in /usr/bin)
* **Compiler:** Cache for Clang (x86 64bit in /usr/bin)
Copy link
Member

Choose a reason for hiding this comment

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

Typo: CCache for Clang (as above)


```
PATH:$PATH:/usr/local/bin
PYTHONPATH:$PYTHONPATH:/usr/local/lib/python3.5/site-packages
Copy link
Member

Choose a reason for hiding this comment

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

3.6

@m-kuhn
Copy link
Member

m-kuhn commented Jan 8, 2017

Another nice round of improvements 👍

@m-kuhn
Copy link
Member

m-kuhn commented Jan 8, 2017

I agree with @dakcarto in this case though because having an easy build path makes things a lot easier if you are trying to use that path from other tools / scripts.

I think @dakcarto 's approach was a bit different to both of ours and will not involve cli intervention but a configuration change in QtCreator and override the (apparently bad) default choice with spaces etc.

@m-kuhn
Copy link
Member

m-kuhn commented Jan 8, 2017

screen shot 2017-01-08 at 14 18 41

@dakcarto
Copy link
Member

dakcarto commented Jan 8, 2017

Hi @timlinux and @m-kuhn. I worked quite a bit on this yesterday, up to the Install section. I specifically left out graphics, until the work is done. Sorry, I forgot to cross-reference it in the other PR before calling it a day.

It is in a separate branch tutorial-qtcreator. If you want I can move it to a PR, so we can discuss and compare/contrast as needed.

@timlinux
Copy link
Member Author

timlinux commented Jan 8, 2017

@m-kuhn Ah ok thanks - sorry I had missed that option.

@dakcarto

It is in a separate branch tutorial-qtcreator. If you want I can move it to a PR, so we can discuss and compare/contrast as needed.

Cool it looks like you made a PR now - I updated mine to use Python 3.6. Feel free to just apply yours if your changes are preferable to mine.

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

Successfully merging this pull request may close these issues.

3 participants