search

AArhin / simple-spring-memcached

Automatically exported from code.google.com/p/simple-spring-memcached
MIT License
0 stars 0 forks source link

aspectjrt jar conflict #31

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Current Spring reference states that [1] [2]:

> At a minimum you will need the following libraries to use the Spring 
Framework's support for AspectJ LTW:
>spring-aop.jar (version 2.5 or later, plus all mandatory dependencies)
>aspectjweaver.jar (version 1.6.8 or later)

I have noticed that some poms have aspectjrt dependency [1].

               <dependency>
                        <groupId>org.aspectj</groupId>
                        <artifactId>aspectjrt</artifactId>
                </dependency>
                <dependency>
                        <groupId>org.aspectj</groupId>
                        <artifactId>aspectjweaver</artifactId>
                </dependency>

Is it possible to remove aspectjrt dependency? 

[1]: 
http://docs.spring.io/spring/docs/3.2.6.RELEASE/spring-framework-reference/htmls
ingle/#aop
[2]: https://jira.springsource.org/browse/SPR-8896
[3]: 
https://code.google.com/p/simple-spring-memcached/source/browse/trunk/simple-spr
ing-memcached/pom.xml

Original issue reported on code.google.com by cemalettin.koc@gmail.com on 19 Feb 2014 at 12:58

GoogleCodeExporter commented 9 years ago 
Why do you want to remove this dependency? 
SSM uses aspectj to handle all SSM annotations. Please take a look at 
https://code.google.com/p/simple-spring-memcached/source/browse/trunk/simple-spr
ing-memcached/src/main/java/com/google/code/ssm/aop/ReadThroughSingleCacheAdvice
.java and other *CacheAdvice classes in SSM.

Original comment by ragno...@gmail.com on 19 Feb 2014 at 1:22

GoogleCodeExporter commented 9 years ago 
I've got it, you want to get rid of dependency to aspectjrt because 
aspectjweaver contains also all classes from aspectjrt.jar, right?

Original comment by ragno...@gmail.com on 19 Feb 2014 at 1:26

GoogleCodeExporter commented 9 years ago 
Yes, It is duplicating classes. 

Sorry for lack of information. I thought I had also written duplication.  

Thanks

Original comment by cemalettin.koc@gmail.com on 19 Feb 2014 at 1:48

GoogleCodeExporter commented 9 years ago 
Done. Fix is on trunk.

Original comment by ragno...@gmail.com on 19 Feb 2014 at 6:52

GoogleCodeExporter commented 9 years ago 
This issue has been moved to github: 
https://github.com/ragnor/simple-spring-memcached/issues/31

Original comment by ragno...@gmail.com on 24 Feb 2014 at 5:51

GoogleCodeExporter commented 9 years ago

Original comment by ragno...@gmail.com on 1 Apr 2014 at 7:27