[ 
https://issues.apache.org/jira/browse/TRAFODION-3241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16712625#comment-16712625
 ] 

ASF GitHub Bot commented on TRAFODION-3241:
-------------------------------------------

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

    https://github.com/apache/trafodion/pull/1753#discussion_r239762073
  
    --- Diff: core/conn/jdbcT4/Makefile ---
    @@ -26,17 +26,20 @@ include ../../macros.gmk #top level
     all: build_all
     
     build_all: LICENSE NOTICE 
    -   echo "$(MAVEN) package -DskipTests"
    -   set -o pipefail && $(MAVEN) package -DskipTests | tee build_jdbct4.log 
| grep --line-buffered -E -e '^\[[^WId]' -e '^\[INFO\] B[Uu][Ii][Ll][Dd]' -e 
'to compile'
    -   cp target/jdbcT4-${TRAFODION_VER}.jar ${TRAF_HOME}/export/lib
    -   mkdir -p ../clients
    -   mv target/jdbcT4-${TRAFODION_VER}.zip ../clients
    -   #ln -sf ${TRAF_HOME}/export/lib/jdbcT4-${TRAFODION_VER}.jar 
${TRAF_HOME}/export/lib/jdbcT4.jar
    -   `cd ${TRAF_HOME}/export/lib;ln -sf jdbcT4-${TRAFODION_VER}.jar 
jdbcT4.jar`
    +   @if [ ! -f target/*.jar ]; then \
    +           echo "$(MAVEN) install -DskipTests"; \
    +           set -o pipefail && $(MAVEN) install -DskipTests | tee 
build_jdbct4.log | grep --line-buffered -E -e '^\[[^WId]' -e '^\[INFO\] 
B[Uu][Ii][Ll][Dd]' -e 'to compile'; \
    +           cp target/jdbcT4-${TRAFODION_VER}.jar ${TRAF_HOME}/export/lib; \
    +           mkdir -p ../clients; \
    +           mv target/jdbcT4-${TRAFODION_VER}.zip ../clients; \
    +           cd ${TRAF_HOME}/export/lib;ln -sf jdbcT4-${TRAFODION_VER}.jar 
jdbcT4.jar; \
    +   fi
     
     clean:
    -   -$(MAVEN) clean | grep ERROR
    -   $(RM) build_jdbct4.log ${TRAF_HOME}/export/lib/jdbcT4*.jar
    +   @if [ -f target/*.jar ]; then \
    --- End diff --
    
    The classes are in folder "target". if this folder does not exist, 'mvn 
clean' actually will do nothing. So here yes it should b -d target.


> When running "mvn site" with JDK1.8, you will have some errors for DCS/JDBCT4
> -----------------------------------------------------------------------------
>
>                 Key: TRAFODION-3241
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-3241
>             Project: Apache Trafodion
>          Issue Type: Bug
>          Components: connectivity-general
>            Reporter: Kevin Xu
>            Priority: Major
>
> [ERROR] 
> /home/trafodion/esgyndb/trafodion/dcs/src/main/java/org/trafodion/dcs/util/InfoServer.java:49:
>  error: element not closed: code
> [ERROR]    * The jsp scripts are taken from src/dcs-webapps/<code>name<code>.
> [ERROR]                                                               ^
> [ERROR] 
> /home/trafodion/esgyndb/trafodion/dcs/src/main/java/org/trafodion/dcs/util/InfoServer.java:49:
>  error: element not closed: code
> [ERROR]    * The jsp scripts are taken from src/dcs-webapps/<code>name<code>.
> [ERROR]                                                     ^
> [ERROR] 
> /home/trafodion/esgyndb/trafodion/dcs/src/main/java/org/trafodion/dcs/util/InfoServer.java:57:
>  warning: no @param for c
> [ERROR]   public InfoServer(String name, String bindAddress, int port, 
> boolean findPort,
> [ERROR]          ^
> [ERROR] 
> /home/trafodion/esgyndb/trafodion/dcs/src/main/java/org/trafodion/dcs/util/RetryCounter.java:52:
>  warning: no description for @throws
> [ERROR]      * @throws InterruptedException
> [ERROR]        ^
> [ERROR]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to