2.7.1 checklist - Githubissues

networkupstools / nut

The Network UPS Tools repository. UPS management protocol Informational RFC 9271 published by IETF at https://www.rfc-editor.org/info/rfc9271 Please star NUT on GitHub, this helps with sponsorships!

https://networkupstools.org/

Other

2.12k stars 355 forks source link

2.7.1 checklist #37

Closed aquette closed 11 years ago

aquette commented 11 years ago

Checklist for the 2.7.1 interim release:

[x] @clepple: update dowload info for git
[x] update client info with walNUT
[x] address issue #30 (NSS support broke upsc output)
[x] address issue #15 (Avoid "2.6.x-Unversioned Directory" version numbers)
[x] check branch bcmxcp and merge if ok
[x] check branch voltronic-driver and merge if ok (but see #57)
[x] #57: check branch blzr and merge if ok; possibly removing voltronic
[x] add Tripp Lite information to drivers.list
[x] check branch powercom-instant-commands and merge if ok
[x] update news/upgrading files

clepple commented 11 years ago

Arnaud: do we want to merge the bcmxcp, voltronic-driver and powercom-instant-commands branches? I think they're pretty localized, but I haven't checked for stray files.

aquette commented 11 years ago

agreed, I've updated the tasks list. I'm currently looking at powercom-instant-commands (update: done). NEWS and UPGRADING are also underway.

aquette commented 11 years ago

Update on voltronic-driver branch:

voltronic_ser.c and voltronic_usb.c have not diverged enough from the master (blazer-{ser,usb}.c) to be worth a new implementation (cypress subdrv, default cable power, no lang fix) => for future improvement
voltronic.c: "god.knows" variables (extra status fields) to be adapted (unknown.1,2,3) ; not much to add ( @aquette to clarify this Qx variation Vs Innova Q5, for latter consideration)
no compilation warning

Conclusion: good to merge, but need to log a bug for future improvements.

clepple commented 11 years ago

I'm confused by the first part: "have not diverged enough ... to be worth a new implementation" versus "good to merge"?

aquette commented 11 years ago

sorry, the "for future improvement" was not bold enough. I didn't wanted to delay further the release, nor to evict voltronic from it. So it's really good to merge, but with a know and mandatory todo. Note that it's really limited to the communication layer (_ser and _usb). The Q1 (more a Q5) idiom is completely new and different. @clepple: if you can / want to handle the merge (including the "god knows") please be my guest. I still have the bcmxcp review, and the pure release process... and not enough time / energy :-/ btw, I've asked to my "Q1" contact if he knows something about the "voltronic" variation.

clepple commented 11 years ago

Merged voltronic-driver to master: de07fc7

clepple commented 11 years ago

Daniele suggests using the new blzr driver instead of voltronic. See pull request #57

clepple commented 11 years ago

I'd like to merge bcmxcp to master so we can get it into 2.7.1. Any objections?

aquette commented 11 years ago

Le 31 oct. 2013 13:46, "Charles Lepple" notifications@github.com a écrit :

I'd like to merge bcmxcp to master so we can get it into 2.7.1. Any objections?

I've still not had a chance to look at it, but let's go ahead now! Thanks Charles.

clepple commented 11 years ago

bcmxcp was merged in 50501d50

clepple commented 11 years ago

v2.7.1 was tagged 6a4729cfd