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

OSGI issues with 5.0.0-M1 build #4744

Open
hmottestad opened this issue Aug 10, 2023 · 5 comments · Fixed by #4746
Open

OSGI issues with 5.0.0-M1 build #4744

hmottestad opened this issue Aug 10, 2023 · 5 comments · Fixed by #4746
Labels
🐞 bug issue is a bug

Comments

@hmottestad
Copy link
Contributor

hmottestad commented Aug 10, 2023

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.1.0:run (enrich-main-artifact-with-provider-configuration-files) on project rdf4j-runtime-osgi: An Ant BuildException has occured: The following error occurred while executing this line:

[ERROR] /home/jenkins/agent/workspace/rdf4j-deploy-release-ossrh/tools/runtime-osgi/build.xml:5: src '/home/jenkins/agent/workspace/rdf4j-deploy-release-ossrh/tools/runtime-osgi/target/serviceResources/raw/raw-service-resources.jar' doesn't exist.

[ERROR] around Ant part ...... @ 4:173 in /home/jenkins/agent/workspace/rdf4j-deploy-release-ossrh/tools/runtime-osgi/target/antrun/build-main.xml

@github-project-automation github-project-automation bot moved this to 🆕 Triage in RDF4J Planning Aug 10, 2023
@hmottestad hmottestad added the 🐞 bug issue is a bug label Aug 10, 2023
@hmottestad hmottestad added this to the 5.0.0 milestone Aug 10, 2023
@hmottestad
Copy link
Contributor Author

This can be reproduced on the develop branch with:

mvn clean
mvn package -Passembly -DskipTests

An workaround is to use mvn install first:

mvn clean
mvn install -DskipTests
mvn package -Passembly -DskipTests

@hmottestad
Copy link
Contributor Author

This is still an issue. It used to work without having to first call mvn install.

@abrokenjester abrokenjester removed their assignment Aug 19, 2023
@kenwenzel
Copy link
Contributor

We could also use a bnd based build like:
https://github.com/komma/komma/tree/master/bundles/vendor/net.enilink.bundles.rdf4j

@hmottestad
Copy link
Contributor Author

We could also use a bnd based build like:
https://github.com/komma/komma/tree/master/bundles/vendor/net.enilink.bundles.rdf4j

I have basically no experience with OSGI so I also don’t know that bnd based build is. From looking at the bnd file in the link it seems a lot simpler than what we are currently doing.

If you can contribute a fix based on bnd I would be very grateful :)

@kenwenzel
Copy link
Contributor

It is not that different from the current approach. The Felix Maven Bundle Plugin just uses bnd under the hood.
But I think we can get rid of ant by using the assembly plugin and simplify a bit by directly using the bnd-maven-plugin.

An even better approach could be to just add correct OSGi manifests to the individual artifacts as mentioned in #4079 and drop the runtime-osgi artifact.

I'll take a look within the next days.

@hmottestad hmottestad modified the milestones: 5.0.0, 5.1.0 Jun 20, 2024
@hmottestad hmottestad removed this from the 5.1.0 milestone Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug issue is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants