Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 29 Jun 2012 08:21:09 -0700
From:      "David O'Brien" <obrien@FreeBSD.org>
To:        "Pedro F. Giffuni" <pfg@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r237624 - in head: cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/llquantize cddl/contrib/opensolaris/lib/libdtrace/common sys/cddl/contrib/opensolaris/uts/common/dtrace sys/cddl/c...
Message-ID:  <20120629152109.GA70522@dragon.NUXI.org>
In-Reply-To: <201206270439.q5R4dU0n081732@svn.freebsd.org>
References:  <201206270439.q5R4dU0n081732@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 27, 2012 at 04:39:30AM +0000, Pedro F. Giffuni wrote:
> Log:
>   Bring llquantize support into Dtrace.
>   Bryan Cantrill implemented the equivalent of semi-log graph
>   paper for Dtrace so llquantize will use one logarithmic and
>   one linear scale.
...
>   Obtained from:	Illumos
...
> Added: head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/llquantize/err.D_LLQUANT_FACTOREVEN.nodivide.d
> ==============================================================================
> --- /dev/null	00:00:00 1970	(empty, because file is newly added)
> +++ head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/llquantize/err.D_LLQUANT_FACTOREVEN.nodivide.d	Wed Jun 27 04:39:30 2012	(r237624)
> @@ -0,0 +1,29 @@
> +/*
> + * CDDL HEADER START
> + *
> + * The contents of this file are subject to the terms of the
> + * Common Development and Distribution License (the "License").
> + * You may not use this file except in compliance with the License.
...
> +/*
> + * Copyright (c) 2011, Joyent, Inc. All rights reserved.
> + */

Pedro,
It looks like you simply 'svn add'ed this file, rather than importing it
into '^/vendor/opensolaris/dist'.  Please correct me if I am wrong.

We've become rather sloppy in the past 1-2 years in our handling of
3rd-party externally developed code.  Something we need to return to
our older ways good ways of handling.

This is something that is especially important with GPL, CDDL, and
similar licenses.

Commercial entities that base products on FreeBSD have to worry about
code provenance for various intellectual property legal reasons and we
muddy the waters when we make it harder to track who and where code came
from.
[Please see http://www.osgeo.org/incubator/process/codereview.html &
http://wiki.osgeo.org/wiki/Code_Provenance_Review to better understand
this issue.]

Not following our established procedure also means that the average
developer(committer) and commercial consumer will have their expectations
fail.  One expects to be able to find the stock vendor sources in
ssh://svn.freebsd.org/base/vendor/ and to be able to find FreeBSD local
changes to the sources thru 'svn diff' against 'ssh://svn.freebsd.org/base/head'.

Please correct this by properly importing the sources, 'svn delete'ing
the files you 'svn add'ed, and do an 'svn copy' of the new files.

thanks,
-- 
-- David  (obrien@FreeBSD.org)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120629152109.GA70522>