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

Issue when the SDK is executed within a Domino agent #1604

Closed
priand opened this Issue Jul 30, 2014 · 2 comments

Comments

Projects
None yet
2 participants
@priand

priand commented Jul 30, 2014

The ExtensionManager (in commons) assumes that OSGi is available based on a system property. This actually doesn't work in case of a domino as the property is defined globalle while OSGi is not available for agent.
The fix consists in trying load the class (one line of real code), like shown bellow, before creating the EclipseProvider:
try {
String prop = System.getProperty("com.ibm.common.ExtensionManager"); //$NON-NLS-1$
if(StringUtil.isEmpty(prop) || StringUtil.equals(prop, "osgi")) { //$NON-NLS-1$
// Make sure that the OSGi framework is actually available
// In case of a Domino agent, for example, the properties are set but OSGi is not available
// Make sure that it exists - if not, it generates an exception that will be caught.
Class.forName("org.eclipse.core.runtime.Platform");
provider = new EclipseProvider();
}
} catch(Throwable t) {}

Can you guys integrate this fix?

@CarlosManias

This comment has been minimized.

Show comment
Hide comment
@CarlosManias

CarlosManias Jul 31, 2014

Created defect #146784

CarlosManias commented Jul 31, 2014

Created defect #146784

@CarlosManias CarlosManias added the bug label Jul 31, 2014

@priand

This comment has been minimized.

Show comment
Hide comment
@priand

priand Aug 14, 2014

@markewallace @CarlosManias Can you guys get this fix checked in? Thanks!

priand commented Aug 14, 2014

@markewallace @CarlosManias Can you guys get this fix checked in? Thanks!

CarlosManias pushed a commit to CarlosManias/SocialSDK that referenced this issue Aug 18, 2014

CarlosManias pushed a commit that referenced this issue Aug 18, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment