You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by ms...@apache.org on 2019/06/05 11:15:54 UTC

[portals-pluto] branch master updated: Added compile-time dependency on pluto tag library to some of the demo portlets that did not have it. The purpose of this is only to avoid JSP validation errors in eclipse.

This is an automated email from the ASF dual-hosted git repository.

msnicklous pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/portals-pluto.git


The following commit(s) were added to refs/heads/master by this push:
     new 18e718a  Added compile-time dependency on pluto tag library to some of the demo portlets that did not have it. The purpose of this is only to avoid JSP validation errors in eclipse.
18e718a is described below

commit 18e718ad69cb66c5a6281d8ade87c20c99ee4548
Author: Scott Nicklous <ms...@apache.org>
AuthorDate: Wed Jun 5 13:08:53 2019 +0200

    Added compile-time dependency on pluto tag library to some of the demo
    portlets that did not have it. The purpose of this is only to avoid JSP
    validation errors in eclipse.
    
    Added dummy context path to example pluto portal config file contained
    in pluto-portal-driver.jar to avoid XML validation errors in eclipse.
---
 demo/chat-room-demo-portlet/pom.xml                |   6 +
 demo/hub-demo-portlet/pom.xml                      |  26 +-
 .../META-INF/pluto-portal-driver-config.xml        |   2 +-
 pluto-testsuite/pom.xml                            | 436 +++++++++++----------
 4 files changed, 248 insertions(+), 222 deletions(-)

diff --git a/demo/chat-room-demo-portlet/pom.xml b/demo/chat-room-demo-portlet/pom.xml
index cf3b8bd..ecc0fe8 100644
--- a/demo/chat-room-demo-portlet/pom.xml
+++ b/demo/chat-room-demo-portlet/pom.xml
@@ -38,6 +38,12 @@
 			<scope>provided</scope>
 		</dependency>
 		<dependency>
+			<groupId>org.apache.portals.pluto</groupId>
+			<artifactId>pluto-taglib</artifactId>
+			<version>${project.version}</version>
+			<scope>provided</scope>
+		</dependency>
+		<dependency>
 			<groupId>javax</groupId>
 			<artifactId>javaee-api</artifactId>
 			<version>6.0</version>
diff --git a/demo/hub-demo-portlet/pom.xml b/demo/hub-demo-portlet/pom.xml
index 98b1de3..b5dde78 100644
--- a/demo/hub-demo-portlet/pom.xml
+++ b/demo/hub-demo-portlet/pom.xml
@@ -34,6 +34,12 @@
 			<scope>provided</scope>
 		</dependency>
 		<dependency>
+			<groupId>org.apache.portals.pluto</groupId>
+			<artifactId>pluto-taglib</artifactId>
+			<version>${project.version}</version>
+			<scope>provided</scope>
+		</dependency>
+		<dependency>
 			<groupId>org.apache.tomcat</groupId>
 			<artifactId>tomcat-servlet-api</artifactId>
 			<scope>provided</scope>
@@ -64,13 +70,18 @@
 			<artifactId>taglibs-standard-jstlel</artifactId>
 			<scope>provided</scope>
 		</dependency>
-		<dependency>
-			<groupId>org.apache.portals.pluto</groupId>
-			<artifactId>pluto-taglib</artifactId>
-			<version>2.1.0-M3</version>
-			<scope>provided</scope>
-		</dependency>
 	</dependencies>
+	<build>
+		<finalName>${project.artifactId}</finalName>
+		<plugins>
+			<plugin>
+				<artifactId>maven-war-plugin</artifactId>
+				<configuration>
+					<failOnMissingWebXml>false</failOnMissingWebXml>
+				</configuration>
+			</plugin>
+		</plugins>
+	</build>
 	<profiles>
 		<profile>
 			<id>liferay</id>
@@ -85,7 +96,6 @@
 		<profile>
 			<id>liferay-cdi</id>
 			<build>
-				<finalName>${project.artifactId}</finalName>
 				<plugins>
 					<plugin>
 						<artifactId>maven-war-plugin</artifactId>
@@ -102,7 +112,6 @@
 		<profile>
 			<id>liferay-spring</id>
 			<build>
-				<finalName>${project.artifactId}</finalName>
 				<plugins>
 					<plugin>
 						<artifactId>maven-war-plugin</artifactId>
@@ -177,7 +186,6 @@
 				</dependency>
 			</dependencies>
 			<build>
-				<finalName>${project.artifactId}</finalName>
 				<plugins>
 					<plugin>
 						<artifactId>maven-war-plugin</artifactId>
diff --git a/pluto-portal-driver/src/main/resources/META-INF/pluto-portal-driver-config.xml b/pluto-portal-driver/src/main/resources/META-INF/pluto-portal-driver-config.xml
index 4a9a31a..38281b8 100644
--- a/pluto-portal-driver/src/main/resources/META-INF/pluto-portal-driver-config.xml
+++ b/pluto-portal-driver/src/main/resources/META-INF/pluto-portal-driver-config.xml
@@ -26,7 +26,7 @@ limitations under the License.
     <page:window-state>token</page:window-state>
   </page:supports>
   <page:portlet-app>
-    <page:context-path>page:context-path</page:context-path>
+    <page:context-path>/dummypath</page:context-path>
     <page:portlets>
       <page:portlet name="token"/>
     </page:portlets>
diff --git a/pluto-testsuite/pom.xml b/pluto-testsuite/pom.xml
index 4336036..56c4c38 100644
--- a/pluto-testsuite/pom.xml
+++ b/pluto-testsuite/pom.xml
@@ -18,155 +18,167 @@
   under the License.
 -->
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
-	<modelVersion>4.0.0</modelVersion>
-	<parent>
-		<groupId>org.apache.portals.pluto</groupId>
-		<artifactId>pluto</artifactId>
-		<version>3.1.1-SNAPSHOT</version>
-		<relativePath>../pom.xml</relativePath>
-	</parent>
-	<artifactId>pluto-testsuite</artifactId>
-	<packaging>war</packaging>
-	<name>Apache Pluto Testsuite Portlet</name>
-	<dependencies>
-		<dependency>
-			<groupId>javax.portlet</groupId>
-			<artifactId>portlet-api</artifactId>
-			<scope>provided</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.apache.tomcat</groupId>
-			<artifactId>tomcat-servlet-api</artifactId>
-			<scope>provided</scope>
-		</dependency>
-		<dependency>
-			<groupId>commons-digester</groupId>
-			<artifactId>commons-digester</artifactId>
-			<scope>${dependency.scope}</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.apache.taglibs</groupId>
-			<artifactId>taglibs-standard-spec</artifactId>
-			<scope>${dependency.scope}</scope>
-			<exclusions>
-				<exclusion>
-					<groupId>org.apache.tomcat</groupId>
-					<artifactId>tomcat-jsp-api</artifactId>
-				</exclusion>
-			</exclusions>
-		</dependency>
-		<dependency>
-			<groupId>org.apache.taglibs</groupId>
-			<artifactId>taglibs-standard-impl</artifactId>
-			<scope>${dependency.scope}</scope>
-			<exclusions>
-				<exclusion>
-					<groupId>org.apache.tomcat</groupId>
-					<artifactId>tomcat-jsp-api</artifactId>
-				</exclusion>
-			</exclusions>
-		</dependency>
-		<dependency>
-			<groupId>org.slf4j</groupId>
-			<artifactId>slf4j-api</artifactId>
-			<scope>provided</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.slf4j</groupId>
-			<artifactId>slf4j-jdk14</artifactId>
-			<scope>provided</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.slf4j</groupId>
-			<artifactId>jcl-over-slf4j</artifactId>
-			<scope>compile</scope>
-		</dependency>
-		<dependency>
-			<groupId>junit</groupId>
-			<artifactId>junit</artifactId>
-		</dependency>
-	</dependencies>
-	<!-- Build configuration -->
-	<build>
-		<resources>
-			<resource>
-				<directory>src/main/resources</directory>
-				<filtering>true</filtering>
-			</resource>
-		</resources>
-		<plugins>
-			<!-- configure maven-war-plugin to use updated web.xml -->
-			<plugin>
-				<groupId>org.apache.maven.plugins</groupId>
-				<artifactId>maven-war-plugin</artifactId>
-				<configuration>
-					<webXml>${project.build.directory}/pluto-resources/web.xml</webXml>
-				</configuration>
-			</plugin>
-			<!-- bind 'pluto:assemble' goal to 'process-resources' lifecycle -->
-			<plugin>
-				<groupId>org.apache.portals.pluto</groupId>
-				<artifactId>pluto-maven-plugin</artifactId>
-				<version>${project.version}</version>
-				<executions>
-					<execution>
-						<phase>generate-resources</phase>
-						<goals>
-							<goal>assemble</goal>
-						</goals>
-					</execution>
-				</executions>
-			</plugin>
-			<!-- workaround for war processing of m-r-r-plugin causing the generated NOTICE and LICENSE file to be put under WEB-INF/classes/META-INF -->
-			<plugin>
-				<groupId>org.apache.maven.plugins</groupId>
-				<artifactId>maven-remote-resources-plugin</artifactId>
-				<executions>
-					<execution>
-						<goals>
-							<goal>process</goal>
-						</goals>
-						<configuration>
-							<outputDirectory>${project.build.directory}/${project.build.finalName}</outputDirectory>
-							<attached>false</attached>
-						</configuration>
-					</execution>
-				</executions>
-			</plugin>
-			<!-- Don't attach source or javadoc archive -->
-			<plugin>
-				<artifactId>maven-source-plugin</artifactId>
-				<configuration>
-					<attach>false</attach>
-				</configuration>
-			</plugin>
-			<plugin>
-				<artifactId>maven-javadoc-plugin</artifactId>
-				<configuration>
-					<skip>true</skip>
-				</configuration>
-			</plugin>
-		</plugins>
-	</build>
-	<profiles>
-		<profile>
-			<id>deploy-testsuite</id>
-			<activation>
-				<activeByDefault>false</activeByDefault>
-				<property>
-					<name>deploy.dir</name>
-				</property>
-			</activation>
-			<build>
-				<plugins>
-					<plugin>
-						<artifactId>maven-antrun-plugin</artifactId>
-						<executions>
-							<execution>
-								<phase>integration-test</phase>
-								<configuration>
-									<tasks>
-										<!--
+   <modelVersion>4.0.0</modelVersion>
+   <parent>
+      <groupId>org.apache.portals.pluto</groupId>
+      <artifactId>pluto</artifactId>
+      <version>3.1.1-SNAPSHOT</version>
+      <relativePath>../pom.xml</relativePath>
+   </parent>
+   <artifactId>pluto-testsuite</artifactId>
+   <packaging>war</packaging>
+   <name>Apache Pluto Testsuite Portlet</name>
+   <dependencies>
+      <dependency>
+         <groupId>javax.portlet</groupId>
+         <artifactId>portlet-api</artifactId>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>org.apache.portals.pluto</groupId>
+         <artifactId>pluto-taglib</artifactId>
+         <version>${project.version}</version>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>org.apache.tomcat</groupId>
+         <artifactId>tomcat-servlet-api</artifactId>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>javax.servlet</groupId>
+         <artifactId>jsp-api</artifactId>
+         <version>2.0</version>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>commons-digester</groupId>
+         <artifactId>commons-digester</artifactId>
+         <scope>${dependency.scope}</scope>
+      </dependency>
+      <dependency>
+         <groupId>org.apache.taglibs</groupId>
+         <artifactId>taglibs-standard-spec</artifactId>
+         <scope>${dependency.scope}</scope>
+         <exclusions>
+            <exclusion>
+               <groupId>org.apache.tomcat</groupId>
+               <artifactId>tomcat-jsp-api</artifactId>
+            </exclusion>
+         </exclusions>
+      </dependency>
+      <dependency>
+         <groupId>org.apache.taglibs</groupId>
+         <artifactId>taglibs-standard-impl</artifactId>
+         <scope>${dependency.scope}</scope>
+         <exclusions>
+            <exclusion>
+               <groupId>org.apache.tomcat</groupId>
+               <artifactId>tomcat-jsp-api</artifactId>
+            </exclusion>
+         </exclusions>
+      </dependency>
+      <dependency>
+         <groupId>org.slf4j</groupId>
+         <artifactId>slf4j-api</artifactId>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>org.slf4j</groupId>
+         <artifactId>slf4j-jdk14</artifactId>
+         <scope>provided</scope>
+      </dependency>
+      <dependency>
+         <groupId>org.slf4j</groupId>
+         <artifactId>jcl-over-slf4j</artifactId>
+         <scope>compile</scope>
+      </dependency>
+      <dependency>
+         <groupId>junit</groupId>
+         <artifactId>junit</artifactId>
+      </dependency>
+   </dependencies>
+   <!-- Build configuration -->
+   <build>
+      <resources>
+         <resource>
+            <directory>src/main/resources</directory>
+            <filtering>true</filtering>
+         </resource>
+      </resources>
+      <plugins>
+         <!-- configure maven-war-plugin to use updated web.xml -->
+         <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-war-plugin</artifactId>
+            <configuration>
+               <webXml>${project.build.directory}/pluto-resources/web.xml</webXml>
+            </configuration>
+         </plugin>
+         <!-- bind 'pluto:assemble' goal to 'process-resources' lifecycle -->
+         <plugin>
+            <groupId>org.apache.portals.pluto</groupId>
+            <artifactId>pluto-maven-plugin</artifactId>
+            <version>${project.version}</version>
+            <executions>
+               <execution>
+                  <phase>generate-resources</phase>
+                  <goals>
+                     <goal>assemble</goal>
+                  </goals>
+               </execution>
+            </executions>
+         </plugin>
+         <!-- workaround for war processing of m-r-r-plugin causing the generated NOTICE and LICENSE file to be put under WEB-INF/classes/META-INF -->
+         <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-remote-resources-plugin</artifactId>
+            <executions>
+               <execution>
+                  <goals>
+                     <goal>process</goal>
+                  </goals>
+                  <configuration>
+                     <outputDirectory>${project.build.directory}/${project.build.finalName}</outputDirectory>
+                     <attached>false</attached>
+                  </configuration>
+               </execution>
+            </executions>
+         </plugin>
+         <!-- Don't attach source or javadoc archive -->
+         <plugin>
+            <artifactId>maven-source-plugin</artifactId>
+            <configuration>
+               <attach>false</attach>
+            </configuration>
+         </plugin>
+         <plugin>
+            <artifactId>maven-javadoc-plugin</artifactId>
+            <configuration>
+               <skip>true</skip>
+            </configuration>
+         </plugin>
+      </plugins>
+   </build>
+   <profiles>
+      <profile>
+         <id>deploy-testsuite</id>
+         <activation>
+            <activeByDefault>false</activeByDefault>
+            <property>
+               <name>deploy.dir</name>
+            </property>
+         </activation>
+         <build>
+            <plugins>
+               <plugin>
+                  <artifactId>maven-antrun-plugin</artifactId>
+                  <executions>
+                     <execution>
+                        <phase>integration-test</phase>
+                        <configuration>
+                           <tasks>
+                              <!--
                                       Use this to build and deploy the testsuite war.
                                       PORTLET_DEPLOY_DIR is an environmental variable 
                                       pointing to the hot-deploy directory of your portal.
@@ -174,38 +186,38 @@
                                       on the command line when invoking maven:
                                       mvn -Ddeploy.dir=/pluto-1.1.4/webapps integration-test
                                     -->
-										<property environment="env" />
-										<property name="deploy.dir" value="${env.PORTLET_DEPLOY_DIR}" />
-										<copy file="target/${project.build.finalName}.war" tofile="${deploy.dir}/testsuite.war" />
-									</tasks>
-								</configuration>
-								<goals>
-									<goal>run</goal>
-								</goals>
-							</execution>
-						</executions>
-					</plugin>
-				</plugins>
-			</build>
-		</profile>
-		<profile>
-			<id>deploy-testsuite-by-env</id>
-			<activation>
-				<activeByDefault>false</activeByDefault>
-				<property>
-					<name>env.PORTLET_DEPLOY_DIR</name>
-				</property>
-			</activation>
-			<build>
-				<plugins>
-					<plugin>
-						<artifactId>maven-antrun-plugin</artifactId>
-						<executions>
-							<execution>
-								<phase>integration-test</phase>
-								<configuration>
-									<tasks>
-										<!--
+                              <property environment="env" />
+                              <property name="deploy.dir" value="${env.PORTLET_DEPLOY_DIR}" />
+                              <copy file="target/${project.build.finalName}.war" tofile="${deploy.dir}/testsuite.war" />
+                           </tasks>
+                        </configuration>
+                        <goals>
+                           <goal>run</goal>
+                        </goals>
+                     </execution>
+                  </executions>
+               </plugin>
+            </plugins>
+         </build>
+      </profile>
+      <profile>
+         <id>deploy-testsuite-by-env</id>
+         <activation>
+            <activeByDefault>false</activeByDefault>
+            <property>
+               <name>env.PORTLET_DEPLOY_DIR</name>
+            </property>
+         </activation>
+         <build>
+            <plugins>
+               <plugin>
+                  <artifactId>maven-antrun-plugin</artifactId>
+                  <executions>
+                     <execution>
+                        <phase>integration-test</phase>
+                        <configuration>
+                           <tasks>
+                              <!--
                                       Use this to build and deploy the testsuite war.
                                       PORTLET_DEPLOY_DIR is an environmental variable 
                                       pointing to the hot-deploy directory of your portal.
@@ -213,35 +225,35 @@
                                       on the command line when invoking maven:
                                       mvn -Ddeploy.dir=/pluto-1.1.4/webapps integration-test
                                     -->
-										<property environment="env" />
-										<property name="deploy.dir" value="${env.PORTLET_DEPLOY_DIR}" />
-										<copy file="target/${project.build.finalName}.war" tofile="${deploy.dir}/testsuite.war" />
-									</tasks>
-								</configuration>
-								<goals>
-									<goal>run</goal>
-								</goals>
-							</execution>
-						</executions>
-					</plugin>
-				</plugins>
-			</build>
-		</profile>
-		<profile>
-			<id>rat</id>
-			<build>
-				<plugins>
-					<plugin>
-						<groupId>org.apache.rat</groupId>
-						<artifactId>apache-rat-plugin</artifactId>
-						<configuration>
-							<excludes combine.children="append">
-								<exclude>src/main/webapp/META-INF/SLF4J-LICENSE</exclude>
-							</excludes>
-						</configuration>
-					</plugin>
-				</plugins>
-			</build>
-		</profile>
-	</profiles>
+                              <property environment="env" />
+                              <property name="deploy.dir" value="${env.PORTLET_DEPLOY_DIR}" />
+                              <copy file="target/${project.build.finalName}.war" tofile="${deploy.dir}/testsuite.war" />
+                           </tasks>
+                        </configuration>
+                        <goals>
+                           <goal>run</goal>
+                        </goals>
+                     </execution>
+                  </executions>
+               </plugin>
+            </plugins>
+         </build>
+      </profile>
+      <profile>
+         <id>rat</id>
+         <build>
+            <plugins>
+               <plugin>
+                  <groupId>org.apache.rat</groupId>
+                  <artifactId>apache-rat-plugin</artifactId>
+                  <configuration>
+                     <excludes combine.children="append">
+                        <exclude>src/main/webapp/META-INF/SLF4J-LICENSE</exclude>
+                     </excludes>
+                  </configuration>
+               </plugin>
+            </plugins>
+         </build>
+      </profile>
+   </profiles>
 </project>