Skip to content

Error add_ok #1

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

Closed
stephpy opened this issue Apr 18, 2012 · 3 comments · Fixed by #2
Closed

Error add_ok #1

stephpy opened this issue Apr 18, 2012 · 3 comments · Fixed by #2

Comments

@stephpy
Copy link
Contributor

stephpy commented Apr 18, 2012

Hi, i've an error when i launch :Phpcs or :Phpmd, i think that's the same issue on Phpcc ...

The problem is there:

exe ":sign place 4782 name=" . sign . " line=" . line . " file=\".expand(\"%:p\")"
#E158: Invalid buffer name: ".expand("%:p")

I fix quote by doing:

exe ":sign place 4782 name=" . sign . " line=" . line . " file=\"".expand("%:p")."\""
#E158: Invalid buffer name: ""

I'm not pro in vim and may you have an idea to retrieve correct buffer name or may it's related to my configuration:

Mac OS X 10.6
VIM 7.3

Have you any idea ?

Thank you.

@joonty
Copy link
Owner

joonty commented Apr 18, 2012

Hi, thanks for raising that. The reason it isn't working for you is that there are two modes that the script can run in: one with perl, one in normal vim. It uses perl if possible for speed, and it's part of the quickhigh vim plugin that I incorporated into my plugin.

Basically it looks like your Vim installation isn't compiled with perl support. You can either try recompiling, or you can wait for me to find the time to fix it :)

If you're using macports, you can uninstall the current vim and re-install with perl support (and more) with:

sudo port install vim +perl +python +tcl +ruby

Hope that helps.

Thanks

@stephpy
Copy link
Contributor Author

stephpy commented Apr 18, 2012

Han ! You're right, i have not perl extension, i'll compile my vim to see what's the benefit :)

IMHO it would be great to disable this method if vim is not compiled with perl to not throw this error, i'll make a PR if you think that's useful.

If i have time, i'll look at how to fix this bug

@joonty
Copy link
Owner

joonty commented Apr 18, 2012

Yeh that's great if you're happy to try and fix it! I'll also take a look
when I can.

On Apr 18, 2012 5:02 PM, "Stéphane PY" <
[email protected]>
wrote:

Han ! You're right, i have not perl extension, i'll compile my vim to see
what's the benefit :)

IMHO it would be great to disable this method if vim is not compiled with
perl to not throw this error, i'll make a PR if you think that's useful.

If i have time, i'll look at how to fix this bug


Reply to this email directly or view it on GitHub:
#1 (comment)

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 a pull request may close this issue.

2 participants