Should I revert?

David Jencks wrote
> If I had added the license header then svn alone would have included the 
> entire legal trail related to the file licensing.  Now, anyone concerned with 
> it has to also consult this mailing list to see that I emailed my agreement 
> to the license change.
> 
> david jencks
> 
>> On Jun 23, 2016, at 8:00 AM, Carsten Ziegeler <[email protected]> wrote:
>>
>> David Jencks wrote
>>> Um, I think you should have let me fix this as I’m the author?  In any case 
>>> I had a similar commit lined up to deliver and indeed intended the file to 
>>> be apache-2.0 licensed.
>>
>> Ehm, why should only you fix a problem? If, for whatever reason, these
>> files cant be apache licensed, you could still revert/remove.
>>
>> The build was failing because of this and I wanted to fix this asap.
>>
>> Thanks
>> Carsten
>>
>>>
>>> david jencks
>>>
>>>> On Jun 23, 2016, at 4:45 AM, [email protected] wrote:
>>>>
>>>> Author: cziegeler
>>>> Date: Thu Jun 23 11:45:33 2016
>>>> New Revision: 1749869
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1749869&view=rev
>>>> Log:
>>>> add missing license header
>>>>
>>>> Modified:
>>>>   
>>>> felix/trunk/scr/src/test/java/org/apache/felix/scr/integration/components/felix5248/Component.java
>>>>
>>>> Modified: 
>>>> felix/trunk/scr/src/test/java/org/apache/felix/scr/integration/components/felix5248/Component.java
>>>> URL: 
>>>> http://svn.apache.org/viewvc/felix/trunk/scr/src/test/java/org/apache/felix/scr/integration/components/felix5248/Component.java?rev=1749869&r1=1749868&r2=1749869&view=diff
>>>> ==============================================================================
>>>> --- 
>>>> felix/trunk/scr/src/test/java/org/apache/felix/scr/integration/components/felix5248/Component.java
>>>>  (original)
>>>> +++ 
>>>> felix/trunk/scr/src/test/java/org/apache/felix/scr/integration/components/felix5248/Component.java
>>>>  Thu Jun 23 11:45:33 2016
>>>> @@ -1,10 +1,28 @@
>>>> +/*
>>>> + * Licensed to the Apache Software Foundation (ASF) under one
>>>> + * or more contributor license agreements.  See the NOTICE file
>>>> + * distributed with this work for additional information
>>>> + * regarding copyright ownership.  The ASF licenses this file
>>>> + * to you under the Apache License, Version 2.0 (the
>>>> + * "License"); you may not use this file except in compliance
>>>> + * with the License.  You may obtain a copy of the License at
>>>> + *
>>>> + *  http://www.apache.org/licenses/LICENSE-2.0
>>>> + *
>>>> + * Unless required by applicable law or agreed to in writing,
>>>> + * software distributed under the License is distributed on an
>>>> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
>>>> + * KIND, either express or implied.  See the License for the
>>>> + * specific language governing permissions and limitations
>>>> + * under the License.
>>>> + */
>>>> package org.apache.felix.scr.integration.components.felix5248;
>>>>
>>>> import java.util.Map;
>>>>
>>>> public class Component
>>>> {
>>>> -    
>>>> +
>>>>    void activate(Map<String, Object> props)
>>>>    {
>>>>        if (props.containsKey( "FAIL" ))
>>>>
>>>>
>>>
>>>
>>
>>
>>
>> -- 
>> Carsten Ziegeler
>> Adobe Research Switzerland
>> [email protected] <mailto:[email protected]>
> 


 
-- 
Carsten Ziegeler
Adobe Research Switzerland
[email protected]

Reply via email to