Hi Henry

I now see that the bundle really contains such a package, but this package
(org.apache.clerezza.rdf.scala.utill) allready exists in another bundle
(rdf.scala.utils), I don't think this is a good idea to have a package in
muliple bundles, as long as EasyGraph is an experiment just leave in a
package or subpackage of where it is used. Most bundles contains only
packages starting with <group-id>.<artifact-id>, I think this is a good
convention.

Cheers,
reto

On Sat, May 7, 2011 at 7:04 PM, Henry Story <[email protected]> wrote:

>
> On 7 May 2011, at 18:58, Reto Bachmann-Gmuer wrote:
>
> did you see this mail to the list?
>
>
> no Reto, thanks for sending it again. If you don't change the Subject of
> the thread to something readable it is very unlikely I'll notice. As you
> imagine I don't read all the commits to the list.
>
> Your mail was titled "*Fwd: svn commit: r1099471 -
> /incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml"*
> *
> *
>
> I guess you're using the wrong directive, but probably none should be
> needed. I suggest you explain on the list what goes wrong without this
> directive.
>
>
> Without the Private-Package directive it just did not compile the classes.
> Let me know if you have different behavior.
>
>
>
> cheers
> reto
>
> ---------- Forwarded message ----------
> From: Reto Bachmann-Gmuer <[email protected]>
> Date: Thu, May 5, 2011 at 8:58 PM
> Subject: Re: svn commit: r1099471 -
> /incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml
> To: [email protected]
>
>
> Hi Henry
>
> Never seen this kind of hack, why do you declare a package provided by
> another bundle as private?
>
> Cheers,
> Reto
>
>
> On Wed, May 4, 2011 at 4:51 PM, <[email protected]> wrote:
>
>> Author: bblfish
>> Date: Wed May  4 14:51:54 2011
>> New Revision: 1099471
>>
>> URL: http://svn.apache.org/viewvc?rev=1099471&view=rev
>> Log:
>> pom changes to get to compile correctly. The EasyGraph class had initially
>> been in the main rdf.scala.utils package, but I had decided last minute to
>> move it here, so that people could consider it more carefully first. But
>> that move requires a pom change
>>
>> Modified:
>>    incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml
>>
>> Modified:
>> incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml?rev=1099471&r1=1099470&r2=1099471&view=diff
>>
>> ==============================================================================
>> --- incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml
>> (original)
>> +++ incubator/clerezza/trunk/parent/platform.security.foafssl/test/pom.xml
>> Wed May  4 14:51:54 2011
>> @@ -117,7 +117,6 @@
>>                                        </execution>
>>                                </executions>
>>                                <configuration>
>> -
>> <scalaVersion>${scala.version}</scalaVersion>
>>                     <charset>UTF-8</charset>
>>                                        <!-- <args>
>>                                                <arg>-target:jvm-1.5</arg>
>> @@ -130,6 +129,7 @@
>>                                <configuration>
>>                                        <instructions>
>>
>>  <Service-Component>OSGI-INF/serviceComponents.xml</Service-Component>
>> +
>>  <Private-Package>org.apache.clerezza.rdf.scala.utils.*</Private-Package>
>>
>>  <Export-Package>org.apache.clerezza.foafssl.test.*</Export-Package>
>>
>>  
>> <Bundle-SymbolicName>${project.groupId}.${project.artifactId}</Bundle-SymbolicName>
>>                                        </instructions>
>>
>>
>>
>
>
> Social Web Architect
> http://bblfish.net/
>
>

Reply via email to