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

main property value fixed #22

Merged
merged 1 commit into from
May 29, 2012
Merged

main property value fixed #22

merged 1 commit into from
May 29, 2012

Conversation

Rayzen
Copy link
Contributor

@Rayzen Rayzen commented May 28, 2012

No description provided.

amcjen pushed a commit that referenced this pull request May 29, 2012
main property value fixed
@amcjen amcjen merged commit fc48d5a into amcjen:master May 29, 2012
@amcjen
Copy link
Owner

amcjen commented May 29, 2012

Thanks!

@lhagan
Copy link
Contributor

lhagan commented May 30, 2012

I'm not sure this is the right change --- it breaks the module for me. The patch in issue #21 is sufficient on its own.

@Rayzen
Copy link
Contributor Author

Rayzen commented May 30, 2012

I pulled the project on my pc, and it said "can't find module clucene"... i've noticed many other project gave the same problem, maybe because of some update of node.js.... doing that way i was able tu run correctly clucene module... if you think it is wrong you can pull and try it and tell us the result..

@amcjen
Copy link
Owner

amcjen commented Jun 11, 2012

Let me do a pull and see if it works for me.

Also, sorry about the late reply on this.

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 this pull request may close these issues.

3 participants