[
https://issues.apache.org/jira/browse/QPID-8361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16931444#comment-16931444
]
ASF GitHub Bot commented on QPID-8361:
--------------------------------------
alex-rufous commented on pull request #36: QPID-8361: [Broker-J] Create a
developer guide for Qpid Broker-J
URL: https://github.com/apache/qpid-broker-j/pull/36#discussion_r325161364
##########
File path: doc/developer-guide/src/main/markdown/build-instructions.md
##########
@@ -0,0 +1,290 @@
+# Build Instructions
+
+[Quick Start Guide](quick-start.md) only gives a minimum set of instructions
to start quickly building and developing
+Qpid Broker-J code. The materials provided here intend to give a dipper
understanding of various options for building,
+and testing Qpid Broker-J.
+
+## Prerequisites
+
+### Sources
+
+The Qpid project employs [Git distributed version-control
system](https://git-scm.com) for tracking changes in source code.
+
+The Git repository can be found at
<https://gitbox.apache.org/repos/asf/qpid-broker-j.git>.
+
+The mirror of Git repository exists on GitHub at
<https://github.com/apache/qpid-broker-j>.
+
+Changes need to be committed into Apache Git repository. They immediately get
propagated into GitHub mirror.
+Only members of Qpid project with commit rights can commit the changes. The
contributors without commit rights need
+to raise [Pull
Request](https://help.github.com/en/articles/creating-a-pull-request) on GitHub
in order to have
+their changes be committed by Qpid project committers.
+
+Git client is required to checkout sources from Git repo
+
+ git clone https://gitbox.apache.org/repos/asf/qpid-broker-j.git
qpid-broker-j
+
+For complete reference and documentation on Git please check [Git
Book](https://git-scm.com/book/en/v2)
+
+### Maven
+
+Qpid Broker-J project uses [Maven](http://maven.apache.org/) as its build and
management tool. Maven version 3 or above
+is required for building the project.
+
+You should set the `M2_HOME` environment variable and include its bin
directory in your `PATH`.
+
+Check that maven is installed on your system by executing the following at
your command prompt.
+
+ mvn --version
+
+For full maven install instructions visit [Maven Installation
Instructions](http://maven.apache.org/download.cgi#Installation).
+
+For complete reference and documentation on Maven please visit the following
online resources.
+
+* [Apache Maven Project](http://maven.apache.org/)
+* [Maven: The Complete Reference by
Sonatype](http://books.sonatype.com/mvnref-book/reference/public-book.html)
+
+
+### Java
+
+The build requires a Java 8 or later. You should set the `JAVA_HOME`
environment variable and include its bin directory
+in your `PATH`.
+
+Check java version by executing the following at your command prompt.
+
+ java -version
+
+## Project structure
+
+Qpid Broker-J consists of a number of modules and sub-modules located in their
own directories. Each Qpid Broker-J module
+has its own POM file (pom.xml) located in its root directory. This file
defines the modules version, dependencies and
+project inheritance as well as the configuration of the relative maven plugins
specific to this module.
+The Qpid Broker-J parent pom.xml is located in the root of the project and
declares all qpid modules, dependencies,
+plugins, etc.
+
+## Building
+
+The project is built by executing maven command in conjunction with
pre-defined profiles. For example, the command below
+cleans previous build output and install all modules to local repository
without running the tests:
+
+ mvn clean install -DskipTests
+
+The following command installs all modules to the local repository after
running all the tests:
+
+ mvn clean install
+
+### Maven Commands
+
+Here is a quick reference guide outlining some of the maven commands you can
run and what the outcome will be.
+
+Please visit [Introduction To The Maven
Lifecycle](https://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html)
+reference guide for full details on all the maven lifecycle phases.
+
+| Command | Description | Notes
|
+|---------------|:-----------------------------------------------------|:---------------------------------------|
+| mvn clean | removes all the files created by the previous build
|Usually means it deletes the contents of the modules `*/target/` directory.|
+| mvn validate | validates that the maven poms are all correct and that no
config is missing | Useful to run when making build modifications to ensure
consistency.|
+| mvn compile | compiles the source code of the project | This will verify
that project dependencies are correct.|
+| mvn test | executes the unit tests | Should not rely on code being
packaged or deployed, only unit tests.|
+| mvn package | packages the code into the its distributable formats (jar,
zip etc)| Each pom specifies what the distribution format is, default is POM.|
+| mvn verify | verifies that the packaged code is valid| This will run the
integration and system tests.|
+| mvn install | installs the package into the local maven repository | This
will result in the module being available, locally, as a depedency|
+| mvn deploy | copies the final artifacts to the remote maven repository
for sharing | This would happen only when modules ready to be shared with other
developers or projects|
+| mvn site:stage| creates a staged version of the maven site with all the
reports | Staging output defaults to the `*/target/staging/` directory |
+| mvn jacoco:report | generates the code coverage report for the tests that
have been executed | Test output appears in the `*/target/site/jacoco/`
directory |
+
+### Maven Profiles
+
+Maven profiles are used to run tests for the supported protocols and storage
options.
+The specific profile can be enabled at the command line using option `-P`.
+If no profile is selected, `java-mms.1-0` is run by default.
+
+An example below executes integration tests using profile `java-bdb.1-0`.
+
+ mvn verify -Pjava-bdb.1-0
+
+Please visit [Introduction To Maven
Profiles](http://maven.apache.org/guides/introduction/introduction-to-profiles.html)
+for full details on the use of profiles in a maven build.
+
+Profiles can be listed using command
+
+ mvn help:all-profiles
+
+The test profile names follow the form *java-store.n-n*, where
+ *store* signifies the storage module and
+ *n-n* the AMQP protocol version number.
+
+For store, the options include:
+
+* *bdb* - Oracle BDB JE
+* *dby* - Apache Derby
+* *mms* - an in-memory store principally used for testing.
+
+For protocols, the options include:
+
+* *1-0* - AMQP 1-0
+* *0-10* - AMQP 0-10
+* *0-9-1* - AMQP 0-9-1
+* *0-9* - AMQP 0-9
+
+To activate profile for BDB and AMQP 1.0 protocol use:
+
+ mvn verify -Pjava-bdb.1-0
+
+When activating AMQP 0-8..0-10 profiles, it is also necessary to pass the
system property *-DenableAmqp0-x*.
+For example,
+
+ mvn verify -Pjava-dby.0-9 -DenableAmqp0-x
+
+The table below lists some of profiles defined in parent pom.xml.
+
+|Profile Name | Description |
+|----------------|--------------------------------------------------------|
+|apache-release | Release profile for performing Apache software releases|
+|dependency-check|Dependency check to validate project licensing ahead of
release|
+|java-mms.1-0 | Test profile to run integration tests using AMQP protocol
1.0 against broker with memory message store|
+|java-bdb.1-0 | Test profile to run integration tests using AMQP protocol
1.0 against broker with bdb message store|
+|java-dby.1-0 | Test profile to run integration tests using AMQP protocol
1.0 against broker with Derby message store|
+|java-mms.0-10 | Test profile to run integration tests using AMQP protocol
0-10 against broker with memory message store|
+|java-bdb.0-10 | Test profile to run integration tests using AMQP protocol
0-10 against broker with bdb message store|
+|java-dby.0-10 | Test profile to run integration tests using AMQP protocol
0-10 against broker with Derby message store|
+|java-mms.0-9-1 | Test profile to run integration tests using AMQP protocol
0-9-1 against broker with memory message store|
+|java-bdb.0-9-1 | Test profile to run integration tests using AMQP protocol
0-9-1 against broker with bdb message store|
+|java-dby.0-9-1 | Test profile to run integration tests using AMQP protocol
0-9-1 against broker with Derby message store|
+
+There is a set of test profiles to run tests against broker with in-memory
Derby store, for example
+
+ mvn verify -Pjava-dby-mem.0-10 -DenableAmqp0-x -DskipITs=false
+
+### Maven Build Output
+
+By default the build output for a maven module will appear in the modules
`target` directory:
+For example, the broker core module output appears in:
+
+ broker-core/target
+
+Depending on the goals specified in the last build, one or all of the
following can be found in the `target` directory.
+
+| Location | Description | Note |
+|----------|-------------|------|
+| */target/|Packaged build artifacts|JAR, WAS, ZIP etc as well as any release
specific artifacts specified in assembly files i.e. release tars|
Review comment:
Fixed as suggested
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> [Broker-J] Create a developer guide for Qpid Broker-J
> -----------------------------------------------------
>
> Key: QPID-8361
> URL: https://issues.apache.org/jira/browse/QPID-8361
> Project: Qpid
> Issue Type: Task
> Components: Broker-J
> Reporter: Alex Rudyy
> Priority: Major
> Fix For: qpid-java-broker-8.0.0
>
>
> The developer documentation is currently scattered over various Qpid
> confluence pages. It could be challenging for people interested in
> contributing to the project to find that documentation. A developer guide
> could be added to cover such aspects as
> * Environment Setup
> * Building project
> * Running tests
> * Releasing
> * Architecture overview
> The following wiki pages are good candidates for inclusion into a developer
> guide:
> [High Level
> Architecture|https://cwiki.apache.org/confluence/display/qpid/High+Level+Architecture]
> [How To Build Qpid
> Broker-J|https://cwiki.apache.org/confluence/display/qpid/How+To+Build+Qpid+Broker-J]
> [Releasing Qpid
> Broker-J|https://cwiki.apache.org/confluence/display/qpid/Releasing+Qpid+Broker-J]
> The wiki pages below might be included as well
> [Java Coding
> Standards|https://cwiki.apache.org/confluence/display/qpid/Java+Coding+Standards]
> [Qpid Java Run
> Scripts|https://cwiki.apache.org/confluence/display/qpid/Qpid+Java+Run+Scripts]
> The developer documentation should be easy to modify, maintain and preview.
> Thus, it can be written in markdown or
> [asciidoc|https://asciidoctor.org/docs/asciidoc-syntax-quick-reference/]. The
> latter is also supported on github.
> Potentially, it can be published on Qpid project site as part of release
> process.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org
