Created attachment 186916 [details]
telldus-core devd fixes
Update devd rules as per hps@ recommendations in mail.
General cleanup to use avoid @preunexec etc, and use @sample for /var/telldus/tellstick-core.conf (which is not really a conf but a state file for telldusd).
This superseeds 198019 which contained same/similar devd patch, which was never applied but lost somewhere along the way of other changes. That contained iconv stuff too, which does not seem to be required any longer (after https://github.com/freebsd/freebsd-ports/commit/ff176a3eb176ed56a19a9ec55117606d2c0c34a4)
Created attachment 186917 [details]
*** Bug 198019 has been marked as a duplicate of this bug. ***
Looks good to me.
FYI: most changed patch files are due to running "make makepatch", it's only the devd file which actually changed.
I see we need to duplicate the rules for the USB device auto plug:
ugen0.3: <Telldus TellStick Duo> at usbus0, cfg=0 md=HOST spd=FULL (12Mbps) pwr=ON (90mA)
bLength = 0x0012
bDescriptorType = 0x0001
bcdUSB = 0x0200
bDeviceClass = 0x0000 <Probed by interface class>
bDeviceSubClass = 0x0000
bDeviceProtocol = 0x0000
bMaxPacketSize0 = 0x0008
idVendor = 0x1781
idProduct = 0x0c31
^^^ we also need to match this connect and disconnect event
bcdDevice = 0x0600
iManufacturer = 0x0001 <Telldus>
iProduct = 0x0002 <TellStick Duo>
iSerialNumber = 0x0003 <XXXXXXXXXXXXXXXXXXXXXXX>
bNumConfigurations = 0x0001
Can you update the patch?
Created attachment 187625 [details]
telldus-core devd fixes, including Tellstick Duo
Updated patch to include vid/pid for Tellstick Duo. Note that I have no duo to test with.
Only difference from previous patch is added devd rules to already existing file.
Fixed some @CMAKE_INSTALL_PREFIX@ in the devd configuration file before commit.
A commit references this bug:
Date: Wed Nov 1 07:55:35 UTC 2017
New revision: 453267
Update devd rules and various build cleanup.
Added support for Tellstick Duo.
Approved by: maintainer (Johan Strom, firstname.lastname@example.org)
Tested by: hselasky @