feat: install.sh installs spf-tools and aggregateCIDR.pl#24
Merged
Conversation
contrib/install.sh now installs both runtime dependencies in addition to creating the postallow user and output directory: - spf-tools scripts cloned to a tmpdir and installed individually with 'install -m 755' into /usr/local/bin/ (no subdirectory, no wildcard chmod, no leftover .git/README) - aggregateCIDR.pl installed the same way (tmpdir pattern was already in place) - Both prompt 'Reinstall to update? [y/N]' when already present and running interactively; skip silently in non-interactive/CI context spftoolspath updated to /usr/local/bin everywhere (ci.yml, conf.in). CI jobs simplified: the now-redundant separate install steps removed from all four Linux jobs and the FreeBSD inline block. README.md: manual installation section expanded with split tables and explicit dep-install steps; ToC added with doctoc markers. Makefile: 'make toc' target added (npx --yes doctoc). .githooks/pre-commit: auto-updates ToC on commit if npx is available.
edmundlod
force-pushed
the
feat/install-deps
branch
from
7225f04 to
9980ef8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
contrib/install.shnow installs both runtime dependencies in addition to creating thepostallowuser and output directoryinstall -m 755into/usr/local/bin/— no subdirectory, no wildcardchmod, no leftover.git/READMEaggregateCIDR.pluses the same tmpdir patternReinstall to update? [y/N]when already present and running interactively; skip silently when non-interactive (CI)spftoolspathupdated to/usr/local/bineverywhere (ci.yml,conf/postallow.conf.in)make toctarget added;.githooks/pre-commitauto-updates ToC on commit ifnpxis available (activate withgit config core.hooksPath .githooks)Test plan
sudo contrib/install.shon a clean Linux system installs spf-tools scripts andaggregateCIDR.plinto/usr/local/bin/sudo contrib/install.shand confirm the reinstall prompt appearsmake tocupdates the ToC inREADME.md