Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Flag for automatic creation of Gem dependencies #375

Open
astrostl opened this Issue Mar 5, 2013 · 8 comments

Comments

Projects
None yet
4 participants

astrostl commented Mar 5, 2013

I'd love a flag which tells FPM to not only package up the requested Gem, but all of its listed dependencies. I would find this easier than the "Convert a gem and all of its dependencies" pattern from https://github.com/jordansissel/fpm/wiki/ConvertingGems , and don't see a downside. But maybe there is one?

Owner

jordansissel commented Mar 5, 2013

+1

I've thought about adding a flag that causes fpm to build the target package as well as fetching and packaging any dependencies.

Not sure what the flag should be called; I thought --recursive but that could easily be confused with directory search behaviors. Trying to think of another term to declare "Also download and package dependencies" but coming up short. Ideas?

astrostl commented Mar 6, 2013

--package-dependencies ?

Contributor

torrancew commented Mar 20, 2013

Another +1 for --package-dependencies here! This relates to the conversation @jordansissel and I had earlier in #logstash earlier today.

Owner

jordansissel commented Mar 20, 2013

Working on this now. Hopefully will have a prototype shortly :)

FWIW, I would have expected it to be the default behavior, and would ultimately prefer a flag to suppress it rather than enable it. I know these kind of "API" changes could be frowned upon for those that have FPM worked into various automations, though. (thought only occurred to be as a result of the parallel dependency talk in the CPAN ticket)

Owner

jordansissel commented Apr 19, 2013

(re my last comment of 'working on it now') - I started working on it, but I think I'll need to do some refactoring to make it work correctly.

Expand to "pip" (and thus all target) dependencies, not just Gems as per the subject :)

Contributor

djhaskin987 commented Apr 17, 2014

I NEED this.

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