just test it,
we define most classpathextensions with non absolute path...
make sure your construct works with absolute and non-absolute
marc
|-----Original Message-----
|From: [EMAIL PROTECTED]
|[mailto:[EMAIL PROTECTED]]On Behalf Of Roberto Leong
|Sent: Saturday, December 02, 2000 6:35 AM
|To: jBoss Developer
|Subject: Re: [jBoss-Dev] ClassPathExtension
|
|
|if no one says anything against making the fix within the next 10 minutes
|i�ll commit a change
|
|Roberto
|
|
|----- Original Message -----
|From: "Bordet, Simone" <[EMAIL PROTECTED]>
|To: "'jBoss Developer'" <[EMAIL PROTECTED]>
|Sent: Sunday, December 03, 2000 2:08 AM
|Subject: RE: [jBoss-Dev] ClassPathExtension
|
|
|Hey Rob,
|
|> guys
|>
|> i don�t know if i�m missing something but i�ve found this behavior:
|>
|> if try to add a ClassPathExtension pointing to a absolute path like
|> c:/java/jakarta-tomcat-3.2/lib/ the class will try to add all files at
|
|I noticed that if you prepend a / like this:
|/c:/java/jakarta-tomcat-3.2/lib/
|everything works fine.
|But maybe the fix you suggest is worth anyway.
|
|Simon
|
|>
|> D:\projects\jboss\dist\lib\ext\jboss.jar\C:\java\jakarta-tomca
|> t-3.2\lib
|>
|> a simple fix would be to replace
|>
|> dir = new
|> File(getClass().getProtectionDomain().getCodeSource().getLocat
|> ion().getFile(
|> ), url);
|>
|> with
|>
|> dir = new File(url);
|>
|> at more or less line 85
|>
|> I�m not doing it myself since i don�t know if i�m doing
|> something wrong or
|> the kind of implications this would have, anyone wants to
|> take a shot at it?
|>
|> regards
|> Roberto
|>
|>
|>
|>
|>
|>
|>
|>
|>
|>
|>
|>
|
|
|
|
