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

nutch conda package is broken #531

Open
ahmadia opened this issue May 26, 2015 · 12 comments
Open

nutch conda package is broken #531

ahmadia opened this issue May 26, 2015 · 12 comments

Comments

@ahmadia
Copy link
Contributor

ahmadia commented May 26, 2015

The nutch conda package invokes a shell script that does some very funny Java hardcode things. This is fragile and we should get rid of it. Who is the maintainer of the nutch conda package?

@brittainhard
Copy link
Contributor

Ben is. I created elasticnutch off of @quasiben's nutch package.

@ahmadia
Copy link
Contributor Author

ahmadia commented Jun 1, 2015

Another example. Error codes are not properly passed through these shell scripts.

@ahmadia
Copy link
Contributor Author

ahmadia commented Jun 1, 2015

@brittainhard - can you point me to the conda recipe for elasticnutch? We can raise an issue to move this over to a Salt install.

@brittainhard
Copy link
Contributor

@ahmadia conda recipie is in the tar files. You can download them here: https://binstar.org/brittainhard/elasticnutch/files

@ahmadia
Copy link
Contributor Author

ahmadia commented Jun 1, 2015

Thanks, found it. @brittainhard - did you keep a record of how this is transformed over from a nutch install? Your recipe doesn't point to a specific commit from the Nutch repository.

@ahmadia
Copy link
Contributor Author

ahmadia commented Jun 1, 2015

(This is not high priority by the way, we can chat about it when I'm in ATX this week)

@brittainhard
Copy link
Contributor

@ahmadia I didn't write that conda recipie, BTW, merely copied what was there. Not sure where you could figure out what version or commit that package pulls from. Couldn't have gotten anywhere fast if I had to. The best person to ask about that is Ben.

@ahmadia
Copy link
Contributor Author

ahmadia commented Jun 1, 2015

@brittainhard - there's a top-level recipe.json file that contains that sort of information. It's not mandatory to fill it out, so it can be hard to back out exactly where the files came from.

@chrismattmann
Copy link
Contributor

This needs to be coordinated with @lewismc - he is going to try and maintain a conda package associated with Apache Nutch and the Nutch releases. Relevant issue: NUTCH-2025 I would love to see you guys working on this over there.

@quasiben
Copy link
Contributor

quasiben commented Jun 3, 2015

@lewismc
Copy link
Member

lewismc commented Jun 3, 2015

Dynamite Ben that is fantastic.
Tomorrow I'm going to make sure I augment the release procedure to ensure
each subsequent Nuth release manager pushes the conda package as well!

On Tuesday, June 2, 2015, Benjamin Zaitlen notifications@github.com wrote:

Nutch recipe here:
https://github.com/ContinuumIO/nutchpy/tree/master/conda-recipes/nutch_recipe


Reply to this email directly or view it on GitHub
#531 (comment)
.

Lewis

@quasiben
Copy link
Contributor

quasiben commented Jun 3, 2015

@ahmadia, if i recall correctly, the reason we built this funky shell script to call nutch and crawl is because the original binaries expect a specific path layout which didn't work nicely with conda envs. The reason out (though arguably short-sighted) was to leave nutch alone and just simply pass all commands from nutch in envs/.../bin to nutch in env/.../lib/nutch/bin . The other benefit we were thinking about at the time was localization of nutch and components in one place.

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

No branches or pull requests

5 participants