Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

[pilight] Added pilight feature #5769

Merged
merged 4 commits into from
Jan 26, 2019
Merged

Conversation

idserda
Copy link
Contributor

@idserda idserda commented Jan 14, 2019

This PR adds the pilight binding to the list of features so it can be easily installed. This fixes #5742 .

Signed-off-by: Jeroen Idserda <[email protected]>
Signed-off-by: Jeroen Idserda <[email protected]>
Copy link
Contributor

@9037568 9037568 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline comments, please.

@@ -108,6 +108,7 @@
<artifact><file>src/main/resources/conf/panasonictv.cfg</file><type>cfg</type><classifier>panasonictv</classifier></artifact>
<artifact><file>src/main/resources/conf/pebble.cfg</file><type>cfg</type><classifier>pebble</classifier></artifact>
<artifact><file>src/main/resources/conf/piface.cfg</file><type>cfg</type><classifier>piface</classifier></artifact>
<artifact><file>src/main/resources/conf/pilight.cfg</file><type>cfg</type><classifier>piface</classifier></artifact>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be causing the build to fail. Copy/Paste error?

13:40:15 [ERROR] Failed to execute goal org.codehaus.mojo:build-helper-maven-plugin:1.9.1:attach-artifact (attach-artifact) on project openhab-addons-external: The artifact with same type and classifier: cfg:piface is used more than once. -> [Help 1]

https://ci.openhab.org/job/PR-openHAB1-Addons/2415/console

# Port of the pilight daemon. You must explicitly configure the port in pilight or otherwise a random port will be used and the binding will not be able to connect.
# <instance>.port=<port>
#
# Optional delay (in millisecond) between consecutive commands. Recommended value without band pass filter: 1000. Recommended value with band pass filter: somewhere between 200-500
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"milliseconds"

@idserda
Copy link
Contributor Author

idserda commented Jan 16, 2019

Thanks! Changed the classifier and the typo. Build is succeeding now.

Copy link
Contributor

@9037568 9037568 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@9037568
Copy link
Contributor

9037568 commented Jan 26, 2019

Thanks, @idserda !

@9037568 9037568 merged commit 3c3311d into openhab:master Jan 26, 2019
@9037568 9037568 added this to the 1.14.0 milestone Jan 26, 2019
@kaikreuzer kaikreuzer changed the title [pilight] Add pilight feature [pilight] Added pilight feature Dec 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Pilight] Not included in any feature. Not installable via normal binding installation methods
2 participants