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.

Sign up for GitHub

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

Jump to bottom

Enable Sourcegraph #44

Closed
sqs opened this issue Sep 16, 2014 · 4 comments
Closed

Enable Sourcegraph #44

sqs opened this issue Sep 16, 2014 · 4 comments

Comments

@sqs
Copy link

sqs commented Sep 16, 2014

I want to use Sourcegraph code search and code review with fsnotify. A project maintainer needs to enable it to set up a webhook so the code is up-to-date there.

Could you please enable fsnotify on @sourcegraph by going to https://sourcegraph.com/github.com/go-fsnotify/fsnotify and clicking on Settings? (It should only take 15 seconds.)

Thank you!
@nathany
Copy link
Contributor

nathany commented Sep 16, 2014

For sure. Thanks for the heads up Quinn.

@nathany nathany closed this as completed Sep 16, 2014
@nathany
Copy link
Contributor

nathany commented Sep 16, 2014

@sqs Does the fancy linking stuff only work for code that compiles on Linux?

I don't see any fancy stuff for this BSD file:
https://sourcegraph.com/github.com/go-fsnotify/fsnotify@master/.tree/kqueue.go

@nathany
Copy link
Contributor

nathany commented Sep 16, 2014

Or maybe I just need to wait a bit... https://sourcegraph.com/github.com/go-fsnotify/fsnotify/.builds/515146

@sqs sqs mentioned this issue Sep 16, 2014
Open
@sqs
Copy link
Author

sqs commented Sep 16, 2014

Currently srclib-go (which performs the Go analysis) compiles it as a Linux machine would, yes. I just posted an issue to add build tag support, though: sourcegraph/srclib-go#16. If this would be important to you, I can expedite it. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sqs @nathany