Re: [UNIKRAFT PATCH 7/8] Correct license headers introduced by UPB

[Razvan Deaconescu <razvan.deaconescu@xxxxxxxxx>]

Yup, I already discussed with Bogdan Lascu. All is OK.

Razvan

Simon Kuenzer <simon.kuenzer@xxxxxxxxx> writes:
> Hi Razvan,
>
> as discussed offline, I will add a note to your `reviewed-by` tag that
> says that you did it `on behalf of` Bogdan Lascu who gave the
> agreement to you.
>
> Thanks,
>
> Simon
>
> On 10.03.21 21:08, Razvan Deaconescu wrote:
>> This is OK from UPB's side.
>>
>> Reviewed-by: Razvan Deaconescu <razvan.deaconescu@xxxxxxxxx>
>>
>> Razvan
>>
>> Simon Kuenzer <simon.kuenzer@xxxxxxxxx> writes:
>>> This commit removes the additional clause
>>> "THIS HEADER MAY NOT BE EXTRACTED OR MODIFIED IN ANY WAY"
>>> from BSD license headers that got introduced by mistake
>>> by Bogdan Lascu with the following commit:
>>>   32ad87545 - lib/vfscore: Add anonymous pipe
>>>
>>> On the one hand, this additional clause is redundant because
>>> the BSD license already states that it must remain and the
>>> copyright notice must be kept.
>>> On the other hand, the clause freezes the file header
>>> and prohibits future contributors from extending the
>>> copyright notice for their contributions. This additional
>>> clause is not part of the official BSD 3-clause.
>>>
>>> The original author consents to the change with a `Signed-off-by`
>>> or `Reviewed-by` tag to this commit.
>>>
>>> Signed-off-by: Simon Kuenzer <simon.kuenzer@xxxxxxxxx>
>>> ---
>>>   lib/vfscore/pipe.c | 2 --
>>>   1 file changed, 2 deletions(-)
>>>
>>> diff --git a/lib/vfscore/pipe.c b/lib/vfscore/pipe.c
>>> index 8d382965e..09597dbcb 100644
>>> --- a/lib/vfscore/pipe.c
>>> +++ b/lib/vfscore/pipe.c
>>> @@ -29,8 +29,6 @@
>>>    * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>>>    * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF 
>>> THE
>>>    * POSSIBILITY OF SUCH DAMAGE.
>>> - *
>>> - * THIS HEADER MAY NOT BE EXTRACTED OR MODIFIED IN ANY WAY.
>>>    */
>>>     #include <uk/config.h>