[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [aspectj-dev] Creating a unique ajSandbox root directory


All,

Thanks for the suggestions. I have moved all the createEmptySandbox() methods to FileUtil and added a simple JUnit test. You should now get sandbox directories like "c:\temp\ajcSandbox\WORKSPACE\ajcTestNNNN.tmp". I have not tested on Linux but the code I added should be OS neutral.

Matthew Webster
AOSD Project
Java Technology Centre, MP146
IBM Hursley Park, Winchester,  SO21 2JN, England
Telephone: +44 196 2816139 (external) 246139 (internal)
Email: Matthew Webster/UK/IBM @ IBMGB, matthew_webster@xxxxxxxxxx

http://w3.hursley.ibm.com/~websterm/

Please respond to AspectJ developer discussions <aspectj-dev@xxxxxxxxxxx>

Sent by:        aspectj-dev-bounces@xxxxxxxxxxx

To:        "AspectJ developer discussions" <aspectj-dev@xxxxxxxxxxx>
cc:        
Subject:        Re: [aspectj-dev] Creating a unique ajSandbox root directory


My only suggested amendment might be just to use the workspace
directory name rather than the full path to it, just to keep names
under control, I have some very long paths to my workspaces.  So for
my:

f:\e32rc27\eclipse\workspaces\aspectjhead

it would be

c:\temp\aspectjhead\ajcSandboxXXXX

Just using that name would solve every case where I have seen this problem.

Andy.

On 14/06/06, Helen Hawkins <HAWKINSH@xxxxxxxxxx> wrote:
> There's also AjdeInteractionTestbed.createEmptySandbox() and
> AjdocTestCase.createEmptySandbox() which do the same as
> Ajc.createEmptySandbox(). These should probably be updated too if you
> change the Ajc one.
>
> Helen
>
>
>
>
>              Matthew
>              Webster/UK/IBM@IB
>              MGB                                                        To
>              Sent by:                  AspectJ developer discussions
>              aspectj-dev-bounc         <aspectj-dev@xxxxxxxxxxx>
>              es@xxxxxxxxxxx                                             cc
>
>                                                                    Subject
>              14/06/2006 14:11          [aspectj-dev] Creating a unique
>                                        ajSandbox root directory
>
>              Please respond to
>              AspectJ developer
>                 discussions
>              <aspectj-dev@ecli
>                  pse.org>
>
>
>
>
>
>
>
> There is currently a nasty gotcha if you run the compiler tests in two
> workspaces at the same time: the tests fail. This is because the
> "ajcTestNNNN.tmp" names can clash and the contents or the whole directory
> can be deleted by the other workspace (I noticed this while running a
> release build in the background which can take an hour). A simple solution
> would be to add the current workspace name (appropriately sanitized not use
> spaces or slashes etc) which would result in names like
> "c:\temp\ajSandox\c__workspaces_org.aspectj\ajcTest49469.tmp". I know the
> names are a little longer but this way you can have results left from
> running tests in 2 workspaces. If there are no objections I will make the
> necessary changes in Ajc.createEmptySandbox().
>
> Cheers
>
> Matthew Webster
> AOSD Project
> Java Technology Centre, MP146
> IBM Hursley Park, Winchester,  SO21 2JN, England
> Telephone: +44 196 2816139 (external) 246139 (internal)
> Email: Matthew Webster/UK/IBM @ IBMGB, matthew_webster@xxxxxxxxxx
> http://w3.hursley.ibm.com/~websterm/
> _______________________________________________
> aspectj-dev mailing list
> aspectj-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/aspectj-dev
>
>
> _______________________________________________
> aspectj-dev mailing list
> aspectj-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/aspectj-dev
>
_______________________________________________
aspectj-dev mailing list
aspectj-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/aspectj-dev