Page MenuHomeFreeBSD

New port: sysutils/solaar
ClosedPublic

Authored by tobik on Mar 27 2017, 10:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 24, 8:23 AM
Unknown Object (File)
Tue, Dec 24, 7:12 AM
Unknown Object (File)
Tue, Dec 24, 4:17 AM
Unknown Object (File)
Fri, Dec 20, 5:22 AM
Unknown Object (File)
Nov 25 2024, 12:10 PM
Unknown Object (File)
Nov 21 2024, 4:57 PM
Unknown Object (File)
Nov 20 2024, 9:35 PM
Unknown Object (File)
Nov 19 2024, 12:02 AM
Subscribers
None

Details

Summary
Solaar is a device manager for Logitech's Unifying Receiver.  It is
able to pair/unpair devices to the receiver, and for most devices read
battery status.

It comes in two flavors, command-line and GUI.  Both are able to list
the devices paired to a Unifying Receiver, show detailed info for each
device, and also pair/unpair supported devices with the receiver.

WWW: https://pwr.github.io/Solaar/

Approved by:	??? (mentor)
Differential Revision:	https://reviews.freebsd.org/D?????
Test Plan

poudriere testport 11.0/amd64 ok
poudriere testport 10.3/i386 ok

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 8331
Build 8598: arc lint + arc unit

Event Timeline

sysutils/solaar/Makefile
14

Should this be a LIB_DEPENDS or is this ok? Solaar loads libhidapi.so via ctypes (dlopen).

sysutils/solaar/Makefile
14

If it is not needed at build-time, then RUN_DEPENDS is ok. Also LIB_DEPENDS may have a special treatment (I'm not sure it does) as in it may only be registered if pkg finds the .so in the elf files.
Also, add a comment before the RUN_DEPENDS line saying that it is only loaded at runtime, so that someone does not change it later on.

22

I was about to ask why you did not use the latest release tag (but it's 4 years old) so I'm going to say something else, when you find a github probject that stopped tagging releases, it may be nice to open an issue there asking gently for them to do so :-)

26–28

Add a GUI option that installs it ?

sysutils/solaar/files/patch-setup.py
13

This is probably wrong, 0.9.2 is from 2013.

Add GUI option, and comments about releases and RUN_DEPENDS

sysutils/solaar/files/patch-setup.py
13

Yes, but that is what's defined in lib/solaar/__init__py. The import here fails for some reason and I decided to just define NAME and __version__ myself with the original values.

0.9.2 appears to be hardcoded in several other places, too.

mat added inline comments.
sysutils/solaar/files/patch-setup.py
13

Oh, ok, fine then :-)

This revision is now accepted and ready to land.Mar 28 2017, 11:46 AM
This revision was automatically updated to reflect the committed changes.