I think we need Edward to run 'mvn release:rollback' as soon as possible
(as latest vote has been canceled) and then commit this again.

Tommaso


2013/9/15 <[email protected]>

> Author: chl501
> Date: Sun Sep 15 10:20:01 2013
> New Revision: 1523425
>
> URL: http://svn.apache.org/r1523425
> Log:
> HAMA-802: Skip Hama Pipes native build when cmake is missing
>
> Modified:
>     hama/trunk/c++/pom.xml
>     hama/trunk/c++/src/main/native/pipes/impl/HamaPipes.cc
>
> Modified: hama/trunk/c++/pom.xml
> URL:
> http://svn.apache.org/viewvc/hama/trunk/c%2B%2B/pom.xml?rev=1523425&r1=1523424&r2=1523425&view=diff
>
> ==============================================================================
> --- hama/trunk/c++/pom.xml (original)
> +++ hama/trunk/c++/pom.xml Sun Sep 15 10:20:01 2013
> @@ -31,7 +31,7 @@
>      <description>Apache Hama Pipes</description>
>      <packaging>pom</packaging>
>
> -    <profiles>
> +    <profiles>
>        <profile>
>          <id>native</id>
>          <activation>
> @@ -49,16 +49,32 @@
>                    <goals><goal>run</goal></goals>
>                    <configuration>
>                      <target>
> -                      <mkdir dir="${project.build.directory}/native" />
> -                      <exec executable="cmake"
> dir="${project.build.directory}/native" failonerror="true">
> -                        <arg line="${basedir}/src/
> -DJVM_ARCH_DATA_MODEL=${sun.arch.data.model}" />
> -                      </exec>
> -                      <exec executable="make"
> dir="${project.build.directory}/native" failonerror="true">
> -                        <arg line="VERBOSE=1" />
> -                      </exec>
> -                      <!-- The second make is a workaround for
> HADOOP-9215.  It can
> -                           be removed when version 2.6 of cmake is no
> longer supported . -->
> -                      <exec executable="make"
> dir="${project.build.directory}/native" failonerror="true" />
> +                      <taskdef
> resource="net/sf/antcontrib/antcontrib.properties"
> classpathref="maven.plugin.classpath" />
> +                      <!-- Check if cmake is installed -->
> +                      <property environment="env" />
> +                      <if>
> +                        <or>
> +                          <available file="cmake" filepath="${env.PATH}"
> />
> +                          <!--  on Windows it can be Path, path -->
> +                          <available file="cmake" filepath="${env.Path}"
> />
> +                          <available file="cmake" filepath="${env.path}"
> />
> +                        </or>
> +                        <then>
> +                          <mkdir dir="${project.build.directory}/native"
> />
> +                          <exec executable="cmake"
> dir="${project.build.directory}/native" failonerror="true">
> +                            <arg line="${basedir}/src/
> -DJVM_ARCH_DATA_MODEL=${sun.arch.data.model}" />
> +                          </exec>
> +                          <exec executable="make"
> dir="${project.build.directory}/native" failonerror="true">
> +                            <arg line="VERBOSE=1" />
> +                          </exec>
> +                          <!-- The second make is a workaround for
> HADOOP-9215.  It can
> +                               be removed when version 2.6 of cmake is no
> longer supported . -->
> +                          <exec executable="make"
> dir="${project.build.directory}/native" failonerror="true" />
> +                        </then>
> +                        <else>
> +                               <echo>CMAKE is missing! Skipping native
> build!</echo>
> +                        </else>
> +                      </if>
>                      </target>
>                    </configuration>
>                  </execution>
> @@ -75,6 +91,13 @@
>                  </execution>
>                  -->
>                </executions>
> +              <dependencies>
> +                <dependency>
> +                  <groupId>ant-contrib</groupId>
> +                  <artifactId>ant-contrib</artifactId>
> +                  <version>20020829</version>
> +                </dependency>
> +              </dependencies>
>              </plugin>
>            </plugins>
>          </build>
>
> Modified: hama/trunk/c++/src/main/native/pipes/impl/HamaPipes.cc
> URL:
> http://svn.apache.org/viewvc/hama/trunk/c%2B%2B/src/main/native/pipes/impl/HamaPipes.cc?rev=1523425&r1=1523424&r2=1523425&view=diff
>
> ==============================================================================
> --- hama/trunk/c++/src/main/native/pipes/impl/HamaPipes.cc (original)
> +++ hama/trunk/c++/src/main/native/pipes/impl/HamaPipes.cc Sun Sep 15
> 10:20:01 2013
> @@ -23,6 +23,7 @@
>  #include <map>
>  #include <vector>
>
> +#include <unistd.h>
>  #include <errno.h>
>  #include <netinet/in.h>
>  #include <stdint.h>
>
>
>

Reply via email to