You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafodion.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/12/06 21:45:00 UTC
[jira] [Commented] (TRAFODION-3241) When running "mvn site" with
JDK1.8, you will have some errors for DCS/JDBCT4
[ https://issues.apache.org/jira/browse/TRAFODION-3241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16712052#comment-16712052 ]
ASF GitHub Bot commented on TRAFODION-3241:
-------------------------------------------
Github user narendragoyal commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1753#discussion_r239625627
--- 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 --
Just my opinion :) : I think the 'clean' target should not be conditional on the existence of the target/*JAR, it should execute the commands and perhaps the 'maven clean' will cleanup the intermediate class files also (even when a target/*.jar does not exist).
> 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)