search

samvera-deprecated / hydra

Project Hydra Stack Dependencies
Other
83 stars 30 forks source link

Added gem 'hydra-head' to Gemfile #80

Closed yvoloshin closed 8 years ago

yvoloshin commented 8 years ago

I found that gem 'hydra-head' had to be in the Gemfile in order for hydra-jetty to start successfully. Without hydra-head in the Gemfile, I would get "Unable to move tmp/jetty_generator/hydra-jetty-master into jetty/ Operation not permitted - (tmp/jetty_generator/hydra-jetty-master, jetty)" when I tried to run the hydra-jetty generator. With hydra-head in the Gemfile, this problem was solved. I tried this after seeing the instructions for hydra-head installation at https://github.com/projecthydra/hydra-head.

mjgiarlo commented 8 years ago

Hi, @yvoloshin. The purpose of the Hydra gem is to wrap a bunch of other dependencies, so while this fixes the behavior for you, I think we should pursue fixing this elsewhere. Thoughts on this, @jcoyne @jeremyf @dchandekstark?

That said, thanks for the pull request! Always glad to see a pull request from someone who's not submitted one to the Hydra community before. In order for us to merge code from you in the future, we'll need to have signed Contributor License Agreements (CLAs) on file for you as an individual and your institution. You can read more about our CLA processes here:

https://wiki.duraspace.org/display/hydra/Hydra+Project+Intellectual+Property+Licensing+and+Ownership

The process itself is fairly simple -- you just need to print out the Hydra Project individual CLA (iCLA) and sign it and then send it to legal@projecthydra.org. Same for the corporate CLA (cCLA) but that document must be signed by someone who has signatory authority for your institution. Let me know if you have any questions about this process, and I'd be happy to help.

mjgiarlo commented 8 years ago

DO NOT MERGE. CLA required.

dchandekstark commented 8 years ago

@mjgiarlo Agree that this should be fixed elsewhere, if possible.

yvoloshin commented 8 years ago

That makes sense, @mjgiarlo. I'll make sure to sign the CLA next time I contribute. Did you also see the issue (https://github.com/projecthydra/hydra/issues/79) I sent in?

mjgiarlo commented 8 years ago

Hi, @yvoloshin -- no, I missed #79. I recommend sending that to the hydra-tech mailing list where more eyes are sure to see it. Thank you for writing it up!

yvoloshin commented 8 years ago

Thank you for the suggestion @mjgiarlo. I posted it on the mailing list.