I guess that most Log4j users are not going to read the comments in the POM.
On Wed, Sep 17, 2014 at 3:34 AM, Matt Sicker <[email protected]> wrote: > We'd certainly like to improve this in the documentation and have been > discussing it recently. I made sure to add a bunch of comments (which > inspired others to do the same afterward) to the pom file to note what each > dependency was used for. > > On 16 September 2014 02:39, Mikael Ståldal < > [email protected]> wrote: > >> log4j-core has quite some Maven dependencies declared >> with <optional>true</optional>. E.g. Jackson. >> >> Is that really wise? I see a risk that users of Log4j is not aware if >> this and will get ClassNotFoundException:s at runtime. >> >> >> http://maven.apache.org/guides/introduction/introduction-to-optional-and-excludes-dependencies.html >> >> -- >> Mikael Ståldal >> Chief Software Architect >> *Appear* >> Phone: +46 8 545 91 572 >> Email: [email protected] >> > > > > -- > Matt Sicker <[email protected]> > -- Mikael Ståldal Chief Software Architect *Appear* Phone: +46 8 545 91 572 Email: [email protected]
