-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Remove dependency on Python #6053
Conversation
This is not strictly needed once mysociety/commonlib#71 is merged |
210bcb2
to
fe951d1
Compare
This would be a step towards #2906 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with this in principle and removing an old dependency is a nice win.
At this point I'd rather this just be bin/config SETTING
so that we don't create extra deprecation issues if we do want a bin/alaveteli
in the future.
Instead of using yaml2sh from commonlib this commits adds a new Ruby script at bin/config which can retrieve individual configuration variables from config/general.yml. To get a configuration variable run: bin/config SITE_NAME This bin command doesn't initialise the Rails stack so is fast It also *doesn't* use lib/configuration.rb Ruby module, although it could if some configuration variables had defaults assigned. These are: 1. BUNDLE_PATH 2. SHARED_DIRECTORIES 3. SHARED_FILES 4. SHARED_FILES_PATH 5. STAGING_SITE As I understand it these haven't be given a default because they are only used from the installation shell scripts. Which seems reasonable to me.
fe951d1
to
abc3b60
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. I've only run bin/config
myself – not the Alaveteli scripts. Assuming these all work okay.
Have double checked to confirm (through building vagrant boxes for #6615) that the install scripts are still working. |
Instead of using yaml2sh from commonlib this commits adds a new Ruby
script at bin/config which can retrieve individual configuration
variables from config/general.yml.
To get a configuration variable run:
bin/config SITE_NAME
This bin command doesn't initialise the Rails stack so is fast
It also doesn't use lib/configuration.rb Ruby module, although it
could if some configuration variables had defaults assigned. These are:
As I understand it these haven't be given a default because they are
only used from the installation shell scripts. Which seems reasonable to
me.