Repository: incubator-sentry
Updated Branches:
  refs/heads/master d40e5c4fb -> e0e70e2cd


SENTRY-168: Trivial fixes to README and pom.xml (Sravya Tirukkovalur via Prasad 
Mujumdar)

Conflicts:
        pom.xml


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/e0e70e2c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/e0e70e2c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/e0e70e2c

Branch: refs/heads/master
Commit: e0e70e2cd36daeac6fb2982728100d1d40def841
Parents: d40e5c4
Author: Sravya Tirukkovalur <[email protected]>
Authored: Thu Apr 10 11:09:16 2014 -0700
Committer: Sravya Tirukkovalur <[email protected]>
Committed: Thu Apr 10 13:59:04 2014 -0700

----------------------------------------------------------------------
 README.md |  7 +++++--
 pom.xml   | 29 ++++++++++++++++++-----------
 2 files changed, 23 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/e0e70e2c/README.md
----------------------------------------------------------------------
diff --git a/README.md b/README.md
index 2e142cf..3aabc60 100644
--- a/README.md
+++ b/README.md
@@ -17,9 +17,12 @@ Building Sentry requires the following tools:
 * Apache Maven 3.0+
 * Java JDK 1.6+
 
+Running hive end to end tests requires:
+* wget
+
 To compile Sentry, run:
 
-mvn compile
+mvn install -DskipTests
 
 To run Sentry tests, run:
 
@@ -29,4 +32,4 @@ To build a distribution, run:
 
 mvn install
 
-The final Sentry distribution artifacts will be in 
$project/sentry-dist/target/.  
+The final Sentry distribution artifacts will be in 
$project/sentry-dist/target/.

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/e0e70e2c/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index b053bbf..c52f7ad 100644
--- a/pom.xml
+++ b/pom.xml
@@ -439,28 +439,35 @@ limitations under the License.
               </goals>
               <configuration>
                 <excludes>
+                  <!-- Git specific files -->
                   <exclude>.git/</exclude>
                   <exclude>.gitignore</exclude>
+                  <!--IDE specific files-->
                   <exclude>.idea/</exclude>
-                  <exclude>maven-repo/</exclude>
-                  <exclude>test-output/</exclude>
                   <exclude>**/*.iml</exclude>
-                  <exclude>*.log</exclude>
                   <exclude>**/nb-configuration.xml</exclude>
-                  <exclude>**.patch</exclude>
-                  <exclude>README*</exclude>
-                  <exclude>**/.project</exclude>
-                  <exclude>**/target/</exclude>
+                  <exclude>**/.classpath</exclude>
                   <exclude>**/.settings/**</exclude>
+                  <exclude>**/.project</exclude>
                   <exclude>**/.metadata/</exclude>
-                  <exclude>**/.classpath</exclude>
+                  <!-- Maven working directory -->
+                  <exclude>**/target/</exclude>
+                  <!-- Pre commit testing generated files -->
+                  <exclude>maven-repo/</exclude>
+                  <exclude>test-output/</exclude>
+                  <!-- Derby files which are created after test run -->
+                  <exclude>**/derby.log</exclude>
                   <exclude>**/service.properties</exclude>
-                  <exclude>**/kv1.dat</exclude>
                   <exclude>**/*.lck</exclude>
-                  <!-- exclude generated solr config files -->
+                  <!-- Patch files which can be lying around -->
+                  <exclude>**.patch</exclude>
+                  <!-- Exclude generated solr config files -->
                   <exclude>**/solr/collection1/conf/**</exclude>
-                  <!-- exclude generated thrift files -->
+                  <!-- Exclude generated thrift files -->
                   <exclude>**/gen/**</exclude>
+                  <!-- README and test data with exact format -->
+                  <exclude>README*</exclude>
+                  <exclude>**/kv1.dat</exclude>
                 </excludes>
               </configuration>
             </execution>

Reply via email to