Hi Imesh,

How about allowing all JDK vendors EXCEPT for OpenJDK?

                    <rules>
                        <requireMavenVersion>
                            <version>[3.0,)</version>
                        </requireMavenVersion>
                        <requireJavaVersion>
                            <version>[1.6,1.8)</version>
                        </requireJavaVersion>
                        <requireProperty>
                            <property>java.runtime.name</property>
                            <message>JAVA_HOME points to OpenJDK which is
not supported for building Stratos.</message>
                            <regex>^((?!OpenJDK).)*$</regex>
                            <regexMessage>JAVA_HOME points to OpenJDK which
is not supported for building Stratos.</regexMessage>
                        </requireProperty>
                    </rules>

It would be good to fail early for OpenJDK if stratos cannot be built with
OpenJDK.

Many thanks,

Chris






On Sun, Nov 10, 2013 at 5:24 PM, <im...@apache.org> wrote:

> Updated Branches:
>   refs/heads/master 1041b1bae -> 973ec4471
>
>
> Removed java vendor required property from root pom.xml as it is too
> restrictive
>
>
> Project: http://git-wip-us.apache.org/repos/asf/incubator-stratos/repo
> Commit:
> http://git-wip-us.apache.org/repos/asf/incubator-stratos/commit/973ec447
> Tree:
> http://git-wip-us.apache.org/repos/asf/incubator-stratos/tree/973ec447
> Diff:
> http://git-wip-us.apache.org/repos/asf/incubator-stratos/diff/973ec447
>
> Branch: refs/heads/master
> Commit: 973ec4471829c13e420191b13746c8a48cdb5f5e
> Parents: 1041b1b
> Author: Imesh Gunaratne <im...@apache.org>
> Authored: Sun Nov 10 22:54:05 2013 +0530
> Committer: Imesh Gunaratne <im...@apache.org>
> Committed: Sun Nov 10 22:54:05 2013 +0530
>
> ----------------------------------------------------------------------
>  pom.xml | 12 ------------
>  1 file changed, 12 deletions(-)
> ----------------------------------------------------------------------
>
>
>
> http://git-wip-us.apache.org/repos/asf/incubator-stratos/blob/973ec447/pom.xml
> ----------------------------------------------------------------------
> diff --git a/pom.xml b/pom.xml
> index 1444b26..13b1e6a 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -348,18 +348,6 @@
>                          <requireJavaVersion>
>                              <version>[1.6,1.8)</version>
>                          </requireJavaVersion>
> -                        <requireProperty>
> -                            <property>java.vendor</property>
> -                            <message>JAVA_HOME does not point to a Sun
> JDK or Oracle JDK.  Only Sun and Oracle JDKs are supported for building
> Stratos.</message>
> -                            <regex>(Sun Microsystems Inc\.|Oracle
> Corporation)</regex>
> -                            <regexMessage>JAVA_HOME does not point to a
> Sun JDK or Oracle JDK.  Only Sun and Oracle JDKs are supported for building
> Stratos.</regexMessage>
> -                        </requireProperty>
> -                        <requireProperty>
> -                            <property>java.runtime.name</property>
> -                            <message>JAVA_HOME does not point to a Sun
> JDK or Oracle JDK.  Only Sun and Oracle JDKs are supported for building
> Stratos.</message>
> -                            <regex>Java\(TM\) SE Runtime
> Environment</regex>
> -                            <regexMessage>JAVA_HOME does not point to a
> Sun JDK or Oracle JDK.  Only Sun and Oracle JDKs are supported for building
> Stratos.</regexMessage>
> -                        </requireProperty>
>                      </rules>
>                      <fail>true</fail>
>                  </configuration>
>
>


-- 
Check out my professional profile and connect with me on LinkedIn.
http://lnkd.in/cw5k69

Reply via email to