Hi Michael, Answers inline below:
On Fri, Jul 17, 2020 at 2:22 AM Michael Bien <[email protected]> wrote: > On 16.07.20 15:49, Dave wrote: > > Wow, these sound great! Please create some PRs, break them up and order > > them as you wish. > > > > Dave > > Hello Dave, > > a few questions: > > 1) I submitted the ICLA but I can't find me on the committer-index yet - > not sure if that is a problem. > Your ICLA is on file according to this page: http://people.apache.org/unlistedclas.html > 2) which branch should the PRs target? roller-6.0.x is up-to-date and my > wip branch is based on it, but this might be the wrong branch for larger > changes. Maybe fork roller-6.0.x into the next version or a fresh master > (current master had some commits missing which caused me to fix things > which were not broken https://github.com/apache/roller/pull/58 ) > I would recommend targeting the master branch. Any changes in roller-6.0.x should be merged to master. I think your changes should go into the next non-patch release, i.e. 6.1.0. > 3) roller currently supports having log4j1 configuration in > roller.properties + roller-custom.properties and merges them to directly > initialize log4j1 (breaking through the commons.logging layer). This > won't work anymore once roller moved to log4j2. The way it would work is > to have a default log4j2.xml in the war, which can be copied and > overwritten by putting it in the classpath of the server. Would this be ok? > I'm OK with that change. Anybody else want to comment? Dave
