[GitHub] [logging-log4net] rgoers commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
rgoers commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-686268036 Thanks. Those should be fixed now. This is an automated message from the Apache Git Service. To respond to th

[GitHub] [logging-log4net] cremor commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
cremor commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-686260395 @rgoers Now the link https://logging.apache.org/log4net/ works, but the link which is on https://logging.apache.org, which is https://logging.apache.org/log4net/2.x/index.htm

[jira] [Commented] (LOGCXX-510) Build problems using CMAKE and Visual Studio 2019 Community

2020-09-02 Thread Robert Middleton (Jira)
[ https://issues.apache.org/jira/browse/LOGCXX-510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17189749#comment-17189749 ] Robert Middleton commented on LOGCXX-510: - That looks pretty good to me - I'll ta

[CI][UNSTABLE] Logging/log4j/master#147 has test failures

2020-09-02 Thread Mr. Jenkins
BUILD UNSTABLE Build URL https://ci-builds.apache.org/job/Logging/job/log4j/job/master/147/ Project: master Date of build: Wed, 02 Sep 2020 18:25:06 + Build duration: 1 hr 37 min and counting JUnit Tests Name: liquibase.ext.logging.log4j2 Failed:

[GitHub] [logging-log4net] fluffynuts commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
fluffynuts commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-685954452 Thanks for sorting it out @rgoers. I'll try to be better prepared next time. This is an automated message

[GitHub] [logging-log4net] rgoers commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
rgoers commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-685947511 FWIW, the documentation for publishing the web site is at https://cwiki.apache.org/confluence/display/LOGGING/Managing+the+Git+Hosted+Web+Site. So you would run your build to

[GitHub] [logging-log4net] rgoers commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
rgoers commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-685945845 Please try it again. This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [logging-log4net] fluffynuts commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
fluffynuts commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-685937486 @cremor thanks for reporting; logging sites are in a bit of a state of upheaval right now (as far as I understand). Good people are On It. --

[GitHub] [logging-log4net] fluffynuts edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685935299 > Reflecting my previous experience adding or dropping targets are rarely considered as breaking\major change, minor at best. it's breaking for people who de

[GitHub] [logging-log4net] fluffynuts edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685935299 > Reflecting my previous experience adding or dropping targets are rarely considered as breaking\major change, minor at best. it's breaking for people who de

[GitHub] [logging-log4net] fluffynuts edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685935299 > Reflecting my previous experience adding or dropping targets are rarely considered as breaking\major change, minor at best. it's breaking for people who de

[GitHub] [logging-log4net] fluffynuts commented on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts commented on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685935299 > Reflecting my previous experience adding or dropping targets are rarely considered as breaking\major change, minor at best. it's breaking for people who depend on it;

[GitHub] [logging-log4net] jazpearson commented on pull request #64: XmlConfigurator: do longer allow dtd processing across all platforms …

2020-09-02 Thread GitBox
jazpearson commented on pull request #64: URL: https://github.com/apache/logging-log4net/pull/64#issuecomment-685875512 Would be great to see this merged as this is blocking us, This is an automated message from the Apache Gi

[GitHub] [logging-log4net] NicholasNoise commented on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
NicholasNoise commented on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685726181 > > > I'd be very careful about dropping target frameworks. This is a breaking change. All projects using that framework will no longer be able to use the newe

[GitHub] [logging-log4net] cremor commented on pull request #62: Move site back where it belongs

2020-09-02 Thread GitBox
cremor commented on pull request #62: URL: https://github.com/apache/logging-log4net/pull/62#issuecomment-685723562 I'm not sure if this belongs to the changes made in this PR, but https://logging.apache.org/log4net/ (with is the top Google result if you search for "log4net") currently lea

[GitHub] [logging-log4net] cremor edited a comment on pull request #59: Update build system

2020-09-02 Thread GitBox
cremor edited a comment on pull request #59: URL: https://github.com/apache/logging-log4net/pull/59#issuecomment-685715057 @fluffynuts Thanks for the pointer to #63 and for the precompiled NuGet package from that PR! I just tested it and it indeed fixes all of the points I mentioned above.

[GitHub] [logging-log4net] cremor commented on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
cremor commented on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685717421 > Also I've dropped `net35-client` and `net40-client` as target frameworks, because unable to compile against. Should we bring them back and test somehow or deprecate and dro

[GitHub] [logging-log4net] cremor commented on pull request #59: Update build system

2020-09-02 Thread GitBox
cremor commented on pull request #59: URL: https://github.com/apache/logging-log4net/pull/59#issuecomment-685715057 @fluffynuts Thanks for the pointer to #63 and for the precompiled NuGet package from that PR! I just tested it and it indeed fixes all of the points I mentioned above.

[GitHub] [logging-log4net] NicholasNoise edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
NicholasNoise edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685695290 @fluffynuts Thank you for quick response! I made this PR from non-personal fork, but already requested branch setting change to an admin of organization (shou

[GitHub] [logging-log4net] NicholasNoise edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
NicholasNoise edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685695290 @fluffynuts Thank you for quick response! I made this PR from non-personal fork, but already requested branch setting change to an admin of organization (shou

[GitHub] [logging-log4net] NicholasNoise commented on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
NicholasNoise commented on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685695290 @fluffynuts Thank you for quick response! I made this PR from non-personal fork, but already requested branch setting change to an admin of organization (should be g

[GitHub] [logging-log4net] SymbioticKilla commented on pull request #64: XmlConfigurator: do longer allow dtd processing across all platforms …

2020-09-02 Thread GitBox
SymbioticKilla commented on pull request #64: URL: https://github.com/apache/logging-log4net/pull/64#issuecomment-685614850 I hope it is fine. If not than sorry => close/delete it. Thanks! This is an automated message from

[GitHub] [logging-log4net] SymbioticKilla opened a new pull request #64: XmlConfigurator: do longer allow dtd processing across all platforms …

2020-09-02 Thread GitBox
SymbioticKilla opened a new pull request #64: URL: https://github.com/apache/logging-log4net/pull/64 …(LOG4NET-575) This patch fixes a security vulnerabiliy reported by Karthik Balasundaram. The security vulnerability was found in the way how log4net parses xml configuration file

[GitHub] [logging-log4net] fluffynuts commented on pull request #59: Update build system

2020-09-02 Thread GitBox
fluffynuts commented on pull request #59: URL: https://github.com/apache/logging-log4net/pull/59#issuecomment-685547877 @cremor could you see if the changes here fix this: https://github.com/apache/logging-log4net/pull/63 Perhaps: 1. clone the PR branch 2. build (if you have no

[GitHub] [logging-log4net] cremor commented on pull request #59: Update build system

2020-09-02 Thread GitBox
cremor commented on pull request #59: URL: https://github.com/apache/logging-log4net/pull/59#issuecomment-685486982 @jvz Thanks. Does this mean I should create tickets for the points in my last comment? This is an automated

[GitHub] [logging-log4j2] vy edited a comment on pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-09-02 Thread GitBox
vy edited a comment on pull request #335: URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-685462219 @ramazanpolat, 1. Unless there is a reason to do so, I would avoid custom classes to represent the logging structure. In your case a simple `logger.info(new MapMessage

[GitHub] [logging-log4j2] vy commented on pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-09-02 Thread GitBox
vy commented on pull request #335: URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-685462219 1. Unless there is a reason to do so, I would avoid custom classes to represent the logging structure. In your case a simple `logger.info(new MapMessage(Map.of("message", messag

[GitHub] [logging-log4net] fluffynuts edited a comment on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts edited a comment on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685442735 @NicholasNoise I really appreciate the PR & the time taken to produce multi-target tests! Good job. There are two changes I'd like to make -- either we can c

[GitHub] [logging-log4net] fluffynuts commented on pull request #63: Implemented proper support of netstandard2.0

2020-09-02 Thread GitBox
fluffynuts commented on pull request #63: URL: https://github.com/apache/logging-log4net/pull/63#issuecomment-685442735 @NicholasNoise I really appreciate the PR & the time taken to produce multi-target tests! Good job. There are two changes I'd like to make -- either we can collabor

[jira] [Comment Edited] (LOGCXX-510) Build problems using CMAKE and Visual Studio 2019 Community

2020-09-02 Thread Jira
[ https://issues.apache.org/jira/browse/LOGCXX-510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17189028#comment-17189028 ] Thorsten Schöning edited comment on LOGCXX-510 at 9/2/20, 7:34 AM:

[jira] [Updated] (LOGCXX-510) Build problems using CMAKE and Visual Studio 2019 Community

2020-09-02 Thread Jira
[ https://issues.apache.org/jira/browse/LOGCXX-510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thorsten Schöning updated LOGCXX-510: - Attachment: cmake_settings_using_vs_gui 02.png > Build problems using CMAKE and Visual St

[jira] [Comment Edited] (LOGCXX-510) Build problems using CMAKE and Visual Studio 2019 Community

2020-09-02 Thread Jira
[ https://issues.apache.org/jira/browse/LOGCXX-510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17189028#comment-17189028 ] Thorsten Schöning edited comment on LOGCXX-510 at 9/2/20, 7:04 AM:

[jira] [Commented] (LOGCXX-510) Build problems using CMAKE and Visual Studio 2019 Community

2020-09-02 Thread Jira
[ https://issues.apache.org/jira/browse/LOGCXX-510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17189028#comment-17189028 ] Thorsten Schöning commented on LOGCXX-510: -- > 2. The directories for APR and APR