Changeset - 6df08d78f8e7
[Not reviewed]
stable
0 1 0
Thomas De Schampheleire - 7 years ago 2019-04-08 22:06:12
thomas.de_schampheleire@nokia.com
cli: fill in git_hook_interpreter at 'config-create' time to really fix potentially invalid interpreter in git hooks (Issue #333)

When generating a configuration file using 'kallithea-cli config-create', it is
probably using the right Python interpreter, so fill in the current Python
executable as 'git_hook_interpreter' in the generated file.

There should thus rarely be any need to configure this manually, and issue #333
will *really* be fixed.

As this causes an absolute path to be encoded inside the ini file, moving
the virtualenv will require updating this path.

For development.ini we do not want to hardcode any path and are happy to leave
it using the old heuristics at runtime.

Suggested-by: Mads Kiilerich <mads@kiilerich.com>
1 file changed with 4 insertions and 1 deletions:
0 comments (0 inline, 0 general)
kallithea/bin/kallithea_cli_config.py
Show inline comments
 
@@ -16,6 +16,7 @@ import click
 
import kallithea.bin.kallithea_cli_base as cli_base
 

	
 
import os
 
import sys
 
import uuid
 
from collections import defaultdict
 

	
 
@@ -59,7 +60,9 @@ def config_create(config_file, key_value
 
    should go.
 
    """
 

	
 
    mako_variable_values = {}
 
    mako_variable_values = {
 
        'git_hook_interpreter': sys.executable,
 
    }
 
    ini_settings = defaultdict(dict)
 

	
 
    section_name = None
0 comments (0 inline, 0 general)