Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0.0-incubating
    • Fix Version/s: 4.1.0-incubating
    • Component/s: None
    • Labels:
      None

      Description

      Current the build documentation states to build via:

      mvn clean package install -Prelease-all assembly:assembly -U
      

      Only the following is neccessary:

      mvn clean install -Prelease-all assembly:assembly -U
      

      otherwise several parts of the build life cycle are repeated...furthermore the questions is why you use a call to assembly:assembly which is deprecated and best it should be done simply by using:

      mvn clean deploy -Prelease-all
      

        Activity

        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 42f78c281cbeb5072b04eaf03b1a8059b8d281a7 in incubator-rocketmq's branch refs/heads/develop from yukon
        [ https://git-wip-us.apache.org/repos/asf?p=incubator-rocketmq.git;h=42f78c2 ]

        ROCKETMQ-168 Polish the BUILDING guide.

        Show
        jira-bot ASF subversion and git services added a comment - Commit 42f78c281cbeb5072b04eaf03b1a8059b8d281a7 in incubator-rocketmq's branch refs/heads/develop from yukon [ https://git-wip-us.apache.org/repos/asf?p=incubator-rocketmq.git;h=42f78c2 ] ROCKETMQ-168 Polish the BUILDING guide.
        Hide
        Yukon yukon added a comment -

        This issue has been resolved in `develop` branch, the quick start[1] guide should be adjusted when develop branch merged to master.

        [1]. http://rocketmq.apache.org/docs/quick-start/

        Show
        Yukon yukon added a comment - This issue has been resolved in `develop` branch, the quick start [1] guide should be adjusted when develop branch merged to master. [1] . http://rocketmq.apache.org/docs/quick-start/
        Hide
        lizhanhui Zhanhui Li added a comment -

        Current command does not generate a distribution package.

        $ mvn -Prelease-all -DskipTests clean install -U

        IMO, we need package phase included here.

        Show
        lizhanhui Zhanhui Li added a comment - Current command does not generate a distribution package. $ mvn -Prelease-all -DskipTests clean install -U IMO, we need package phase included here.
        Hide
        khmarbaise Karl Heinz Marbaise added a comment -

        The point is you need to add the supplemental call of maven-assembly-plugin goal call assembly:assembly apart from being deprecated and better being integrated into the build to use simply mvn -Prelease-all -DskipTests clean install and should be reduced to to mvn -Prelease-all clean install...

        Show
        khmarbaise Karl Heinz Marbaise added a comment - The point is you need to add the supplemental call of maven-assembly-plugin goal call assembly:assembly apart from being deprecated and better being integrated into the build to use simply mvn -Prelease-all -DskipTests clean install and should be reduced to to mvn -Prelease-all clean install ...
        Hide
        zander dongeforever added a comment -

        This issue has been resolved in `develop` branch, the quick start[1] guide should be adjusted when develop branch merged to master.
        [1]. http://rocketmq.apache.org/docs/quick-start/

        Show
        zander dongeforever added a comment - This issue has been resolved in `develop` branch, the quick start [1] guide should be adjusted when develop branch merged to master. [1] . http://rocketmq.apache.org/docs/quick-start/

          People

          • Assignee:
            Yukon yukon
            Reporter:
            khmarbaise Karl Heinz Marbaise
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development