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

install all eggs included into manifest #18

Closed
wants to merge 1 commit into from
Closed

install all eggs included into manifest #18

wants to merge 1 commit into from

Conversation

235
Copy link

@235 235 commented Feb 25, 2013

Otherwise libraries required for a project are just ignored

Otherwise libraries required for a project are just ignored
@barrywhart
Copy link
Contributor

Petrel has a similar feature already. If your manifest.text includes a file "setup.sh", Petrel will automatically execute it before launching the spout or bolt. Typically this script is used for installing eggs. By letting you write the script yourself, Petrel gives you more control, for example whether to use pip or easy_install, what options they are called with, etc.

Here's an example setup.sh file from one of my projects:

set -e

# $1 will be non-zero if creating a new virtualenv.
if [ $1 -ne 0 ]; then
    for f in Shapely==1.2.15 pyproj==1.9.0 pycassa==1.7.0 \
             configobj==4.7.2 greenlet==0.4.0 gevent==1.0b3
    do
        echo "Installing $f"
        pip install $f
    done
fi

You could easily modify this code to look for "*.egg" and install them all (although you might have to be careful with installation order dependencies).

I will add this information to the README.

@barrywhart barrywhart closed this Feb 25, 2013
@barrywhart barrywhart reopened this Feb 25, 2013
@barrywhart
Copy link
Contributor

Sorry, I shouldn't have closed this without hearing from you first. Does the setup.sh feature meet your needs?

@235
Copy link
Author

235 commented Feb 25, 2013

Yes, this will do, thanks!

@235 235 closed this Feb 25, 2013
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.

2 participants