CMR, please.  You can say I reviewed it.

Thanks!


On Oct 29, 2008, at 9:21 AM, Matthias Jurenz wrote:

On Mi, 2008-10-29 at 08:43 -0400, Jeff Squyres wrote:

On Oct 29, 2008, at 8:14 AM, Matthias Jurenz wrote:

> The problem should be fixed in the trunk. VampirTrace also comes now
> with an own
> implementation of 'snprintf'. More precisely, the corresponding
> sources are based on
> 'opal/util/printf.<c|h>' and located in 'ompi/contrib/vt/vt/util/
> util.c'.
> Concerning this matter, I want to know, whether there is a copyright/
> license problem.
> Should our 'util.c' also contain your copyright headlines from
> 'printf.c', for instance?

IANAL, but I believe that that is correct.  Check the terms listed in
OMPI's LICENSE file.  Note that we haven't done the final copyright
update in the LICENSE file for v1.3 yet (it's on the to-do list...).

Speaking of which, what copyright line(s) should we use for U.
Dresden?  I see these in ompi/contrib/vt/vt/COPYING:

Copyright (c) 2005-2008, ZIH, TU Dresden, Federal Republic of Germany
Copyright (c) 1998-2005, Forschungszentrum Juelich GmbH, Federal
Republic of Germany

Are these right?

Yes, you can use these lines.

> BTW: We are rather sure, that we found a memory leak in the function
> 'opal_vsnprintf()'.
> It seems, that there is a 'free' missing at the and of this
> function. Can you confirm?

Yes, I believe that you are correct.  Thanks!

Do you guys want to add it to the file, or do you want me to?

I've fixed this in the trunk. Can you move this change to the 1.3- branch or should
I go the CMRs way?


--
Matthias Jurenz,
Center for Information Services and
High Performance Computing (ZIH), TU Dresden,
Willersbau A106, Zellescher Weg 12, 01062 Dresden
phone +49-351-463-31945, fax +49-351-463-37773
_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel


--
Jeff Squyres
Cisco Systems

Reply via email to