Fwd: Re: FYI [icedtea-web] xml output for junit, for daily report - no styles

Dr Andrew John Hughes ahughes at redhat.com
Tue May 17 07:08:16 PDT 2011


On 15:42 Tue 17 May     , Jiri Vanek wrote:

snip...

> >> diff -r 0e6b12424423 Makefile.am
> >> --- a/Makefile.am	Wed May 11 14:56:32 2011 +0100
> >> +++ b/Makefile.am	Thu May 12 14:13:40 2011 +0200
> >> @@ -472,7 +472,10 @@
> >>   	done ; \
> >>   	echo $$class_names ; \
> >>   	CLASSPATH=$(NETX_DIR)/lib/classes.jar:$(JUNIT_JAR):$(JUNIT_RUNNER_JAR):. \
> >> -	  $(BOOT_DIR)/bin/java -Xbootclasspath:$(RUNTIME) CommandLine $$class_names
> >> +	  $(BOOT_DIR)/bin/java -Xbootclasspath:$(RUNTIME) CommandLine $$class_names \
> >> +	>  stdout.log 2>  stderr.log ; \
> >> +	cat stdout.log ; \
> >> +	cat stderr.log>&2
> >>
> >
> > Is there any reason you want these separate and don't just use 2>&1 | tee log ?
> 
> yap. I want to keep each stream where it belongs.
> I tried something like  > >(tee stdout.log) 2> >(tee stderr.log >&2) m which work fine in shell, but  I was unable to make it work in make :(

Ok.

snip...

> >
> > I assume this produces the correct output.  What confuses me is why you didn't use the standard XML APIs
> > rather than rolling your own?
> 
> After some rather nasty experience with standart java xml api/tools, I fell in love to love Apache axiom for everyhing worthy. And for every simple output much more rather to keep the class stright and without terrible standart api.
> If you mind, I will rewrite it. But I would rather to keep it straight-forward.
> 

Well, as it's written, you may as well commit it as is.  You've already done the extra work that using
the API (creating a DOM tree and outputting it) would have saved.

> Regards J.
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37



More information about the distro-pkg-dev mailing list