Krysalis-Version Resolving (was RE: First comments from a newbee)

Francois wrote:

>	    - "VersionTool" -c says that it don't find the version of
>	org.apache.commons.logging with the jar file in classpath and version
>	information in the manifest file.

Hmm. Yes. Hmm.

Ok, here is some background:



>	The source file "Resolver.java" seems strange :
>	=================================
>	 public VersionInformation resolve(
>	  AnnotationScratchpad pad,
>	  VersionIdentifier identifier)
>	  throws VersionException {
>
>	  VersionInformation candidateInfo = null;
>
>	  VersionLoadContext loadContext = m_stampLoader.load(pad, identifier);
>
>	  if (null == loadContext)
>	     m_packageLoader.load(pad, identifier);
>	=================================
>	    last line seems to be better with : "loadContext =
m_packageLoader.load(pad, identifier);"

Actually it is saying, if I didn't get a version stamp, then try for a
package. That said, I think I need a "if not stamp or package looke in the
registry ... 'cos that is where the brute force load was". Let me look into
this more, this area has changed a lot in the last few weeks.

>	Well, nothing change, because package information in PackageVersionLoader
is loaded trought
>	Package.getPackage() which return "null" if the package was not already in
use by one classe.
>	(I wonder if  I have not see something about this problem somewhere in the
Krysalis site...).

Yeah, annoying. More noodling required, thanks.

regards

Adam



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01