Github user wilderrodrigues commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1075#discussion_r45169717
  
    --- Diff: packaging/package.sh ---
    @@ -18,26 +18,37 @@
     
     function usage() {
         echo ""
    -    echo "usage: ./package.sh [-p|--pack] [-h|--help] [ARGS]"
    +    echo "usage: ./package.sh [-h|--help] -d|--distribution <name> 
[-r|--release <version>] [-p|--pack oss|OSS|noredist|NOREDIST] [-s|--simulator 
default|DEFAULT|simulator|SIMULATOR]"
         echo ""
    -    echo "The commonly used Arguments are:"
    -    echo "-p|--pack oss|OSS             To package with only 
redistributable libraries (default)"
    -    echo "-p|--pack noredist|NOREDIST   To package with 
non-redistributable libraries"
    -    echo "-d centos7|centos63|fedora20|fedora21  To build a package for a 
distribution"
    -    echo "-s simulator|SIMULATOR        To build for Simulator"
    +    echo "The supported arguments are:"
    +    echo "  To package with only redistributable libraries (default)"
    +    echo "    -p|--pack oss|OSS"
    +    echo "  To package with non-redistributable libraries"
    +    echo "    -p|--pack noredist|NOREDIST"
    +    echo "  To build a package for a distribution (mandatory)"
    +    echo "    -d|--distribution centos7|centos63|fedora20|fedora21"
    +    echo "  To set the package release version (optional)"
    +    echo "  (default is 1 for normal and prereleases, empty for SNAPSHOT)"
    +    echo "    -r|--release version(integer)"
    +    echo "  To build for Simulator (optional)"
    +    echo "    -s|--simulator default|DEFAULT|simulator|SIMULATOR"
    +    echo "  To display this information"
    +    echo "    -h|--help"
         echo ""
    -    echo "Examples: ./package.sh -p|--pack oss|OSS"
    -    echo "          ./package.sh -p|--pack noredist|NOREDIST"
    -    echo "          ./package.sh (Default OSS)"
    -    exit 1
    +    echo "Examples: ./package.sh --pack oss"
    +    echo "          ./package.sh --pack noredist"
    +    echo "          ./package.sh --pack oss --distribution centos7 
--release 42"
    --- End diff --
    
    @davidamorimfaria, what are the implications in giving a release number? 
For example, giving 4.6 or nothing will reflect in a different package or is it 
just for the final RPM file name?
    
    For the looks on the code, it seems to be only the file name. :)
    
    No big deal, I'm just curious. I will take some time to test your PR today.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to