Skip to content

Commit

Permalink
optional-dev-requirements: add syslog-ng-cfg-helper
Browse files Browse the repository at this point in the history
This tool helps in development, too.

You can find it in devshell at:
/source/dbld/build/venv/bin/syslog-ng-cfg-helper

---

I have added this as an optional dev requirement, because old platforms
(centos-7) does not support it.

Signed-off-by: Attila Szakacs <[email protected]>
  • Loading branch information
alltilla committed Nov 3, 2023
1 parent 56e91ab commit d7c636c
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ EXTRA_DIST = $(filter-out ${NODIST_BUILT_SOURCES},${BUILT_SOURCES}) VERSION NEW
syslog-ng-ctl/CMakeLists.txt \
requirements.txt \
dev-requirements.txt \
optional-dev-requirements.txt \
README.md \
.astylerc

Expand Down
1 change: 1 addition & 0 deletions optional-dev-requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
syslog-ng-cfg-helper
4 changes: 3 additions & 1 deletion scripts/build-python-venv.sh
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,15 @@ PYTHON=$1
PYTHON_VENV_DIR=$2
top_srcdir=$3
REQUIREMENTS_FILE=${top_srcdir}/dev-requirements.txt
OPTIONAL_REQUIREMENTS_FILE=${top_srcdir}/optional-dev-requirements.txt

set -e

echo "Building dev virtualenv for syslog-ng at ${PYTHON_VENV_DIR} from ${REQUIREMENTS_FILE}"
echo "Building dev virtualenv for syslog-ng at ${PYTHON_VENV_DIR} from ${REQUIREMENTS_FILE} and ${OPTIONAL_REQUIREMENTS_FILE}"

rm -rf ${PYTHON_VENV_DIR}
${PYTHON} -m venv ${PYTHON_VENV_DIR}
${PYTHON_VENV_DIR}/bin/python -m pip install --upgrade pip
${PYTHON_VENV_DIR}/bin/python -m pip install --upgrade setuptools
${PYTHON_VENV_DIR}/bin/python -m pip install -r $REQUIREMENTS_FILE
${PYTHON_VENV_DIR}/bin/python -m pip install -r $OPTIONAL_REQUIREMENTS_FILE || echo "Some optional pip packages were not installed. Continuing..."

0 comments on commit d7c636c

Please sign in to comment.