Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a foss-boot-parent based on Spring Boot #86

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mfriedenhagen
Copy link
Contributor

Spring Boot makes it easy to create stand-alone, production-grade Spring based Applications that you can "just run". We take an opinionated view of the Spring platform and third-party libraries so you can get started with minimum fuss. Most Spring Boot applications need very little Spring configuration. (Excerpt from Spring Boot Website )

Spring Boot is great to create µServices and other applications, but to use all its features it requires that you use the spring-boot-starter-parent as parent pom. Since maven only allows single inheritance it would be useful to have a version of the foss-parent, e.g. foss-boot-parent that has spring-boot-starter-parent as parent.

    <parent>
        <groupId>org.springframework.boot</groupId>
        <artifactId>spring-boot-starter-parent</artifactId>
        <version>1.2.5.RELEASE</version>
    </parent>

@mfriedenhagen
Copy link
Contributor

@leonard84, I just tried out what you have to do for using foss-parent together with spring-boot, see https://github.com/mfriedenhagen/spring-boot-sample/blob/master/pom.xml.

  • Include spring-boot-dependencies in dependencyManagement.
  • Depend on spring-boot-starter-web (line 50).
  • Call repackage of spring-boot-maven-plugin (line 93) and manage scope of logback-classic and logback-core to runtime if you want an executable JAR.

IMO having a deep hierarchy of POMs is always a bit confusing, so I am not really sure whether this issue is an enhancement 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants