-
Notifications
You must be signed in to change notification settings - Fork 67
Make install error #7
Comments
Narcissus in doctorjs is now synced with the upstream repo. The files have been moved to the root of the project. They should probably be installed in /usr/local/lib/narcissus during install. It should not be too hard to patch the make file if you have a look. |
Erm, the narcissus directory in the doctorjs is empty. I suppose it was supposed to be a subrepo of narcissus. How would I go about fixing this issue? Sorry, it's not apparent to me. D: |
Yes, apologies. From within the docotorjs directory run the following command: git submodule update --init --recursive I'm not sure if a submodule update always moves to the latest version of that repo. If narcissus becomes incompatible, use git checkout in the narcissus directory to move narcissus back to the time or version in the doctorjs commit. Sorry for the trouble, but this updated narcissus is a good thing. It became very stale before and this arrangement removes barriers to stay up-to-date. |
diff --git a/Makefile b/Makefile
index bd43365..97bb089 100644
--- a/Makefile
+++ b/Makefile
@@ -7,7 +7,7 @@ LIB_SRC=$(addprefix lib/jsctags/,getopt.js log.js paperboy.js traits.js \
underscore.js)
LIB_CTAGS_SRC=$(addprefix lib/jsctags/ctags/,index.js interp.js nativefn.js \
reader.js writer.js)
-LIB_NARCISSUS_SRC=$(addprefix lib/jsctags/narcissus/,index.js jscfa.js \
+LIB_NARCISSUS_SRC=$(addprefix narcissus/lib/,index.js jscfa.js \
jsdefs.js jslex.js jsparse.js)
install: My diff on my change to the Make file for the root Narcissus. I get further in the install, but still missing 2 files:
Is this narcissus being incompatible? How would I do the git checkout properly on the narcissus directory? Sorry to be a nuisance. |
jscfa.js is not a part of narcissus, it just happened to be in the narcissus directory in the past. It now lives in the lib/cfa2 directory of doctorjs. |
Well, I've messed with the Make file a lot, installed it, got an error about something missing... I think for sanity's sake, I'm going to punt on this for now. Thanks. |
Please, use git checkout in the doctorjs directory to move your clone to several days ago. Then you can install and use the version before this narcissus version. Don't give up, you have options. I am sorry for the inconvenience, but please remember that when you are using git on a project that does not have release branches you are always using the bleeding edge. Use git checkout to move back to a specific revision. You can use the history feature on github to find a good point in time (maybe here: f63c1e0). Sorry for the inconvenience, but a little leverage from git and you can replace that sinking feeling of this-project-is-broken with just having it work. When someone gets time to patch the problem then you can move git back to the master branch. |
Perfect! Sorry I got a bit short, I understand bleeding edge - I was just getting frustrated because I didn't know how to fix the issues. Works great now. |
Thanks for your patience! More people will want to use doctorjs as a general ctags generator and I am sure the project will respond with some proper release management soon. |
Until we fix the Makefile, you can still use the latest doctorjs if you add the bin subdirectory to your PATH. (e.g., if you 've downloaded the source in your home directory, you would add /home/beardedprojamz/doctorjs/bin/ to your PATH.) Then you can get tags for a file foo.js by doing: jsctags.js foo.js |
It's a doctorjs problem, the Makefile is broken right now. The paths need changing. Hopefully we'll fix that soon. Until then, you can just call bin/jstags.js directly without installing. |
Right, I was trying that but did in fact have an environment problem. It's (sort of) sorted now. Thanks! |
Just starting with node and npm here, and found this ticket only after pulling a lot of hair, thinking I was messing it up :( please, use git's tags feature to point it to a sane point in the timeline, and drop a note about it in the readme. That aside, thanks for doctorjs! |
Trying to make install doctorjs - I get the following error:
Any ideas?
The text was updated successfully, but these errors were encountered: