Date: Fri, 16 May 2014 18:03:24 +0200 From: "J.R. Oldroyd" <fbsd@opal.com> To: freebsd-xfce@freebsd.org Subject: sysutils/xfce4-wavelan-plugin Message-ID: <20140516180324.2fcf73f0@shibato>
next in thread | raw e-mail | index | archive | help
--Sig_/Wwr+Aerncsw+7BpBAotEHPK Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable sysutils/xfce4-wavelan-plugin no longer works. Looking at the code, there are various problems. It looks for a sysctl oid that doesn't exist, it returns the interface rssi value instead of properly calculating the signal quality, it uses that incorrect rssi value also as the signal percentage, but worse... all the FreeBSD code is compiled out anyway due to tests for FreeBSD_version being done before sys/param.h is included!! I've posted two patches (URLs below) that fix these problems and make it work again. I also fix the percentage values at which the color changes, I use 25%, 50% and 75% instead of the strange values 40%, 55% and 70% that are there now. I've named these files extra-patch-foo for now, but they should be renamed to patch-foo and merged in with the existing patches already there. The code in wi_bsd.c is a mess! There is no need to be using the kernel values __FreeBSD_kernel__ or __FreeBSD_kernel_version here or even __FreeBSD_kernel_kernel__ (yes, really)! I didn't clean that stuff up with this patch, but if desired, I can do so. Maybe there's a reason it's like this? -jr http://opal.com/jr/freebsd/ports/sysutils/xfce4-wavelan-plugin/extra-patch-= panel-plugin__wavelan.c http://opal.com/jr/freebsd/ports/sysutils/xfce4-wavelan-plugin/extra-patch-= panel-plugin__wi_bsd.c --Sig_/Wwr+Aerncsw+7BpBAotEHPK Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (FreeBSD) iEYEARECAAYFAlN2NtAACgkQls33urr0k4nNtQCfbg/fQ2/+QO6jDcmEo22nK8kq yIIAnjeBNaFT5EpvZJIDLizdDFLtNGBi =6vB8 -----END PGP SIGNATURE----- --Sig_/Wwr+Aerncsw+7BpBAotEHPK--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140516180324.2fcf73f0>