Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bin2json] bin2json Transformation #5348

Merged
merged 8 commits into from
Apr 1, 2019

Conversation

paulianttila
Copy link
Contributor

Binary To JSON transformation service

Signed-off-by: Pauli Anttila [email protected]

Signed-off-by: Pauli Anttila <[email protected]>
@paulianttila paulianttila requested a review from a team as a code owner March 31, 2019 12:58
@paulianttila paulianttila added work in progress A PR that is not yet ready to be merged new transform labels Mar 31, 2019
Copy link
Member

@davidgraeff davidgraeff left a comment

Choose a reason for hiding this comment

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

I would say we merge the transform here, and people can move it over to core at some point if they want to.

Looks good so far, except the bundled jar file.

Please notify when you are done


== Third-party Content

jbbp
Copy link
Member

Choose a reason for hiding this comment

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

Can the jbbp-1.4.1.jar library be provided by maven as dependency instead of adding the jar file?

Copy link
Member

@davidgraeff davidgraeff left a comment

Choose a reason for hiding this comment

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

That's much better. You need to adapt the karaf feature file as well, so that the dependency is downloaded when installing the transformation.

Signed-off-by: Pauli Anttila <[email protected]>
Signed-off-by: Pauli Anttila <[email protected]>
@paulianttila paulianttila removed the work in progress A PR that is not yet ready to be merged label Mar 31, 2019
@paulianttila paulianttila changed the title [WIP][bin2json] Initial commit [bin2json] Initial commit Mar 31, 2019
@paulianttila
Copy link
Contributor Author

Thanks David. I added it to Karaf features, even it most probably cause merge problems if merge is delayed.

@paulianttila
Copy link
Contributor Author

paulianttila commented Mar 31, 2019

Forgotten that JBBP jar is not a OSGi bundle so build fails. I add static jar file back to binding.

Signed-off-by: Pauli Anttila <[email protected]>
@davidgraeff
Copy link
Member

No you should use the "wrap" prefix. Have a look at all the other bundles in the karaf feature file that are using wrap already.

If there is a maven release, a jar file will no longer be accepted.

@paulianttila
Copy link
Contributor Author

@davidgraeff, that's sounds great...will do.

@davidgraeff davidgraeff merged commit f9f4948 into openhab:master Apr 1, 2019
@wborn wborn added this to the 2.5 milestone Apr 14, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Jun 19, 2019
Binary To JSON transformation service

Signed-off-by: Pauli Anttila <[email protected]>
Signed-off-by: Pshatsillo <[email protected]>
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
Binary To JSON transformation service

Signed-off-by: Pauli Anttila <[email protected]>
Signed-off-by: Maximilian Hess <[email protected]>
@kaikreuzer kaikreuzer changed the title [bin2json] Initial commit bin2json Transformation Dec 8, 2019
@kaikreuzer kaikreuzer changed the title bin2json Transformation [bin2json] bin2json Transformation Dec 8, 2019
@paulianttila paulianttila deleted the bin2json-transform branch March 14, 2020 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants