[Dependency] Exclude transitive spring dependencies from dubbo-all or dubbo-spring-boot-starter

chickenlj commented 3 months ago

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component

Java SDK (apache/dubbo)

Dubbo Version

3.3.0-beta.2

Steps to reproduce this issue

Remove the following transitive dependencies from dubbo-all, if not possible, at least exclude them in dubbo-spring-boot-starter.

  <!-- Transitive dependencies -->
    <dependency>
      <groupId>org.springframework</groupId>
      <artifactId>spring-core</artifactId>
    </dependency>
    <dependency>
      <groupId>org.springframework</groupId>
      <artifactId>spring-beans</artifactId>
    </dependency>
    <dependency>
      <groupId>org.springframework</groupId>
      <artifactId>spring-context</artifactId>
    </dependency>

Try to completely get rid of the following dependency:

    <dependency>
      <groupId>com.alibaba.spring</groupId>
      <artifactId>spring-context-support</artifactId>
    </dependency>

What you expected to happen

When using spring, the recommended way is spring boot, so transitive spring dependencies is not necessary, let spring-boot-starter do this for users.
When using API, we don't need spring dependencies.

Anything else

No response

Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

heliang666s commented 3 months ago

hello,I think I can do it,please assign me,thanks~

AlbumenJ commented 3 months ago

hello,I think I can do it,please assign me,thanks~

done

heliang666s commented 3 months ago

@AlbumenJ You seem to be getting people wrong

AlbumenJ commented 3 months ago

@AlbumenJ You seem to be getting people wrong

My fault😭

fastkdm commented 3 weeks ago

transitive dependencies exists 3.3.0-Beta.3

wcy666103 commented 3 weeks ago

transitive dependencies exists 3.3.0-Beta.3

Yes, it hasn't been merged yet, details are here https://github.com/apache/dubbo/pull/14137

apache / dubbo