-
Notifications
You must be signed in to change notification settings - Fork 609
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
System Update Settings - GitSync Options #263
System Update Settings - GitSync Options #263
Conversation
Add GitSync options selectors to post update GitSync. Minimal, Diff, Show Files, Show Command, Dry Run The GitSync options have utility for post update sync also. Dependency: Will merge cleanly but for the options to be functional, pfsense/FreeBSD-ports#263 is required.
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.
Please bump PORTVERSION
if [ "${show_command}" = "true" ]; then | ||
options=${options}' --show-command' | ||
mute='' | ||
ignore_result='' |
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.
Disabling ignore_result will make pfSense-upgrade to crash if gitsync fails, what is not a good idea IMO
Add GitSync options support. Minimal, Diff, Show Files, Show Command, Dry Run
71e5ea9
to
ea13447
Compare
Requested changes made. |
Patch release with miscellaneous bug/doc/build fixes. Excerpt from release tag: [#269] fix memory leak in V3fArrayFromBuffer [#268] Add <cstdint> for int64_t [#263] Initialize x in testRoots.cpp:solve() to suppress compiler warning [#262] Fix gcc compiler warning in testFun.cpp [#261] Test return value of extractSHRT to avoid uninitialized reference [#260] Fix example code so it compiles as is [#259] Cuda safety in several headers [#256] Fix markdown and typos in README.md [#255] Do not warn if half.h has already being included [#248] Update sphinx version ChangeLog: https://github.com/AcademySoftwareFoundation/Imath/releases/tag/v3.1.6 MFH: 2022Q4
ChangeLog: https://www.nlnetlabs.nl/news/2023/Jun/07/nsd-4.7.0-released/ 4.7.0 ================ FEATURES: - Merge #263: Add bash autocompletion script for nsd-control. - Fix #267: Allow unencrypted local operation of nsd-control. - Merge #269 from Fale: Add systemd service unit. - Fix #271: DNSTAP over TCP, with dnstap-ip: "127.0.0.1@3333". - dnstap over TLS, default enabled. Configured with the options dnstap-tls, dnstap-tls-server-name, dnstap-tls-cert-bundle, dnstap-tls-client-key-file and dnstap-tls-client-cert-file. BUG FIXES: - Fix #239: -Wincompatible-pointer-types warning in remote.c. - Fix configure for -Wstrict-prototypes. - Fix #262: Zone(s) not synchronizing properly via TLS. - Fix for #262: More error logging for SSL read failures for zone transfers. - Merge #265: Fix C99 compatibility issue. - Fix #266: Fix build with --without-ssl. - Fix for #267: neater variable definitions. - Fix #270: reserved identifier violation. - Fix to clean more memory on exit of dnstap collector. - Fix dnstap to not check socket path when using IP address. - Fix to compile without ssl with dnstap-tls code. - Dnstap tls code fixes. - Fix include brackets for ssl.h include statements, instead of quotes. - Fix static analyzer warning about nsd_event_method initialization. - Fix #273: Large TXT record breaks AXFR. - Fix ixfr create from adding too many record types. - Fix cirrus script for submit to coverity scan to libtoolize the configure script components config.guess and config.sub. - Fix readme status badge links. - make depend. - Fix for build to run flex and bison before compiling code that needs the headers. - Fix to remove unused whitespace from acx_nlnetlabs.m4 and config.h. - For #279: Note that autoreconf -fi creates the configure script and also the needed auxiliary files, for autoconf 2.69 and 2.71. - Fix unused variable warning in unit test, from clang compile. - Fix #240: Prefix messages originating from verifier. - Fix #275: Drop unnecessary root server checks. PR: 272096 Reported by: [email protected] (maintainer)
Add GitSync options support.
Minimal, Diff, Show Files, Show Command, Dry Run