Skip to content

Restore Apache Felix Logback module - #543

Open
wilx wants to merge 24 commits into
apache:masterfrom
wilx:restore-felix-logback
Open

Restore Apache Felix Logback module#543
wilx wants to merge 24 commits into
apache:masterfrom
wilx:restore-felix-logback

Conversation

@wilx

@wilx wilx commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

I would like to restore the Logback component. I have restored the original source and updated it with Logback 1.5.x, plus some changes I see as beneficial. It works for me in production like this.

@wilx
wilx marked this pull request as ready for review August 11, 2026 18:26
@rombert

rombert commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Speaking as a user of Felix Logback that would IMO be good. I am a Felix committer but never worked with the logback bundle. What do other Felix committers/PMC members think ( /cc @apache/felix-committers ) ?

@cziegeler

Copy link
Copy Markdown
Contributor

We removed this couple of years ago due to lack of maintainers . We had a similar situation with another Apache Felix module where we actually reverted the removal, added a new committer promising to help on that module. And it resulted in a larger unmaintained code base, as after an initial fix nothing else happened.
I dont want to repeat that - the code is under the ASL 2 so it is easy to "continue" it somewhere else.
Now, if we decide to bring it back in, I suggest we directly revert the removal commit - and not do it via a PR

@wilx

wilx commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

What maintenance does this need beside keeping it on par with some Logback releases?

@cziegeler

Copy link
Copy Markdown
Contributor

Right, keeping all dependencies up to date, releasing whenever required - fixing issues if they get reported.
It is not a long list, but nevertheless it needs someone to do it regularly and not just for one release

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.

4 participants