docker-compose: Update to version 2.30.2

Release notes:
https://github.com/docker/compose/releases/tag/v2.30.2

Signed-off-by: Javier Marcet <javier@marcet.info>

owut: update to 2024.10.27

Bug fix:
  efahl/owut@8c7e42f owut: force top-level when adding a package

Enhancement:
  efahl/owut@e26df83 argparse: add an exclusive store mechanism

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>

alist: Update to 3.39.1

Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>

netatalk: Remove apfstats feature to solve dependency issues

afpstats requires several libraries that would increase significantly final install size.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>

nginx-util: drop dummy-package extra dependency

In dropping the dummy-package, also the extra dependency for
nginx-ssl-util needed to be dropped.

Fixes: #25250
Fixes: b75050d59e40 ("nginx-util: drop nginx-util dummy packages")
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: improve libexslt optional patch

Improve libexslt optional patch to silent confusing output.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: add patch fixing libxslt library detection

Nginx hardcode the libxml2 include path to /usr/include/libxml2. This
works in a local build as pretty much everyone have the libxml2 library
installed but doesn't on buildbot container as the library doesn't
exist.

This effectively makes the host library leak intro library detection but
doesn't actually link to it as linking is still done with the correct
library in staging dir.

To fix this add a patch to define custom libxml2 include directory
instead of hardcoding it to host library.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx-util: drop nginx-util dummy packages

Enough transition period has passed for nginx-util dummy package. Finally
drop it for good and drop the extra dependency.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: drop extra dependency <2 for nginx-util

Drop extra dependency <2 for nginx-util. The reason is not clear and
cause problems with APK package creation.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: drop nginx dummy packages

Enough transition period has passed for nginx dummy package. Finally
drop it for good and add Provides for nginx default.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: mute warning on init.d script enable

Mute warning on procd init.d script enable as nginx_util is tried to be
called from /usr/bin host system. Limit it to be called only if
nginx-util is present.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

owut: fix order of uci-defaults

Move uci-defaults file to run level 51, so it is executed immediately
after the attendedsysupgrade-common package's uci-defaults script.

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>

kadnode: Update to version 2.4.0

* add support for mbedtls 3.0.0
* fix newline in init script

Signed-off-by: Moritz Warning <moritzwarning@web.de>

atop: add missing nls.mk include

If nls.mk is not included and BUILD_NLS is set compilation will fail with:
aarch64-openwrt-linux-musl-gcc atop.o version.o various.o  deviate.o   procdbase.o acctproc.o photoproc.o photosyst.o cgroups.o rawlog.o ifprop.o parseable.o showgeneric.o drawbar.o showlinux.o  showsys.o showprocs.o atopsar.o  netatopif.o netatopbpfif.o gpucom.o  json.o utsnames.o -o atop -lncursesw -lz -lm -lrt -Lstaging_dir/toolchain-aarch64_cortex-a53_gcc-13.3.0_musl/usr/lib -Lstaging_dir/toolchain-aarch64_cortex-a53_gcc-13.3.0_musl/lib -fuse-ld=bfd -znow -zrelro  -Lstaging_dir/target-aarch64_cortex-a53_musl/usr/lib -lglib-2.0 -lintl
staging_dir/toolchain-aarch64_cortex-a53_gcc-13.3.0_musl/lib/gcc/aarch64-openwrt-linux-musl/13.3.0/../../../../aarch64-openwrt-linux-musl/bin/ld.bfd: cannot find -lintl: No such file or directory
collect2: error: ld returned 1 exit status

So make sure to include nls.mk.

Fixes: #25231
Signed-off-by: Robert Marko <robimarko@gmail.com>

netatalk: Enable AppleTalk feature

Starting v4.0.0, netatalk brings back AppleTalk features lost in v3.
Updated description.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>

netatalk: Clean up Makefile

Updated title.
Updated description.
Reordered to match most common layout

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>

netatalk: Update to 4.0.4

Maintainer: Antonio Pastor / @APCCV
Compile tested: ipq806x (23.05.3, snapshot)
Run tested: ipq806x - C2600: start server, connect from MacOS, read/write files to home share, create/update TimeMachine backups

Description:
No changes to package other than using latest available upstream code base. Starting Netatalk 4.x build uses meson instead of autotools.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>

tgt: enable support for thin_provisioning

Signed-off-by: Maxim Storchak <m.storchak@gmail.com>

libdeflate: update to 1.22

Release note:
https://github.com/ebiggers/libdeflate/blob/master/NEWS.md#version-122

Signed-off-by: Gábor Deé <dee.gabor@gmail.com>

geth: remove

I don't think anyone uses this package, which currently does not build.

In addition, the cryptocurrency hype died off a long time ago.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

luajit2: update to v2.1-20240815

Signed-off-by: Javier Marcet <javier@marcet.info>

pservice: modify date for apk compatibility

Signed-off-by: Rosen Penev <rosenp@gmail.com>

v2ray-geodata: Update to latest version

Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>

v2raya: Update to 2.2.6.2

Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>

cloudflared: Update to 2024.10.1

Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>

v2ray-core: Update to 5.20.1

Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>

adblock-fast: update to 1.1.2-10

* bump compat version to accommodate new strings
* improve the output() function (thanks @bigsmile74)
* implement support for user-configurable per-instance dnsmasq confdirs
  for dnsmasq.conf|dnsmasq.ipset|dnsmasq.nftset options

Signed-off-by: Stan Grishin <stangri@melmac.ca>

apfree-wifidog: Update to 7.10.2082

Update Makefile to include dependencies: conntrack and libmosquitto.

Conntrack: Added to resolve a bug in Apfree-Wifidog that caused long-lived user connections to persist after logout.
Libmosquitto: Integrated as a preparatory measure for future support of the MQTT protocol; implementation is still in progress.
For detailed information, please refer to the release notes:
https://github.com/liudf0716/apfree-wifidog/releases/tag/7.10.2082.

Additionally, a new /etc/wifidogx directory has been created to store the local authentication portal page for users uploading their own portal pages.

Signed-off-by: Dengfeng Liu <liudf0716@gmail.com>

apfree-wifidog: improve structure and readability

- Refactored wifidogx.init to improve code structure and readability.
- Added local authentication support, allowing authentication without a server.

Signed-off-by: Dengfeng Liu <liudf0716@gmail.com>

pbr: update to 1.1.7-47

* bump compat version to accommodate new strings
* update dnsmasq-related code to better support separate confdirs
  for separate instances
* remove procd_lan_interface as it didn't reflect that it's a list of devices
* introduce procd_lan_device list
* improve the output() function (thanks @bigsmile74)
* remove duplicate uci_get_device
* improve ipv6 detection and interface setup
* improve dhcp force detection for interfaces name differently from lan
* fix array/element parameters for some json operations
* remove unneeded null redirects for `try` calls
* remove (iptables-only) capitalized chain names form validation
* working pbr-netifd flavor

Signed-off-by: Stan Grishin <stangri@melmac.ca>

openconnect: introduced script parameter

This allows specifying a custom vpnc-compatible config script.

Signed-off-by: Yury Potapov <me@immensus.net>

libseccomp: update to 2.5.5

Signed-off-by: Rosen Penev <rosenp@gmail.com>

bcm27xx-eeprom: update to latest version

This update contains multiple improvements for BCM2711 and BCM2712.
Switch from default (critical) to stable firmwares.

Full changelog: https://github.com/raspberrypi/rpi-eeprom/compare/v2024.06.05-2712...v2024.09.23-2712

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>

attendedsysupgrade-common: give the uci-defaults script a run level

Move the init script to '50-attendedsysupgrade', so it is run in the
middle of the init sequence, rather than after all the explicitly
ordered ones.  This allows later scripts, specifically the 99-level
ones, to modify the contents of the attendedsysupgrade configuration.

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>

apparmor: update to 3.0.13

Use PKG_SOURCE_PROTO for smaller tarballs.

Removed upstreamed patches

Backport basename patch for musl compatibility.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

nsutils: update to latest master

Remove fancy versioning. Adds compatibility with apk.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

luasoap: update to latest release

Adds compatibility with apk.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

docker-compose: Update to version 2.30.1

Release notes:
https://github.com/docker/compose/releases/tag/v2.30.1

Signed-off-by: Javier Marcet <javier@marcet.info>

docker-compose: Update to version 2.30.0

Release notes:
https://github.com/docker/compose/releases/tag/v2.30.0

Signed-off-by: Javier Marcet <javier@marcet.info>

pptp: fix compilation with GCC14

Needed forward declaration for missing function.

Removed some no longer needed musl fixes.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

ddns-scripts: porkbun bugfix for subdomain on A/AAAA record

If the "name" parameter is not provided, then updates to a subdomain
A/AAAA record result in the subdomain being removed from the record.
This change always provides the "name" parameter to correct that behavior.

Signed-off-by: Dillon Dixon <github@dillon.io>

knxd: bump to new version 0.14.66

Signed-off-by: Othmar Truniger <github@truniger.ch>

libuwifi: update to latest master

Fix version to be apk compatible.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

mwan3: bump PKG_RELEASE

Forgot to bump the 'PGK_RELEASE' in the last change in 'mwan3'.

Fixes: 619629ce8599 ("mwan3: close flock fd when starting mwan3.user scripts")
Signed-off-by: Florian Eckert <fe@dev.tdt.de>

keepalived: bump to 2.3.1

Release notes:
https://www.keepalived.org/release-notes/Release-2.3.1.html

Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>

keepalived: bump to 2.3.0

Release notes:
https://www.keepalived.org/release-notes/Release-2.3.0.html

Drop upstreamed patch 0001-vrrp-remove-logging-on-status-output.patch.

Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>

luajit2: switch to GIT source and adapt to APK friendly version

Switch to GIT source and adapt to APK friendly version.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

mwan3: close flock fd when starting mwan3.user scripts

Fixes deadlock with multiple init script calls

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

pbr: update to 1.1.7-29

Changes from @stangri
* remove unneeded `\n` escapes
* cosmetic improvements to make code more consistent
* remove duplicate uci_get_device()
* add more output on start/stop
* remove wan up detection on boot/start
* address Tor policies errors
* prevent interface_routing() failures for downed interfaces

Changes from @bigsmile74:
* improve is_integer()
* improve is_domain()
* improve filter_options()
* imrove is_ipv4() so that is_ipv4_netmask() can be retired
* improve is_phys_dev so that is_phys_dev_quick() can be retired
* add the dhcp.lan.force=0 warning

Signed-off-by: Stan Grishin <stangri@melmac.ca>

adblock-fast: bugfix: fix allowed domains for dnsmasq.servers

Signed-off-by: Stan Grishin <stangri@melmac.ca>

libmicrohttpd: Fix too long title

Signed-off-by: Evgeny Grin (Karlson2k) <k2k@narod.ru>

eza: update to 0.20.5

Release notes:
0.20.1: https://github.com/eza-community/eza/releases/tag/v0.20.1
0.20.2: https://github.com/eza-community/eza/releases/tag/v0.20.2
0.20.3: https://github.com/eza-community/eza/releases/tag/v0.20.3
0.20.4: https://github.com/eza-community/eza/releases/tag/v0.20.4
0.20.5: https://github.com/eza-community/eza/releases/tag/v0.20.5

Signed-off-by: Jonas Jelonek <jelonek.jonas@gmail.com>

croc: update to 10.0.13

Changelog:
https://github.com/schollz/croc/releases/tag/v10.0.13

Signed-off-by: Jonas Jelonek <jelonek.jonas@gmail.com>

tailscale: update to 1.76.3

Changelog: https://tailscale.com/changelog#2024-10-21

Signed-off-by: Sandro Jäckel <sandro.jaeckel@gmail.com>

schroot: add boost-1.85 compatibility patch

Signed-off-by: Javier Marcet <javier@marcet.info>

clamav: bump PKG_RELEASE

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: freshclam: Add documented options

Enable syslogging

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: freshclam: Remove UpdateLogFile option

( to instead use syslog options )

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: milter: Add Debug option

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: Add syslogging options

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: Align with milter options and remove LogFile* options

( to instead use syslog options )

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: milter: Remove deprecated AllowSupplementaryGroups option

clamav-milter 1.3.0 and onward log:

WARNING: Ignoring deprecated option AllowSupplementaryGroups at /tmp/clamav/clamav-milter.conf:5

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

clamav: Fix UCI error

Hyphens in named sections are a no-no

uci: Parse error (invalid character in name field) at line 1, byte 37

Changed '-' to '_' to fix error to allow UCI parsing.

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

libmicrohttpd: Use external hashing functions

When compiled with GnuTLS external functions can be used to calculate
hashes. Enable them to reduce the library binary size (by disabling
internal code for hash calculations).
Note: libmicrohttpd 1.0.0 or later required

Signed-off-by: Evgeny Grin (Karlson2k) <k2k@narod.ru>

libmicrohttpd: Fixed 'TITLE'

Note dropped period at the end of the string to be nicely combined with
postfix of SSL version.

Signed-off-by: Evgeny Grin (Karlson2k) <k2k@narod.ru>

libmicrohttpd: Drop unneeded libgcrypt dependency

libgcrypt was used by early GnuTLS 2.x versions. For GnuTLS versions
without ability to automatically initialise libgcrypt, MHD was able to
initialise libgcrypt. However, it is not required for a long time.

Signed-off-by: Evgeny Grin (Karlson2k) <k2k@narod.ru>

libmicrohttpd: Enable 'poll()' usage

If 'poll()' function is disabled, then in thread-per-connection mode
'select()' is used. This is much less efficient.
'epoll' is not used in thread-per-connection mode as it would require
additional epoll FD per every connection.

Signed-off-by: Evgeny Grin (Karlson2k) <k2k@narod.ru>

libmicrohttpd update to 1.0.1

Signed-off-by: Jen Wolf <jenwolf@protonmail.com>

micropython: bump to 1.23.0 release

Bump micropython to 1.23.0. This is done to fix support for MbedTLS 3.0

Lots changed between 1.21 and 1.23 for the library side.

Mainly the barkeley-db library was moved to micropython own fork and the
cdef patch required lots of changes. (we have to manually include a copy
of cdef.h as it was dropped by them)
Also a new patch is required to mute some compilarion error that are
false-positive. (variable not initialized but that are ALWAYS
initialized)

Also refresh the extmod patch with new files included.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

adblock: update to 4.2.2-8

* get rid of remaining opkg calls and static adblock versioning
* fixed bind autodetection

Signed-off-by: Dirk Brenken <dev@brenken.org>

travelmate: update to 2.1.3-2

* fixed gathering/printing of system information in travelmate status
* make use of a central command selector function

Signed-off-by: Dirk Brenken <dev@brenken.org>

adblock: update to 4.2.2-7

* fixed gathering/printing of system information in adblock status
* added missing hagezi category (samsung tracker)

Signed-off-by: Dirk Brenken <dev@brenken.org>

banIP: update to 1.0.0-9

* fixed gathering/printing of system information in banIP status
* removed broken iblocklist.com feeds
* updated readme

Signed-off-by: Dirk Brenken <dev@brenken.org>

mptcpd: add package

Package Multipath TCP daemon (mptcpd) and wrapper (mptcpize).

Signed-off-by: Daniel Golle <daniel@makrotopia.org>

ell: add package

Add package for Embedded Linux Library (ell) from kernel.org.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>

yggdrasil: bump to 0.5.9

Signed-off-by: Ivan Chashkin <mail@cupivan.ru>

gperftools:  bump to version 2.16

Changelog: https://github.com/gperftools/gperftools/releases/tag/gperftools-2.16

Build-tested: x86/64/AMD Cezanne
Run-tested: x86/64/AMD Cezanne

Signed-off-by: John Audia <therealgraysky@proton.me>

ruby: fix yjit for target and host build

Since ruby 3.3.0, yjit was converted into rust code. During build, ruby will try try to use the whatever rustc is available in $PATH, including the one provided by the OS. Variations in that rustc can generate something between a perfect funcional build with yjit enabled and a broken build like this (from github actions):

  2024-10-16T05:06:05.9863422Z linking static-library libruby-static.a
  2024-10-16T05:06:06.0625182Z LLVM ERROR: Invalid encoding
  2024-10-16T05:06:06.1531894Z make[4]: *** [Makefile:318: libruby-static.a] Aborted (core dumped)

Ruby 3.3.5 still only supports yjit for x86_64 and aarch64. Even for those targets, ruby build does not support cross-compiling.

This commit adds rust as a dependency for those supported archs, even when cross-compiling, to let it work when host and target arch matches.

We don't need yjit for host build and we can disable it.

Closes #25151, #25052

Signed-off-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>

ruby: update to 3.3.5

This release is a routine update that includes minor bug fixes.

Link: https://www.ruby-lang.org/en/news/2024/09/03/3-3-5-released/
Signed-off-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>

sstp-client: bump to 1.0.20 release

Bump sstp-client to 1.0.20 release. This fix compilation error with new
PPPd version 2.5.0 as some API changed.

Patch automatically refreshed.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

libuhttpd: update to 3.14.2

Signed-off-by: Jianhui Zhao <zhaojh329@gmail.com>

owut: update to 2024.10.25

Enhancements:
    efahl/owut@8c8907c owut: more ux improvements
    efahl/owut@00de355 owut: improve aesthetics of build monitor

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>

netbird: update to 0.30.3

changelog: https://github.com/netbirdio/netbird/releases/tag/v0.30.3

Signed-off-by: Wesley Gimenes <wehagy@proton.me>

adblock-fast: explicitly allowing domains for dnsmasq.servers

New features:
* explicitly add allowed domains on download/allow to the dnsmasq.servers
  config file

Cleanup:
* remove unneeded EXTRA_COMMANDS
* remove unneeded EXTRA_HELP
* remove unnecessary `\n` escapes
* remove unnecessary line breaks from output on download

Optimization:
* simplify is_integer()
* replace long if with case
* more verbose output on download

Signed-off-by: Stan Grishin <stangri@melmac.ca>

keepalived: fix dnsmasq reload

When "set_reload_if_sync" is set, the service is reloaded when the
configuration changes. For dnsmasq this means that the service, if
stopped, is started, and we don't want this in the backup node.

Signed-off-by: Francesco Benini <francy.benini@gmail.com>

keepalived: fix ambigiuos functions in hotplug.sh

When "set_reload_if_sync" is not set in the hotplug script, the service
is not expected to reload. That is not true because even if not set, the
value is set to the default 1 (reload active) or equals the parameter
set when "keepalived_hotplug" is called.
The default behavior should be:
    - Reload if set_reload_if_sync is called
    - NOT reload if set_reload_if_sync is NOT called
A similar fix is ported to "set_update_target".

Signed-off-by: Francesco Benini <francy.benini@gmail.com>

keepalived: add option to override service running check

Some init.d scripts like firewall and sqm do not return the actual state
of the service if called with "running" parameter. This result in the
init script called with "start" parameter and the service may not load
the new configuration. Firewall init script is one of this
An option is added in order to skip the "running" check for the service.

Signed-off-by: Francesco Benini <francy.benini@gmail.com>

netbird: update to 0.30.2

changelog: https://github.com/netbirdio/netbird/releases/tag/v0.30.2

Signed-off-by: Wesley Gimenes <wehagy@proton.me>

owut: update to 2024.10.15

Enhancements:
    efahl/owut@cdfbc24 owut: rework build status monitor to use HEAD requests
    efahl/owut@6704884 owut: rework download statistics

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>

nginx: Add support for NJS module

Add support for NJS module. Various patch are required to make this
module correctly compile with the required library mainly related on
detecting config flags.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: add patch to make libexslt optional

Add patch to make libexslt library optional as nginx autoenable it if
detected.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

nginx: disable additional entry in feature test

Disable additional entry in feature test probably added in new version
of Nginx and never notice. Also declare "no" instead of empty value for
cc test.

While at it also refresh the patch.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

libxslt: install header for libxlt by default

Install header for libxlt by default to handle case when dependent
package depends on the library but doesn't select the configuration.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

rp-pppoe: update to 4.0

This fixes compatibility with ppp-2.5.0 and newer.

We also need to change the PKG_SOURCE_URL since the previous URL is no
longer working and automated downloads from the project homepage are
not desired due to bandwidth restrictions. Have a look at the project
homepage [1] for details.

Signed-off-by: Martin Schiller <ms@dev.tdt.de>
[1] https://dianne.skoll.ca/projects/rp-pppoe/

modemmanger: bump PKG_RELEASE number

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

modemmanager: add pin check attempts

This new check in the proto modemanager prevents the SIM card from being
blocked and therefore PUK is not required. If the PIN is entered incorrectly
in the 'uci' configuration, it makes no sense to try this several times
until the PUK is required. Should it nevertheless happen that the PUK
is required, then this will logged.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

modemmanager: improvement of readability by reversing the query

No functional change. The change makes reading the source clearer.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

modemmanager: add missing interface argument to function modemmanager_check_state

The variable interface is used in the sub-function 'modemmanager_check_state'.
However, this is not an argument of the function and so the global value
is used. Addding the variable as an function call argument fixes this.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

modemmanager: move modemmanager_check_state failed and locked into sub functions

To make the source clearer, the program parts for the 'locked' and 'failed'
cases are outsourced to sub-functions.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>

modemmanager: fix value call with modemmanager_get_field function

The correct values are prefixed with 'modem.generic'. This is missing
for the value 'state' and 'state-failed-reason.

While we're at it, let's move the readout of 'state-failed-reason' to the
failed case, because that's the only place it's needed.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>