Paul, This is great. I love it!
I will take a look in more detail. Can you check the target in for the build.xml? Thanks, -Mark > -----Original Message----- > From: Paul Smith [mailto:[EMAIL PROTECTED] > Sent: Thursday, May 19, 2005 3:51 PM > To: Log4J Developers List > Subject: JDIff report > > I managed to get the log4j 1.2.9 tag, and build a jdiff report. Of > course I would first treat this output with suspicion since it is my > first jdiff attempt and I'm bound to have got something wrong. > Unfortunately it's over 1meg and the dev list is rejecting it. I > will instead send it to the log4j committers directly. > > I have also uploaded this to our site, but it is not "appearing" yet: > > http://logging.apache.org/log4j/docs/jdiff_report/changes.html > > I uploaded this to people.apache.org and transfered it into the > logging site area. I'm not sure if the directories are rsync'd > periodically to the real www now? Anyway, that url is 404. Any > ideas appreciated. > > Once we've verified that this report does smell right, I could go > ahead and _try_ to add it to gump > > Here's what I did to generate this: > > * Got fresh logging-log4j head > * got v1_2_9 tag logging-log4j > * using the following ant build file after downloading JDiff: > <project default="easyjdiff" basedir="."> > > <target name="easyjdiff" depends=""> > <!-- Must be set to the root of where JDiff is installed. --> > <property name="JDIFF_HOME" value="/Users/paulsmith/ > Downloads/jdiff"/> > <!-- Just one simple way to tell Ant about the JDiff task --> > <taskdef name="jdiff" classname="jdiff.JDiffAntTask" > classpath="${JDIFF_HOME}/lib/antjdiff.jar"/> > > <!-- Compare Version 1 of a project to Version 2 --> > <jdiff verbose="on"> > <old name="Version 1.2.9"> > <dirset dir="logging-log4j_1_2_9/src/java/" > includes="org/**"/> > </old> > <new name="Version 1.3"> > <dirset dir="logging-log4j/src/java/" includes="org/**"/> > </new> > </jdiff> > > </target> > > </project> > > I then had a look at the report, looked 'reasonable' and tar'd it up. > > Please, a sanity check on this would be helpful! > > cheers, > > Paul > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
