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.

Sign up for GitHub

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

Jump to bottom

Update setup script to handle deploying config files #2

Closed
deoren opened this issue Aug 6, 2018 · 0 comments · Fixed by #3
Closed

Update setup script to handle deploying config files #2

deoren opened this issue Aug 6, 2018 · 0 comments · Fixed by #3
Assignees
@deoren

Comments

@deoren
Copy link
Contributor

deoren commented Aug 6, 2018

As part of some quick work on WhyAskWhy/mysql2sqlite#3 I plan to rename the config files currently active so that they're templates instead. This will make it easier to maintain that config file with new entries without risk of clobbering user settings should they wish to work from a Git clone/fork.
@deoren deoren self-assigned this Aug 6, 2018
deoren added a commit to deoren/mysql2sqlite-dev that referenced this issue Aug 8, 2018
@deoren
Look for config files, deploy templates otherwise
0258608 
Wrapper script accepts cmdline option in order to
mimic main script functionality and to pass-through
to main script.

cmdline option takes precedence over other locations.

refs WhyAskWhy#2
refs WhyAskWhy/mysql2sqlite#3
deoren added a commit to deoren/mysql2sqlite that referenced this issue Aug 8, 2018
@deoren
Template INI files, cmdline config path support
981a08c 
- repo-provided INI files have been renamed to *.tmpl in order
  to prevent them from being used as-is or risk clobbering
  local/custom config files if user opts to install
  directly from Git repo.

- Adjust log formatters to include line number for messages

- Replace explicit exit calls from within our library
  function calls.

- Adjust system-wide config path from /etc/DOMAIN/PROJECT
  to just /etc/PROJECT in order to better match the norm
  for /etc content layout.

- Improve exception handling for config file parsing issues

- Provide support for user-level config files
  - cmdline
  - $HOME/.config/PROJECT

- Fall back to logging warning/error messages to the console
  if settings object is not provided to constructor. Adjust
  filter for handler later once the settings object is properly
  constructed.

refs WhyAskWhy/mysql2sqlite-dev#2
refs WhyAskWhy#3
deoren added a commit to deoren/mysql2sqlite that referenced this issue Aug 8, 2018
@deoren
Template INI files, cmdline config path support
623a167 
- repo-provided INI files have been renamed to *.tmpl in order
  to prevent them from being used as-is or risk clobbering
  local/custom config files if user opts to install
  directly from Git repo.

- Adjust log formatters to include line number for messages

- Replace explicit exit calls from within our library
  function calls.

- Adjust system-wide config path from /etc/DOMAIN/PROJECT
  to just /etc/PROJECT in order to better match the norm
  for /etc content layout.

- Improve exception handling for config file parsing issues

- Provide support for user-level config files
  - cmdline
  - $HOME/.config/PROJECT

- Fall back to logging warning/error messages to the console
  if settings object is not provided to constructor. Adjust
  filter for handler later once the settings object is properly
  constructed.

refs WhyAskWhy/mysql2sqlite-dev#2
refs WhyAskWhy#3
@deoren deoren mentioned this issue Aug 8, 2018
Merged
@deoren deoren closed this as completed in #3 Aug 8, 2018
@deoren deoren mentioned this issue Aug 8, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deoren deoren

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Look for config files, deploy templates otherwise deoren/mysql2sqlite-dev
1 participant
@deoren