[GitHub] [activemq-artemis] gemmellr commented on pull request #3290: ARTEMIS-2933 Upgrade com.sun.winsw to 2.7.0-net4

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

gemmellr commented on pull request #3290:
URL: https://github.com/apache/activemq-artemis/pull/3290#issuecomment-921615621


   I would maybe avoid adding simply '-net4' in the name/patch at this point given 6 is already in RC phase?
   
   The support pages https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/versions-and-dependencies suggest all versions of windows back to 8/server2012 have some .Net 4 framework version installed by default, and all versions back to 7/server2008 can get one if not installed.
   
   Given that it doesnt seem unreasonable to require it at this point, so what about the reverse possibility? Make an alternative 'new' "-net3" version that ships for a limited time as a fallback option if we think it necessary (given above, I probably wouldnt), and then anyone actually affected by the change can use it instead if they dont want to use one of the many 4+ versions already available to them. 
   
   I wouldn't say dropping support of a .Net version in a minor release is all that different than dropping support of a Java version in a minor release, which has been done before and is about to be done again.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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