..and from testing it's clear that commons-logging should have it's scope changed from <scope>testing</scope> to <scope>runtime</scope>
I guess, more accurately, it should really be a transitive dependency from something, right? Some dependency being used must need commons-logging. Oh well, for now, setting to runtime in the parser POM is the best option. -Stephen On 6/17/06, Stephen Duncan <[EMAIL PROTECTED]> wrote:
Yeah, I set that correctly in the patch. I was just wondering if commons-logging is required at runtime, or if it's only required by something in the tests at runtime... -Stephen On 6/17/06, James M Snell <[EMAIL PROTECTED]> wrote: > Woodstox is required at runtime to provide the stax implementation. > (actually, it could be any stax implementation, but woodstox is what > I've been using to this point). > > - James > > Stephen Duncan wrote: > > Cool. How about adding the the wstx-asl dependency? I've attached a > > patch (though I could use some quick help on how to better generate > > one, as Subclipse is putting my local paths into the file...) > > > > Also, I want to verify that the scope of "test" is correct. Is that > > only needed by something test specific, or is a dependency users of > > the library will need at runtime? If so, the scope should be changed > > to runtime... > > > > -Stephen > > > > On 6/16/06, James M Snell <[EMAIL PROTECTED]> wrote: > >> Fixed! > >> > >> Stephen Duncan wrote: > >> > Ok, nevermind. I know why the NullPointerException's are happening > >> > when building clean from Maven. The xml files in src/test/java should > >> > be moved to src/test/resources. > >> > > >> > > > > > -- Stephen Duncan Jr www.stephenduncanjr.com
-- Stephen Duncan Jr www.stephenduncanjr.com
