Remove unused logging.properties that is shipped on the classpath by basil · Pull Request #102 · jenkinsci/swarm-plugin (original) (raw)

After merging #99, I realized there was another logging.properties checked in at client/src/main/java/hudson/plugins/swarm/logging.properties. This was completely unused. I thought it was confusing that we ship dead code on the classpath, so I simply deleted this. The one I checked in at client/logging.properties was almost the same. I fixed it up to include a few other goodies from the original client/src/main/java/hudson/plugins/swarm/logging.properties (e.g., %u in the filename, and java.util.logging.SimpleFormatter.format). Otherwise, the only other differences were increased verbosity, which is actually a good thing given that users should be only using it when they are in a debugging situation. With these differences having been reconciled, I am deleting the old client/src/main/java/hudson/plugins/swarm/logging.properties. It should now be clear that the JAR ships with no logging configuration, and if users want to turn up logging they will need to copy the file from client/logging.properties to the filesystem (not classpath!) and enable it as described in the documentation.