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

[not working] [jsscripting] Use OSGI-ified GraalVM dependencies #18053

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

florian-h05
Copy link
Contributor

@florian-h05 florian-h05 commented Jan 6, 2025

@florian-h05
Copy link
Contributor Author

The following package-info.java did not show any effect:

/**
 * Copyright (c) 2010-2025 Contributors to the openHAB project
 *
 * See the NOTICE file(s) distributed with this work for additional
 * information.
 *
 * This program and the accompanying materials are made available under the
 * terms of the Eclipse Public License 2.0 which is available at
 * http://www.eclipse.org/legal/epl-2.0
 *
 * SPDX-License-Identifier: EPL-2.0
 */
@Requirement(namespace = ExtenderNamespace.EXTENDER_NAMESPACE, filter = "(osgi.extender=osgi.serviceloader.processor)")
@Requirement(namespace = "osgi.serviceloader", filter = "(osgi.serviceloader=org.graalvm.polyglot.impl.AbstractPolyglotImpl)", cardinality = Requirement.Cardinality.MULTIPLE)
@Header(name = Constants.DYNAMICIMPORT_PACKAGE, value = "*")
@Header(name = Constants.IMPORT_PACKAGE, value = "org.graalvm.home,org.graalvm.home.impl,org.graalvm.options,org.graalvm.polyglot,org.graalvm.polyglot.impl,org.graalvm.polyglot.io,org.graalvm.polyglot.management,org.graalvm.polyglot.proxy")
package org.openhab.automation.jsscripting;

/**
 * Additional information for JavaScript Scripting package
 *
 * @author Florian Hotze - Initial contribution
 *
 */

import org.osgi.annotation.bundle.Header;
import org.osgi.annotation.bundle.Requirement;
import org.osgi.framework.Constants;
import org.osgi.namespace.extender.ExtenderNamespace;

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

Successfully merging this pull request may close these issues.

1 participant