Skip to content

Conversation

@jefferyto
Copy link
Member

Maintainer: @oskar456
Compile tested: armvirt-32, 19.07.7 sdk
Run tested: armvirt-32 (qemu), 19.07.7

Description:
This cherry picks commits from master to update addrwatch to 1.0.2 and apply some fixes. There are less changes compared to #15899 (no separate output module packages, no config file changes) to be more compatible with existing installs.

neheb and others added 4 commits July 15, 2021 16:03
Fixes compilation on musl.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit 8003eea)
Signed-off-by: Jeffery To <jeffery.to@gmail.com>
Switch to standard tarball to avoid autoreconfig.

Fix license information.

Add PKG_BUILD_PARALLEL for faster compilation.

Add PKG_INSTALL for consistency with other packages.

Removed upstreamed patches. Refresh remaining one.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit f1a7d50)
Signed-off-by: Jeffery To <jeffery.to@gmail.com>
This is 704e733 but applied for
addrwatch only.

Signed-off-by: Jeffery To <jeffery.to@gmail.com>
Makefile changes include:

* Include syslog output module

* Move main binary (back) to /usr/sbin, as it is system administration
  related and requires superuser privileges

New patches:

* 003-add-space-for-null-byte.patch - from
  fln/addrwatch@374cfd2

* 004-more-specific-library-linking.patch - from
  fln/addrwatch@27b57d9

* 005-use-c99-format-macro-constants.patch - from
  fln/addrwatch#28

Init script changes include:

* Fix command-line option names and format (from
  https://forum.openwrt.org/t/cant-start-addrwatch-service/60499/3)

* Always use the --quiet command-line option, as the procd instance is
  not configured to capture stdout/stderr

* Change the syslog config option to start the syslog output module

Signed-off-by: Jeffery To <jeffery.to@gmail.com>
@neheb neheb merged commit b031b21 into openwrt:openwrt-19.07 Jul 15, 2021
@jefferyto jefferyto deleted the addrwatch-fixes-openwrt-19.07 branch July 16, 2021 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants