Latest commit does not fix the issue properly #6

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants
Contributor

lzap commented Apr 5, 2013

The latest commit 9695af3 fixes the case when there is already a directory named "tags". It works for cases when the file was not present and when there was a directory, but it does not work for cases when file is present.

@lzap lzap fixing previous commit for directory named tags
The previous commit was not working properly for all cases.

I have to admit I was drawing a boolean table to solve this issue :-)
aa6b1f6

lzap referenced this pull request Apr 5, 2013

Closed

Error with ruby 2.0.0p0 #5

jroes commented Jun 4, 2013

👍

Contributor

lzap commented Jun 14, 2013

Bump. Please note the referenced issue was closed, but this PR is still valid.

bambery commented Jul 9, 2013

I just installed gem-ctags today and when it hits backports, I am getting the same unhelpful error "ERROR: While executing gem ... (Errno::EISDIR)
Is a directory - tags"

+1 on ruby 1.9.3

tpope closed this Aug 2, 2013

lzap deleted the lzap:fix-existing-dir branch Aug 2, 2013

raszi commented Aug 7, 2013

Please bump up the version and release it w/ this fix. Thanks.

Contributor

lzap commented Aug 8, 2013

Little bit of advertisment - I cloned Tim's work and included this fix, also modified it to use ripper-tags instead of ctags. Ripper tags gives better results (but obviously is a little bit slower):

https://github.com/lzap/gem-ripper-tags

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment