the link http://books.sonatype.com/maven-book/examples-1.0 is broken

Note from the Author or Editor:
Fixed the URL

I am reading this book using Safari online so I cannot provide a page number, so I provided the section heading.

"mvn install" for the simple-weather program failed. Compile failed "generics are not supported in -source 1.3". I read ahead in section 10 and found I needed to modify the pom.xml to add new source and target attributes for the compiler plugin. Problem corrected.


"mvn install" for the simple-weather program failed, org.apache.commons.io package was missing. A dependency was forgotten - the WeatherFormatterTest file depends on commons-io. I added it to the POM and "mvn install" worked.

Note from the Author or Editor:
The POM in Example 4.1 shows this source and target configuration for the compiler plugin, but we didn't mention that you need to add this configuration to the POM.

Right before Example 4.1 add the following sentence:

"The POM shown in Example 4.1 also contains configuration for the Maven Compiler plugin which sets the source and target Java versions to 1.5. This configuration is contained in the build element, and it necessary because the example project uses Java 5 language features."

It seems like there's either a problem with technical proofreading of this book or the Safari version is outdated. I've found 3 mistakes in less than 2 hours and I'm not even at the half of the book yet.

Line 17-19 in source code for Main class in the example reads

try {
zipcode = args[0]);
} catch( Exception e ) {}

where it should have been

try {
zipcode = args[0];
} catch( Exception e ) {}

There's an extra closing parens before semicolon.

Another one is the line 33 reads

InputStream dataIn = new YahooRetriever().retrieve( zip );

it should have been

InputStream dataIn = new YahooRetriever().retrieve( Integer.parseInt(zip) );

Note from the Author or Editor:
In example 4.6, change:

public InputStream retrieve(int zipcode) throws Exception {


to:

public InputStream retrieve(String zipcode) throws Exception {

first line reads

package org.sonatype.mavenbook.weather.yahoo;

where it should have been

package org.sonatype.mavenbook.weather;

Code does not compile

Note from the Author or Editor:
Yes, examples 4-11 and 4-12 should both begin with

package org.sonatype.mavenbook.weather;

first line reads

package org.sonatype.mavenbook.weather.yahoo;

where it should have been

package org.sonatype.mavenbook.weather;

Code does not compile

Note from the Author or Editor:
Examples 4-11 and 4-12 should begin with:

package org.sonatype.mavenbook.weather;

Example reads

<project>
[...]
<build>
<finalName>simple-webapp</finalName>
<plugins>
<plugin>
<groupId>org.mortbay.jetty</groupId>
<artifactId>maven-jetty-plugin</artifactId>
</plugins>
</plugins>
</build>
[...]
</project>

Where it should have been

<project>
[...]
<build>
<finalName>simple-webapp</finalName>
<plugins>
<plugin>
<groupId>org.mortbay.jetty</groupId>
<artifactId>maven-jetty-plugin</artifactId>
</plugin> <--- This should have been singular
</plugins>
</build>
[...]
</project>

Note from the Author or Editor:
Yes, this is confirmed. Please update the example as the reader has noted.

I think there is an error on page 66. Running the following command I received an error:
“Exception in thread "main" java.lang.NoClassDefFoundError: org/sonatype/mavenbook/weather/Main”.

The command in the book is: java –cp simple-weather-1.0-jar-with-dependencies.jar org.sonatype.mavenbook.weather.Main 10002

Running the following command works:
java -cp mavenbook-examples-1-SNAPSHOT-jar-with-dependencies.jar org.sonatype.mavenbook.weather.Main 10002

Note from the Author or Editor:
Right, the command line is consistent with the book but not consistent with the downloaded examples. I've updated the downloaded examples, there is no need to update the book.

Note to O'Reilly: this form needs to change; online versions of the book do not have page numbers.

Should "mvn echo:blah" be "mvn blah:echo"?

Note from the Author or Editor:
that's right it should be blah:echo instead of echo:blah