search

palewire / django-calaccess-raw-data

A Django app to download, extract and load campaign finance and lobbying activity data from the California Secretary of State's CAL-ACCESS database
http://django-calaccess.californiacivicdata.org/
MIT License
64 stars 143 forks source link

Installing backend dependencies should be optional #1465

Closed adborden closed 8 years ago

adborden commented 8 years ago

We're using django-calaccess-raw-data with mysql, but pip install django_calaccess_raw_data dies when trying to install psycopg2 because pg_config isn't installed. It's quite cumbersome for us to have to install the mysql/postgres tools even though we're only using one of them.

palewire commented 8 years ago

Very good point and likely an oversight on our part. We didn't include psycopg2 in setup.py, but it might be a dependency of django-postgres-copy.

palewire commented 8 years ago

Yup. That's where it comes from. https://github.com/california-civic-data-coalition/django-postgres-copy/blob/master/setup.py#L47

palewire commented 8 years ago

We could remove the parent dependency here or the child one there. Not sure what the proper approach is, or if there's a third way.

palewire commented 8 years ago

@adborden, moments ago I released an updated version of our underlying dependency, django-postgres-copy, that no longer includes psycopg2 as a requirement. I believe that if you were to begin your installation again it would not longer try to install it and you would be able to continue without PostgreSQL.

palewire commented 8 years ago

I think we got this, so unless @adborden tells me otherwise I think this ticket can be closed.