NicholasNoise commented on pull request #66:
URL: https://github.com/apache/logging-log4net/pull/66#issuecomment-688843984


   > @cremor 
   > 
   > Some of those are the same as are now manually defined in the 
log4net.csproj and some are different. This might be confusing for developers 
who don't check if the constants are defined by the project itself.
   > 
   > But it also seems like migration to the official constants is not that 
easy. Because with them it seems like the version specific constants are 
**not** enabled for lower versions when a higher version is built.
   
   You're right, no other way about it. Build-in features are preferable.
   But it seems unnecessary since this project is dormant.
   
   I've spent almost two working day to enable netstandard2.0, so yes, is not 
easy... until old targets are supported.
   Should be much easier without legacy preprocessor symbols like 'NETCF', 
'CLIENT_PROFILE' and even 'NET20'


----------------------------------------------------------------
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.

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


Reply via email to