search

eclipse-ee4j / mojarra

Mojarra, a Jakarta Faces implementation
Other
165 stars 112 forks source link

Fix #5484: api classes should not have impl dependencies #5485

Closed BalusC closed 2 months ago

BalusC commented 3 months ago

https://github.com/eclipse-ee4j/mojarra/issues/5484

volosied commented 3 weeks ago

@arjantijms @BalusC

This change did not make it in to 4.1.1. https://repo1.maven.org/maven2/jakarta/faces/jakarta.faces-api/4.1.1/

The source shows: 

import com.sun.faces.config.InitFacesContext;
...

    private static final ConcurrentHashMap<Thread,FacesContext> threadInitContext = new ConcurrentHashMap<>(2);
    private static final ConcurrentHashMap<Thread,InitFacesContext> initContextServletContext = new ConcurrentHashMap<>(2);

Can we get another release?

BalusC commented 3 weeks ago

Nope it's in Mojarra 4.1.2 which was released yday. I don't know the correct procedure to release the API JAR, only @arjantijms does. While at it can you please document the procedure so others can do it in the future as well?