[GitHub] [logging-log4j2] garydgregory commented on pull request #684: Exclude Eclipse project file from RAT in root POM

[GitBox <gi...@apache.org>]

garydgregory commented on pull request #684:
URL: https://github.com/apache/logging-log4j2/pull/684#issuecomment-1006027274


   > The [test failures here](https://github.com/vorburger/logging-log4j2/runs/4718590938?check_suite_focus=true) appears to be unrelated to this PR, looks like someone broke the build with a failing test in a recently merged change, @carterkozak @garydgregory @rgoers FYI:
   > 
   > ```
   > org.apache.log4j.config.PropertiesConfigurationTest.testSystemProperties
   > Error:    Run 1: PropertiesConfigurationTest.testSystemProperties:138 expected:</tmp/[/]hadoop.log> but was:</tmp/[]hadoop.log>
   > Error:    Run 2: PropertiesConfigurationTest.testSystemProperties:138 expected:</tmp/[/]hadoop.log> but was:</tmp/[]hadoop.log>
   > Error:    Run 3: PropertiesConfigurationTest.testSystemProperties:138 expected:</tmp/[/]hadoop.log> but was:</tmp/[]hadoop.log>```
   > ```
   Hi,
   Rebase on master and it should work. I initially wrote the test on macOS and later found it failed when I ran it on Windows.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org