-
Notifications
You must be signed in to change notification settings - Fork 40
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
Series of changes to enable plugin to work on CLion 2016.3.2. #30
base: master
Are you sure you want to change the base?
Conversation
…cing with custom UI form. This should fix all errors in 2016.3.2
Updated compiled plugin.
This looks like it's intended to address #29 right? |
Guys, could you please merge this pull request and update the plugin in repository? Updated version works perfectly for me and I believe for others too. |
I am still encountering this bug as a user, six months after a PR has been submitted with the bug fix. |
The pull request is still open and has not been submitted yet. Maintainer
doesn't seem to be active. You'll have to manually patch or build from the
pull request branch.
…On Jul 28, 2017 6:30 AM, "Bobby Watson" ***@***.***> wrote:
I am still encountering this bug as a user, six months after a PR has been
submitted with the bug fix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#30 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABiuyKuk-836o6qbJcsQyRc2IpWWFE0oks5sSeJugaJpZM4LgMSq>
.
|
…a separate plugin on JetBrains repository.
…rk in JetBrains plugin repo.
Great Job on resurrecting this project! Build works fine. CLion 2017 found some bugs in the Arduino library. Having an issue with the edit window source library path. When I hit ALT Enter to fix dependency issues CLion inserts the whole path. #include "../../../../Program Files (x86)/Arduino/hardware/arduino/avr/cores/arduino/USBAPI.h" Where it should just add USBAPI.h It also it highlights the #include <Arduino.> in red. How do we tell configure CLion parser to use the correct Arduino source paths |
This PR fixed my issues as well. Highly suggest merging! CLion and Arduino is a very powerful combo! |
Hey @rjuang ! You can find the new arduino-cmake here: https://github.com/arduino-cmake/arduino-cmake Thank you! |
ah. appologies. This seemed to have gotten filtered and only spotted this
e-mail now. Been on a long break. It might take me a bit of time to figure
out how to update to the new arduino-cmake. Do you want to send a patch ?
…On Mon, Jan 15, 2018 at 9:59 AM, Felix E. ***@***.***> wrote:
Hey @rjuang <https://github.com/rjuang> !
Would it be possible for you to update your fork with the new
arduino-cmake project?
The one used here ist quite old (as you may know ;) and is broken for more
modern versions of the Arduino-SDK.
You can find the new arduino-cmake here: https://github.com/arduino-
cmake/arduino-cmake
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#30 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABiuyKbCDHgaZ3QndK_MB3WZbuN_g_xAks5tK5HogaJpZM4LgMSq>
.
|
This change updates code to utilize the API in CLion 2016.3.2. Because NewCMakeProjectStepAdapter was deprecated, this change adds a new class to replace the adapter. It makes use of a custom UI (simpler than original NewCMakeProjectStepAdapter) to create new projects.