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

rtags-rdm-includes crash rdm start #1353

Closed
SimeonEhrig opened this issue Nov 24, 2019 · 2 comments

Comments

@SimeonEhrig
Copy link

@SimeonEhrig SimeonEhrig commented Nov 24, 2019

Describe the bug
I want to set the include path for the std C++ lib at start of rdm. So I added the following line to my init file (setq rtags-rdm-includes "/usr/include/c++/5.5.0"). I use the c++-mode-hook that starts the rdm server when I open a .cpp file. Without rtags-rdm-includes the rdm server is starts and I can use rtags commands. If I set the variable, I get the error message File mode specification error: (wrong-type-argument sequencep 47) in my minibuffer and rdm is not started.

To Reproduce
Steps to reproduce the behavior:

  1. Add to your init file:
;; change the path depending on your system 
(setq rtags-rdm-includes "/usr/include/c++/5.5.0")
(add-hook 'c++-mode-hook 'rtags-start-process-unless-running)
  1. Reload the init file or restart emacs
  2. Open a .cpp file
  3. Look at the minibuffer or Message buffer

Expected behavior
The rdm server should start when I open a .cpp file and the include path should be set.

Environment (please complete the following information):

  • OS/Version: Ubuntu 16.04.6
  • RTags(rdm,rc): 2.34
  • Emacs: 26.3
  • LLVM/Clang: 8.0

Additional context
At the moment, I'm using (setq rtags-process-flags "-I/usr/include/c++/5.5.0) as workaround.

@cslux

This comment has been minimized.

Copy link
Collaborator

@cslux cslux commented Nov 25, 2019

The :type of rtags-rdm-includes is wrong. rtags-rdm-includes MUST be a list of strings not a string.

(setq rtags-rdm-include '("/usr/include/c++/5.5.0"))
@cslux cslux closed this in ca131c1 Nov 25, 2019
@cslux

This comment has been minimized.

Copy link
Collaborator

@cslux cslux commented Nov 25, 2019

@SimeonEhrig Fixed via ca131c1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.