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.

Sign up for GitHub

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

Jump to bottom

jakarta.enterprise.cdi-el-api needs EL version update and Manifest updates #732

Closed
starksm64 opened this issue Dec 12, 2023 · 2 comments · Fixed by #733
Closed

jakarta.enterprise.cdi-el-api needs EL version update and Manifest updates #732

starksm64 opened this issue Dec 12, 2023 · 2 comments · Fixed by #733

Comments

@starksm64
Copy link
Contributor

In the M1, an update of the jakarta.enterprise.cdi-el-api artifact was not updated from 5. It should be 6.0.0-M1 or later for the next release.
@arjantijms
Copy link

Additionally, the MANIFEST.MF misses two exports. Maybe that should be a different issue.

I patched the MANIFEST and this seems to work:

Manifest-Version: 1.0
Created-By: Apache Maven Bundle Plugin 5.1.9
Build-Jdk-Spec: 11
Bundle-Description: API for integrating CDI with Unified EL
Bundle-DocURL: https://www.eclipse.org
Bundle-License: https://repository.jboss.org/licenses/apache-2.0.txt
Bundle-ManifestVersion: 2
Bundle-Name: CDI EL integration API
Bundle-SymbolicName: jakarta.enterprise.cdi-el-api
Bundle-Vendor: Eclipse Foundation
Bundle-Version: 4.1.0.M1
Export-Package: jakarta.enterprise.inject.spi.el;version="4.1";uses:="ja
 karta.el"
Import-Package: jakarta.el;version="6.0",jakarta.enterprise.context,jaka
 rta.enterprise.inject.spi
Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=11))"
Tool: Bnd-6.3.1.202206071316

In the current one it just says:

Import-Package: jakarta.el;version="5.0"

@starksm64 starksm64 changed the title jakarta.enterprise.cdi-el-api needs EL version update jakarta.enterprise.cdi-el-api needs EL version update and Manifest updates Dec 12, 2023
@starksm64
Copy link
Contributor Author

Ok, that is fine in this issue and I have updated the title

starksm64 added a commit to jakartaredhat/cdi-master that referenced this issue Dec 12, 2023
@starksm64
Update jakarta.el to 6.0.0-M1 and update osgi manifest export/imports, …
a5f77b8 
…jakartaee#732

Signed-off-by: Scott M Stark <starksm64@gmail.com>
starksm64 added a commit that referenced this issue Dec 12, 2023
@starksm64
Update jakarta.el to 6.0.0-M1 and update osgi manifest export/imports, …
0e49d4b 
…#732 (#733)

* Update jakarta.el to 6.0.0-M1 and update osgi manifest export/imports, #732
* Avoid using SE 11 on el module since it now depends on a jar with SE 17 classes

---------

Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64 starksm64 linked a pull request Dec 12, 2023 that will close this issue
Update jakarta.el to 6.0.0-M1 and update osgi manifest export/imports, #732 #733
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update jakarta.el to 6.0.0-M1 and update osgi manifest export/imports, #732 jakartaredhat/cdi-master
2 participants
@starksm64 @arjantijms