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

Re-enable use of the CKAN_INI environment variable. #3522

Merged
merged 1 commit into from Apr 3, 2017

Conversation

davidread
Copy link
Contributor

It's really handy to use CKAN_INI environment variable for specifying where your development.ini is, for paster commands.

This was originally implemented in #2005 in Nov 2014 but got blown out at some point - I think the default='development.ini' got added back in by a duff merge.

Also: Improve error message when there is no config specified.

Test

(default)vagrant@vagrant-ubuntu-trusty-64:~$ paster --plugin=ckan user admin -c /etc/ckan/default/development.ini
<works>
(default)vagrant@vagrant-ubuntu-trusty-64:~$ paster --plugin=ckan user admin
ERROR: You need to specify the CKAN config (.ini) file path.
Use the --config parameter or set environment variable CKAN_INI or have development.ini
in the current directory.
(default)vagrant@vagrant-ubuntu-trusty-64:~$ export CKAN_INI=/etc/ckan/default/development.ini
(default)vagrant@vagrant-ubuntu-trusty-64:~$ paster --plugin=ckan user admin
<works>

@davidread
Copy link
Contributor Author

@KrzysztofMadejski I don't suppose you could test this, since you showed interest in using it?

@KrzysztofMadejski
Copy link
Contributor

I was interested in it, also for archiver's celery tasks. I'll try to test it tomorrow.

@smotornyuk
Copy link
Member

I tested it - everything just work fine

@davidread
Copy link
Contributor Author

Great! What happens next - do you merge it or reassign to someone who can?

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.

None yet

4 participants