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

    https://github.com/apache/tinkerpop/pull/802#discussion_r189340476
  
    --- Diff: gremlin-console/pom.xml ---
    @@ -336,5 +336,59 @@ limitations under the License.
                     </plugins>
                 </build>
             </profile>
    +
    +        <profile>
    +            <id>docker-images</id>
    +            <activation>
    +                <activeByDefault>false</activeByDefault>
    +                <file>
    +                    <exists>.docker</exists>
    +                </file>
    +            </activation>
    +            <build>
    +                <plugins>                    
    +                    <plugin>
    +                        <groupId>com.spotify</groupId>
    +                        <artifactId>dockerfile-maven-plugin</artifactId>
    +                        <executions>
    +                            <execution>
    +                                <id>docker-image-build</id>
    +                                <goals>
    +                                    <goal>build</goal>
    +                                </goals>
    +                                <configuration>
    +                                    <tag>${project.version}</tag>
    +                                    <buildArgs>
    +                                        
<GREMLIN_CONSOLE_DIR>target/apache-tinkerpop-${project.artifactId}-${project.version}-standalone</GREMLIN_CONSOLE_DIR>
    +                                    </buildArgs>
    +                                </configuration>
    +                            </execution>
    +                            <execution>
    +                                <id>docker-image-tag-minor-version</id>
    +                                <goals>
    +                                    <goal>tag</goal>
    +                                </goals>
    +                                <configuration>                            
                         
    +                                    <tag>3.2</tag>
    --- End diff --
    
    Making it dynamic would of course be the best solution and it looks like we 
can again use the `build-helper-maven-plugin` for this:
    
    > The parse-version goal can be used to access the component parts of a 
version string. For example, the major version or the qualifier by themselves.
    
    Source: http://www.mojohaus.org/build-helper-maven-plugin/usage.html
    
    I will try that out and then either update the PR accordingly or add 
something to the dev docs if it doesn't work.


---

Reply via email to