You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by mc...@apache.org on 2008/02/20 01:53:58 UTC
svn commit: r629290 - in
/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin:
ObrDeploy.java ObrDeployFile.java ObrInstall.java ObrInstallFile.java
ObrUpdate.java ResourcesBundle.java
Author: mcculls
Date: Tue Feb 19 16:53:52 2008
New Revision: 629290
URL: http://svn.apache.org/viewvc?rev=629290&view=rev
Log:
FELIX-498: use attached javadoc as default documentation (unless Bundle-DocURL or project URL is set)
Modified:
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeploy.java
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeployFile.java
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstall.java
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstallFile.java
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrUpdate.java
felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ResourcesBundle.java
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeploy.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeploy.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeploy.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeploy.java Tue Feb 19 16:53:52 2008
@@ -135,6 +135,11 @@
*/
private Artifact m_sourceArtifact;
+ /**
+ * Attached doc artifact
+ */
+ private Artifact m_docArtifact;
+
public void execute() throws MojoExecutionException
{
@@ -149,14 +154,17 @@
return;
}
- // check for any attached sources
+ // check for any attached sources or docs
for ( Iterator i = attachedArtifacts.iterator(); i.hasNext(); )
{
Artifact artifact = ( Artifact ) i.next();
if ( "sources".equals( artifact.getClassifier() ) )
{
m_sourceArtifact = artifact;
- break;
+ }
+ else if ( "javadoc".equals( artifact.getClassifier() ) )
+ {
+ m_docArtifact = artifact;
}
}
@@ -271,13 +279,19 @@
}
URI bundleJar = ObrUtils.getArtifactURI( localRepository, artifact );
- URI sourceJar = null;
+ URI sourceJar = null;
if ( null != m_sourceArtifact )
{
sourceJar = ObrUtils.getArtifactURI( localRepository, m_sourceArtifact );
}
- update.updateRepository( bundleJar, sourceJar );
+ URI docJar = null;
+ if ( null != m_docArtifact )
+ {
+ docJar = ObrUtils.getArtifactURI( localRepository, m_docArtifact );
+ }
+
+ update.updateRepository( bundleJar, sourceJar, docJar );
}
}
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeployFile.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeployFile.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeployFile.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrDeployFile.java Tue Feb 19 16:53:52 2008
@@ -194,7 +194,7 @@
update = new ObrUpdate( repositoryXml, obrXmlFile, project, mavenRepository, userConfig, log );
update.parseRepositoryXml();
- update.updateRepository( bundleJar, null );
+ update.updateRepository( bundleJar, null, null );
update.writeRepositoryXml();
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstall.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstall.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstall.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstall.java Tue Feb 19 16:53:52 2008
@@ -87,6 +87,11 @@
*/
private Artifact m_sourceArtifact;
+ /**
+ * Attached doc artifact
+ */
+ private Artifact m_docArtifact;
+
public void execute()
{
@@ -101,14 +106,17 @@
return;
}
- // check for any attached sources
+ // check for any attached sources or docs
for ( Iterator i = attachedArtifacts.iterator(); i.hasNext(); )
{
Artifact artifact = ( Artifact ) i.next();
if ( "sources".equals( artifact.getClassifier() ) )
{
m_sourceArtifact = artifact;
- break;
+ }
+ else if ( "javadoc".equals( artifact.getClassifier() ) )
+ {
+ m_docArtifact = artifact;
}
}
@@ -150,13 +158,19 @@
}
URI bundleJar = ObrUtils.getArtifactURI( localRepository, artifact );
- URI sourceJar = null;
+ URI sourceJar = null;
if ( null != m_sourceArtifact )
{
sourceJar = ObrUtils.getArtifactURI( localRepository, m_sourceArtifact );
}
- update.updateRepository( bundleJar, sourceJar );
+ URI docJar = null;
+ if ( null != m_docArtifact )
+ {
+ docJar = ObrUtils.getArtifactURI( localRepository, m_docArtifact );
+ }
+
+ update.updateRepository( bundleJar, sourceJar, docJar );
}
}
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstallFile.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstallFile.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstallFile.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrInstallFile.java Tue Feb 19 16:53:52 2008
@@ -103,7 +103,7 @@
update = new ObrUpdate( repositoryXml, obrXmlFile, project, mavenRepository, userConfig, log );
update.parseRepositoryXml();
- update.updateRepository( bundleJar, null );
+ update.updateRepository( bundleJar, null, null );
update.writeRepositoryXml();
}
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrUpdate.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrUpdate.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrUpdate.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ObrUpdate.java Tue Feb 19 16:53:52 2008
@@ -146,10 +146,11 @@
*
* @param bundleJar path to the bundle jar file
* @param sourceJar path to the source jar file
+ * @param docJar path to the docs jar file
*
* @throws MojoExecutionException if the plugin failed
*/
- public void updateRepository( URI bundleJar, URI sourceJar ) throws MojoExecutionException
+ public void updateRepository( URI bundleJar, URI sourceJar, URI docJar ) throws MojoExecutionException
{
m_logger.debug( " (f) repositoryXml = " + m_repositoryXml );
m_logger.debug( " (f) bundleJar = " + bundleJar );
@@ -229,20 +230,10 @@
throw new MojoExecutionException( "BindexException" );
}
- String sourcePath = null;
- if ( null != sourceJar )
- {
- if ( m_userConfig.isPathRelative() )
- {
- sourcePath = ObrUtils.getRelativeURI( m_baseURI, sourceJar ).toASCIIString();
- }
- else
- {
- sourcePath = sourceJar.toASCIIString();
- }
- }
+ String sourcePath = relativisePath( sourceJar );
+ String docPath = relativisePath( docJar );
- m_resourceBundle.construct( m_project, bindexExtractor, sourcePath );
+ m_resourceBundle.construct( m_project, bindexExtractor, sourcePath, docPath );
Element rootElement = m_repositoryDoc.getDocumentElement();
if ( !walkOnTree( rootElement ) )
@@ -251,6 +242,22 @@
String id = m_resourceBundle.getId();
searchRepository( rootElement, id );
}
+ }
+
+
+ private String relativisePath( URI uri )
+ {
+ if ( null != uri )
+ {
+ if ( m_userConfig.isPathRelative() )
+ {
+ return ObrUtils.getRelativeURI( m_baseURI, uri ).toASCIIString();
+ }
+
+ return uri.toASCIIString();
+ }
+
+ return null;
}
Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ResourcesBundle.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ResourcesBundle.java?rev=629290&r1=629289&r2=629290&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ResourcesBundle.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/obrplugin/ResourcesBundle.java Tue Feb 19 16:53:52 2008
@@ -382,9 +382,10 @@
* @param project project information given by maven
* @param ebi bundle information extracted from bindex
* @param sourcePath path to local sources
+ * @param docPath path to local docs
* @return true
*/
- public boolean construct( MavenProject project, ExtractBindexInfo ebi, String sourcePath )
+ public boolean construct( MavenProject project, ExtractBindexInfo ebi, String sourcePath, String docPath )
{
if ( ebi.getPresentationName() != null )
@@ -444,25 +445,32 @@
setDescription( project.getDescription() );
}
+ // fallback to javadoc if no project URL
+ String documentation = project.getUrl();
+ if ( null == documentation )
+ {
+ documentation = docPath;
+ }
+
if ( ebi.getDocumentation() != null )
{
setDocumentation( ebi.getDocumentation() );
- if ( project.getUrl() != null )
+ if ( documentation != null )
{
- m_logger.debug( "pom property override:<documentation> " + project.getUrl() );
+ m_logger.debug( "pom property override:<documentation> " + documentation );
}
}
else
{
- setDocumentation( project.getUrl() );
+ setDocumentation( documentation );
}
if ( ebi.getSource() != null )
{
setSource( ebi.getSource() );
- if ( project.getScm() != null )
+ if ( sourcePath != null )
{
- m_logger.debug( "pom property override:<source> " + project.getScm() );
+ m_logger.debug( "pom property override:<source> " + sourcePath );
}
}
else