LCDProc package version 0.11.2

jimp

LCDProc package version 0.11 is building now and will be available shortly.

It is comprised primarily of PHP fixes and code cleanup but due to those, significant portions of the code changed, so be on the lookout for potential problems (e.g. screens not formatted properly/as expected).

Update syntax for PHP 8.x
Clean up code syntax for style/formatting and logic as needed
Fix a few errors/bugs spotted along the way
- IPsec status was broken or hadn't worked in years
- Interface stats would stop early if there was a removed/unassigned OPT interface in the middle of the list
- Likely other fixes for various screens as they should all now be using more up-to-date code for polling data
Fix line terminators in shortcuts file

Version 0.11.1 with more fixes:

Remove redundant functions already in base with the same output
Leverage base functions where possible otherwise
Start service on sync if it's stopped (comments suggested it should, but it didn't)
Fix formatting on Matrix Orbital help text
Cleanup some more unused code

Version 0.11.2 with a couple new additions:

Added a background color selector for Adafruit LCD backpack displays with RGB support and support the extended commands.
Added CPU Temperature screen
Added Package Info screen, primarily to show some indication that there are updated packages available. Displays the total number of packages installed, how many need updating, and also displays a count of broken packages if there are any.

jimp

I tested it using an Adafruit USB + Serial LCD Backpack with a 20x4 negative LCD and it works as expected there when configured properly (Driver=Matrix Orbital, Type=LKD, Adjustable backlight=unchecked, Brightness 100%, Contrast 100%). You do have to solder it together but it's just a row of pins so it's pretty simple. It's about ~$40+shipping so much cheaper than some of the alternatives I've seen around. If you can make do with a 16x2 LCD it's $10 cheaper, but for the difference it's worth having the larger screen.

The very first time I powered on the device I had to send it a command to fix the screen size to be 20x4 (printf "%b" '\0376\0321\0024\0004' > /dev/cuaU0), not sure if that's worth handling in the package or not though since it is not something you can do live. It gets stored in EEPROM after setting and requires power cycling the device to pick up the change.

fireodo

Hi!

First of all: Thank you for your very good work!

Would you also consider to implement the CPU-Temperature screen? (see reference: https://github.com/pfsense/FreeBSD-ports/pull/1125)

Thanks and i wish a fine Weekend,
fireodo

jimp

@fireodo said in LCDProc package version 0.11.1:

First of all: Thank you for your very good work!

Would you also consider to implement the CPU-Temperature screen? (see reference: https://github.com/pfsense/FreeBSD-ports/pull/1125)

I can work that in. A lot of the code in that PR is irrelevant/broken now compared to what is in the current package, but I can pick out just the relevant CPU temp parts and bring that in. It seems to work here in some local testing.

fireodo

@jimp said in LCDProc package version 0.11.1:

It seems to work here in some local testing.

Works here too - many thanks!