Re: [Chainsaw] Minor fixes

2021-05-08 Thread Matt Sicker
CI fixed as of https://ci-builds.apache.org/job/Logging/job/chainsaw/job/master/16/ now using Java 11 to build. On Sat, 8 May 2021 at 14:41, Robert Middleton wrote: > > > > On Sat, May 8, 2021, 11:19 AM Matt Sicker wrote: > > > > As for committing the repository, > > > > you are a committer and

Re: [Chainsaw] Minor fixes

2021-05-08 Thread Robert Middleton
> > On Sat, May 8, 2021, 11:19 AM Matt Sicker wrote: > > > As for committing the repository, > > > you are a committer and can do so. We do commit-then-review here which > > > allows people to asynchronously review changes with lazy consensus. Sounds fine to me - I just didn't want to go and star

Re: [Chainsaw] Minor fixes

2021-05-08 Thread Matt Sicker
Looks like Jenkins is under maintenance at the moment. Will check if my recent changes fixed that build when it's up again. On Sat, 8 May 2021 at 13:22, Scott Deboy wrote: > > Chainsaw Mac integration leverages java9 features now (Mac prefs menu > integration). I can review merge requests. > > On

Re: [Chainsaw] Minor fixes

2021-05-08 Thread Scott Deboy
Chainsaw Mac integration leverages java9 features now (Mac prefs menu integration). I can review merge requests. On Sat, May 8, 2021, 11:19 AM Matt Sicker wrote: > It appears that Chainsaw updated its minimum Java version to 9. I'll > get that fixed in the Jenkins build. As for committing the re

Re: [Chainsaw] Minor fixes

2021-05-08 Thread Matt Sicker
It appears that Chainsaw updated its minimum Java version to 9. I'll get that fixed in the Jenkins build. As for committing the repository, you are a committer and can do so. We do commit-then-review here which allows people to asynchronously review changes with lazy consensus. On Fri, 7 May 2021

[Chainsaw] Minor fixes

2021-05-07 Thread Robert Middleton
I've been doing some messing around with chainsaw the past few days, and I've fixed a few minor issues(and put in some PRs for them) - does somebody(Scott?) want to take a look at them? I can merge stuff if needed, but I'm not sure if somebody else prefers to take some lead on chainsaw. Side note