-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
CLion 2018.1.6 requires drake/tools/clion/bazel_wrapper (re-titled) #9282
Comments
All issues must have owners. @sherm1 could you suggest one for this one? (@jwnimmer-tri or @amcastro-tri ?) |
Assigned to @jwnimmer-tri for now. |
I attempted to reproduce this:
Almost all symbols seem to work fine (i.e., I cannot repro this issue). I see in To make progress here, I'll need someone who is experiencing the problem to nuke their setup and screenshare me through their full installation steps and demonstration of the problem. My best guess at this point is that there is some installation step that we're doing differently, that is under-documented. |
@amcastro-tri did a fresh Clion 2018.1.6 install on a new computer, and it seems to be resolving symbols just fine so far (no more than the usual 0.1% of red squiggles). Assuming his setup keeps working, I think I will have to do live screenshare debugging with someone who is experiencing this, to make progress. |
@amcastro-tri and I have identical machines so this is a disappointing result! Alejandro, please look in file class Context : public ContextBase always showed ContextBase in red. Is yours actually right? |
Now I'm wondering if I did something wrong in importing my bazel project. The instructions say:
I'm not sure what's meant by a "project data directory". There is a Also, I have CLion installed in /opt/clion-1018.1.6 which is not a writable directory (however that's where I have CLion 2017.3.4 installed also). |
Oh, rats. I found the "project data directory" -- it is the first field in the form and I had been ignoring it thinking it was just the project directory (because it defaults to that directory). That violates the above instruction! So I fixed it to point somewhere outside the Next thought: I have my Drake repo in directory |
Alejandro did not change the Alejandro had Alejandro and I both had CLion unpacked within |
Per slack -- the problem was that CLion wasn't using |
@sherm1, Re the fact that ... until you realize it actually took you to some wacky virtual directory. Aghh!!! I don't know if this helps. Bottom line, @sherm1's problem is not solved in my installation of CLion and who knows what else is broken. FTR, it works perfect in CLion 2017.2.3 with Bazel plugin 2017.11.20.0.4 |
@amcastro-tri Did you fix your setup to use |
nope, sry, going through my github emails in the order they arrive and I just saw your PR. I'll try next |
Although our CLion instructions as of PR #9212 claim that CLion 2018.1.6 can be used with Drake, some installations using the recommended combination of ingredients fail to find many Drake symbols, leaving them red and preventing navigation. I saw this problem on a brand-new Puget running Ubuntu 16.04, and it has also been reported on an intern's new laptop installation. OTOH, when I upgraded my own laptop to 2018.1.6 it has no trouble finding symbols.
Edit: I am seeing red in some locations on my laptop also, including
examples/multibody/cart_pole/cart_pole_passive_simulation.cc
, while everything is fine inexamples/multibody/cart_pole/bouncing_ball_run_dynamics.cc
.On the virgin machine I installed Bazel 0.16.1, Plug in 2018.06.11.0.3, and CLion 2018.1.6. No combination of clearing cache, re-syncing, and importing a new project resulted in the symbols being found. Dropping back to 2017.3.4 and its matching plugin worked fine, even using the same CLion project for which 2018.1.6 could not find symbols. It may also be important to note that symbols were located correctly in many Drake subdirectories, but were missing in many others (I was mostly looking in systems/framework, for example see context.h, but there were red symbols in other locations also).
Here is the link to a Slack discussion on this topic.
The text was updated successfully, but these errors were encountered: