[jira] Created: (LABS-410) [eclipse] Linked source folder for project which are a dependency

[eclipse] Linked source folder for project which are a dependency
-----------------------------------------------------------------

                 Key: LABS-410
                 URL: https://issues.apache.org/jira/browse/LABS-410
             Project: Labs
          Issue Type: Improvement
          Components: Magma
    Affects Versions: Current
            Reporter: Simone Gianni
             Fix For: Current


The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.

When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.

AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.

So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

    [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12743842#action_12743842 ]

Simone Gianni commented on LABS-410:
------------------------------------

Adding a project present on the Maven classpath to both inpath and aspectpath for AJDT triggers an AJDT bug, which could be worth investigating.

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Current
>            Reporter: Simone Gianni
>             Fix For: Current
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

    [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12743844#action_12743844 ]

Simone Gianni commented on LABS-410:
------------------------------------

This could bring another advantage, that is not requiring to clean install dependencies separately when magma:run is invoked for a web project. In fact, directing output of project linked source folders to a specific output folder, and including it in the magma:run classpath, can ease the pain of rebuilding them.

Since there could be exist a problem with duplicate resources however, this solution has to be validated against generated magma.locals and possible enhancements on that part before.

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Current
>            Reporter: Simone Gianni
>             Fix For: Current
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

    [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12743924#action_12743924 ]

Simone Gianni commented on LABS-410:
------------------------------------

Basic experiments on having the source folder of another project linked is giving less than optimal results :
- Error markers are placed on the linked source folder on the other project instead that on the main project
- Despite being derived, results from the linked folder still appears in searches
- F3 from project opens linked directory files instead of java file in dependency project

Don't know if this behaviour can be fixed somehow, or if adding dependencies to both inpath and aspectpath can be a solution.

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Current
>            Reporter: Simone Gianni
>             Fix For: Current
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

    [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12748429#action_12748429 ]

Simone Gianni commented on LABS-410:
------------------------------------

As a side note, if the project is a dependency that is also installed, Magma should notice that the project is in the workspace and NOT create a magmalnk folder to the unzipped sources.

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Current
>            Reporter: Simone Gianni
>             Fix For: Current
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

    [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12762484#action_12762484 ]

Simone Gianni commented on LABS-410:
------------------------------------

Thanks to recent fixes in AJDT it's now possible to add stuff to both inpath and aspectpath.

The trick is to add the entire maven container to both paths, and the restrict it using names of dependencies we want included (in our case, dependency projects).

This works from a binary POV. AJDT does not take into account that it also has the source code for those projects to build its model, so the model is limited, giving a suboptimal editor feedback.

As it is now, having an aspect in the main project weaving a dependency class is correctly reported (cause the source is considered), while having an aspect in a dependency weaving a class in the main project is not correctly reported (cause the source is NOT considered).

This is however the best solution actually available, cause it still compiles properly (also when running junit tests), does not give any Eclipse sideffects, and leaves to Eclipse and AJDT the task of properly coordinating and optimizing the build cycle between interdependant projects, thus reducing cpu cycles and memory consumption.

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Next
>            Reporter: Simone Gianni
>             Fix For: Next
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...

     [ https://issues.apache.org/jira/browse/LABS-410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Simone Gianni resolved LABS-410.
--------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: Next)
                   Current
         Assignee: Simone Gianni

> [eclipse] Linked source folder for project which are a dependency
> -----------------------------------------------------------------
>
>                 Key: LABS-410
>                 URL: https://issues.apache.org/jira/browse/LABS-410
>             Project: Labs
>          Issue Type: Improvement
>          Components: Magma
>    Affects Versions: Next
>            Reporter: Simone Gianni
>            Assignee: Simone Gianni
>             Fix For: Current
>
>
> The Magma Eclipse plugin currently creates linked source folders for sources of Magma dependencies. This gives AJDT the ability to properly resolve all Magma aspects, ITDs, and also to weave in user's own aspects.
> When a project in the workspace is added as a dependency, however, only the binary output folder of the project is exposed to AJDT, giving access only to stuff already weaved there, but not using aspects in that project or giving the ability to weave into it.
> AJDT only offers ways to add the project to the inpath or to the aspectpath, as opposed to a LTW-like environment.
> So, we need linked sources also for those projects.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: labs-unsubscribe@...
For additional commands, e-mail: labs-help@...