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

A refference to packages available in Qivicon target platform file. #306

Closed
wants to merge 1 commit into from

Conversation

YordanDZhelev
Copy link
Contributor

Signed-off-by: YordanDZhelev zhelev.yordan@gmail.com

Signed-off-by: YordanDZhelev <zhelev.yordan@gmail.com>
@maggu2810
Copy link
Contributor

Shouldn't we add also the versions of that packages (e.g. guava)?

@kaikreuzer
Copy link
Contributor

@YordanDZhelev , there are several problems with this PR:

  • You do not seem to have a CLA
  • QIVICON should not be referenced in any way. This is documentation about Eclipse SmartHome, not QIVICON.
  • As @maggu2810 points out, we should document the available versions
  • Source bundles should not be listed here
  • Feature groups are no bundles
  • As discussed before, there should be some categorization, i.e. which dependencies are meant for what (what do I use as an HTTP client?, what for JSON parsing?, etc).
  • We should define a subset of dependencies that every ESH-solution HAS to have available and a set of optional bundles that addons can choose to depend on (and are hence only required if those addons are used).
  • Furthermore it should be defined what needs to be done, if you require a dependency that is NOT listed.

I'm closing this PR, feel free to open a new one.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants