You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by Patrick Linskey <pl...@bea.com> on 2007/04/11 01:11:46 UTC

RE: svn commit: r527320 [1/2] - in /incubator/openjpa/branches/0.9.7-incubating: ./ openjpa-all/ openjpa-examples/ openjpa-integration/ openjpa-integration/examples/ openjpa-integration/tck/ openjpa-jdbc-5/ openjpa-jdbc/ openjpa-kernel-5/ openjpa-kernel/

Speaking of licenses, I just went through trunk to look for files that
don't contain the URL of the license file. I got the following:

./openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/H2Dictionary.ja
va
./openjpa-jdbc/src/main/resources/META-INF/services/org.apache.openjpa.l
ib.conf.ProductDerivation
./openjpa-jdbc/src/main/resources/org/apache/openjpa/jdbc/meta/java-keyw
ords.rsrc
./openjpa-jdbc/src/main/resources/org/apache/openjpa/jdbc/schema/schemas
-doctype.rsrc
./openjpa-jdbc/src/main/resources/org/apache/openjpa/jdbc/sql/sql-keywor
ds.rsrc
./openjpa-kernel/src/main/java/org/apache/openjpa/conf/BrokerValue.java
./openjpa-kernel/src/main/java/org/apache/openjpa/enhance/Reflection.jav
a
./openjpa-kernel/src/main/java/org/apache/openjpa/kernel/FinalizingBroke
rImpl.java
./openjpa-kernel/src/main/resources/META-INF/services/org.apache.openjpa
.kernel.exps.ExpressionParser
./openjpa-persistence/src/main/resources/META-INF/services/javax.persist
ence.spi.PersistenceProvider
./openjpa-persistence/src/main/resources/META-INF/services/org.apache.op
enjpa.lib.conf.ProductDerivation
./openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/
orm-xsd.rsrc
./openjpa-persistence/src/main/resources/org/apache/openjpa/persistence/
persistence-xsd.rsrc
./openjpa-persistence-jdbc/src/main/resources/META-INF/services/org.apac
he.openjpa.lib.conf.ProductDerivation

Some of these files are formats / get parsed in ways that do not allow
comments, but certainly all of the Java files should have licenses in
them. I would just add the license headers in myself, except that then
you'd need to patch your branch; can you add the headers to the files in
this list that support comments?

There were also 22 files in openjpa-*/src/test directories that don't
have the headers. I'm assuming that when we package up the source, we
don't include the test classes, and am therefore leaving those files
out.

Thanks,

-Patrick

-- 
Patrick Linskey
BEA Systems, Inc. 

_______________________________________________________________________
Notice:  This email message, together with any attachments, may contain
information  of  BEA Systems,  Inc.,  its subsidiaries  and  affiliated
entities,  that may be confidential,  proprietary,  copyrighted  and/or
legally privileged, and is intended solely for the use of the individual
or entity named in this message. If you are not the intended recipient,
and have received this message in error, please immediately return this
by email and then delete it. 

> -----Original Message-----
> From: Craig.Russell@Sun.COM [mailto:Craig.Russell@Sun.COM] 
> Sent: Tuesday, April 10, 2007 3:26 PM
> To: open-jpa-dev@incubator.apache.org
> Cc: open-jpa-commits@incubator.apache.org
> Subject: Re: svn commit: r527320 [1/2] - in 
> /incubator/openjpa/branches/0.9.7-incubating: ./ openjpa-all/ 
> openjpa-examples/ openjpa-integration/ 
> openjpa-integration/examples/ openjpa-integration/tck/ 
> openjpa-jdbc-5/ openjpa-jdbc/ openjpa-kernel-5/ openjpa-kernel/ 
> 
> Hi Mike,
> 
> Did you accidentally remove the licenses from the xml files???
> 
> Craig
> 
> On Apr 10, 2007, at 2:59 PM, mikedd@apache.org wrote:
> 
> > Modified: incubator/openjpa/branches/0.9.7-incubating/openjpa- 
> > examples/pom.xml
> > URL: http://svn.apache.org/viewvc/incubator/openjpa/branches/0.9.7- 
> > incubating/openjpa-examples/pom.xml? 
> > view=diff&rev=527320&r1=527319&r2=527320
> > 
> ==============================================================
> ======== 
> > ========
> > --- incubator/openjpa/branches/0.9.7-incubating/openjpa-examples/ 
> > pom.xml (original)
> > +++ incubator/openjpa/branches/0.9.7-incubating/openjpa-examples/ 
> > pom.xml Tue Apr 10 14:59:02 2007
> > @@ -1,22 +1,4 @@
> > -<?xml version="1.0" encoding="UTF-8"?>
> > -<!--
> > - Copyright 2006 The Apache Software Foundation.
> > -
> > - Licensed 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.
> > --->
> > -<project xmlns="http://maven.apache.org/POM/4.0.0"
> > -         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> > -         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0  
> > http://maven.apache.org/maven-v4_0_0.xsd">
> > +<project xmlns="http://maven.apache.org/POM/4.0.0"  
> > xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"  
> > xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http:// 
> > maven.apache.org/maven-v4_0_0.xsd">
> >      <modelVersion>4.0.0</modelVersion>
> >      <groupId>org.apache.openjpa</groupId>
> >      <artifactId>openjpa-examples</artifactId>
> > @@ -27,7 +9,7 @@
> >      <parent>
> >          <groupId>org.apache.openjpa</groupId>
> >          <artifactId>openjpa</artifactId>
> > -        <version>0.9.7-incubating-SNAPSHOT</version>
> > +        <version>0.9.7-incubating</version>
> 
> Craig Russell
> Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
> 408 276-5638 mailto:Craig.Russell@sun.com
> P.S. A good JDO? O, Gasp!
> 
> 

Notice:  This email message, together with any attachments, may contain information  of  BEA Systems,  Inc.,  its subsidiaries  and  affiliated entities,  that may be confidential,  proprietary,  copyrighted  and/or legally privileged, and is intended solely for the use of the individual or entity named in this message. If you are not the intended recipient, and have received this message in error, please immediately return this by email and then delete it.