[Design the new POJO MicroContainer] - Reflection and MetaDataRepository refactoring
by adrian@jboss.org
The devil is always in the detail ;-)
I've found a couple of other wrong dependencies while splitting out this code.
The first is the ArrayInfo using the Signature class from the metadata repository
to determine primitive array class information.
This should clearly be delegating to PrimitiveInfo in both cases.
The second is this horrible method in BeanInfo
| /**
| * Bean may have additional dependencies
| * that the kernel cannot initially resolve. (currently defined by ClassAdapter)
| *
| * @param metaData the metadata
| * @return the list of dependencies
| */
| List<DependencyBuilderListItem<?>> getDependencies(MetaData metaData);
|
This has always bothered me since we first started the aop integration.
It doesn't really belong there. It's really part of the Kernel api.
The original intention isn't even used (instance ClassAdapters for AOP
instance containers).
And the latest rev has some horrible opaque type which is seen here in
the <?> which is actually the KernelControllerContext.
The issue came up again since it causes jboss-reflection to be based
on the metadata repository for no logical reason.
I'm finally going to bite the bullet and move this api to the kernel project
and remove all reference to the DependencyBuilder from the reflection project.
Like I said only aop-mc-int implements it and then only from the MC Pojo lifecycle.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4133822#4133822
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4133822
18 years, 1 month
[Design of JBoss Build System] - Problem running tests in microcontainer/aop-mc-int
by adrian@jboss.org
I can't get the tests to run in aop-mc-int.
Output using mvn -X test -Ptests-no-weave
| [DEBUG] Artifact not found - using stub model: Unable to determine the latest version
|
| org.apache.maven.plugins:maven-surefire-report-plugin:pom:LATEST
|
|
| [DEBUG] Using defaults for missing POM org.apache.maven.plugins:maven-surefire-report-plugin:pom:LATEST
| [DEBUG] maven-surefire-report-plugin: using locally installed snapshot
| [DEBUG] Artifact not found - using stub model: Unable to determine the release version
|
| org.apache.maven.plugins:maven-surefire-report-plugin:pom:RELEASE
|
|
| [DEBUG] Using defaults for missing POM org.apache.maven.plugins:maven-surefire-report-plugin:pom:RELEASE
| [INFO] ------------------------------------------------------------------------
| [ERROR] BUILD ERROR
| [INFO] ------------------------------------------------------------------------
| [INFO] The plugin 'org.apache.maven.plugins:maven-surefire-report-plugin' does not exist or no valid version could be found
| [INFO] ------------------------------------------------------------------------
| [DEBUG] Trace
| org.apache.maven.lifecycle.LifecycleExecutionException: The plugin 'org.apache.maven.plugins:maven-surefire-report-plugin' does not exist or no valid version could be found
|
I've got this workaround which is to disable the reference to the plugin
| Index: pom.xml
| ===================================================================
| --- pom.xml (revision 70351)
| +++ pom.xml (working copy)
| @@ -396,13 +396,13 @@
| </execution>
| </executions>
| </plugin>
| - <plugin>
| + <!--plugin>
| <artifactId>maven-surefire-report-plugin</artifactId>
| <configuration>
| <reportsDirectory>${project.build.directory}/surefire-reports/no-weave</reportsDirectory>
| <outputName>surefire-report-no-weave</outputName>
| </configuration>
| - </plugin>
| + </plugin-->
| </plugins>
| </build>
| </profile>
| @@ -665,4 +665,4 @@
| </build>
| </profile>
| </profiles>
| -</project>
| \ No newline at end of file
| +</project>
|
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4133819#4133819
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4133819
18 years, 1 month