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

osgi-bundle for oslc4j-client. #7

Closed
wants to merge 3 commits into from

Conversation

yashkhatri
Copy link
Member

@yashkhatri yashkhatri commented Aug 9, 2018

Created an OSGI Bundle for Lyo Java Client.

Closes #6

@yashkhatri
Copy link
Member Author

Not Verified!! Need a use case to verify this. However, building fine and a plugin is created.

Created an OSGI Bundle for Lyo Java Client.

Change-Id: I8719f933eb0ce04446fb2710c516af27f9b39271
Signed-off-by: Yash Khatri <yash.khatri@scania.com>
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>

<groupId>org.eclipse.lyo.clients</groupId>
Copy link
Member

Choose a reason for hiding this comment

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

is it the right group? I don't recall clients being multiple

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes. This was the group id in client.java project

<groupId>org.eclipse.lyo.clients</groupId>


<properties>
<bundle.symbolicName>oslc4j-client-bundle</bundle.symbolicName>
<bundle.namespace>org.oslc4j.client.bundle</bundle.namespace>
Copy link
Member

Choose a reason for hiding this comment

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

I think everything we do here should begin with org.eclipse.lyo

Copy link
Member Author

Choose a reason for hiding this comment

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

Agree! We need to change in oslc4j-bundle too. I was just trying to keep it in sync with that.
https://github.com/eclipse/lyo.core/blob/82ec00cf193bb2df00d45321a9d173f1126bb9ce/oslc4j-bundle/pom.xml#L19

org.eclipse.lyo.client.oslcresources,
</_exportcontents>
<Import-Package>
javax.servlet;version="2.5.0",
Copy link
Member

Choose a reason for hiding this comment

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

I think the POM file for the client was quite big. Are you sure it will work with such a small number of packages imported once someone tries to invoke it?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think it will work fine. Similar was the structure for the OSLC4J-Bundle. However, need to see once we try to use it.

@berezovskyi berezovskyi added this to the 2.4.0 milestone Aug 9, 2018
@berezovskyi
Copy link
Member

Just put it to the 2.4.0 milestone, or shall I leave it for later, @jamsden

Change-Id: Ibe96b417e91ee71eb5a09c3f0c1ac70dd17d8e21
Signed-off-by: Yash Khatri <yash.khatri@scania.com>
Change-Id: I42ddedc3abe8d3b75ad4b1a677c4ef5acd9ab0f0
Signed-off-by: Yash Khatri <yash.khatri@scania.com>
@berezovskyi berezovskyi modified the milestones: 2.4.0, 4.0.0 Nov 11, 2018
@berezovskyi
Copy link
Member

switching milestone to 4.0.0

@berezovskyi
Copy link
Member

Resolved in LyoD

@berezovskyi berezovskyi closed this Nov 9, 2020
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.

Bundling the client as osgi bundle
2 participants