search

mjtrangoni / flexlm_exporter

Prometheus exporter for FLEXlm License Manager information
Apache License 2.0
83 stars 30 forks source link

Windows and Linux files lmstat_ should be the same. Fixes Issue #53 #62

Closed SuperSonicFlea closed 2 years ago

SuperSonicFlea commented 2 years ago

Hi @mjtrangoni ,

This is my first time doing a pull request. So if I do anything wrong please let me know.

I did a compare of lmstat_windows and lmstat_linux and found them to be mostly identical with added functionality in Linux file I did the same Comapre on lmstat_feature_exp_windows and _linux and also most the same with added functionality in Linux file

I copied Linux to windows for both lmstat adn lmstat_feature_exp and have compiled and am running with successful Expiry dates coming through on a windows Setup.

coveralls commented 2 years ago

Coverage Status

Coverage remained the same at 37.71% when pulling 69fa93819037893189fdfb81b649b601df7344b5 on SuperSonicFlea:Windows-Fix into 54d57fb163d71e1f4e3283f08386c300f940ee49 on mjtrangoni:master.

mjtrangoni commented 2 years ago

@SuperSonicFlea thank you! I will check how to reduce this duplication here.