Added ability to include/exclude paths/classes via real Regex

(cherry picked from commit 076df6211d01a1a9904a741566d645e9ab36c118)
johnrengelman · Jan 8, 2020 · 6b97759 · 6b97759
1 parent c8b4631
commit 6b97759
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/src/main/groovy/com/github/jengelman/gradle/plugins/shadow/relocation/SimpleRelocator.groovy b/src/main/groovy/com/github/jengelman/gradle/plugins/shadow/relocation/SimpleRelocator.groovy
@@ -98,6 +98,16 @@ class SimpleRelocator implements Relocator {
         return this
     }
 
+    SimpleRelocator includeRegex(String pattern) {
+        this.includes.add("%regex[$pattern]")
+        return this
+    }
+
+    SimpleRelocator excludeRegex(String pattern) {
+        this.excludes.add("%regex[$pattern]")
+        return this
+    }
+
     private static Set<String> normalizePatterns(Collection<String> patterns) {
         Set<String> normalized = null
 

diff --git a/...t/groovy/com/github/jengelman/gradle/plugins/shadow/relocation/SimpleRelocatorTest.groovy b/...t/groovy/com/github/jengelman/gradle/plugins/shadow/relocation/SimpleRelocatorTest.groovy
@@ -87,6 +87,30 @@ class SimpleRelocatorTest extends TestCase {
         assertEquals(true, relocator.canRelocatePath("/org/f"))  // equal to path pattern with /
     }
 
+    void testCanRelocatePathWithRegex() {
+        SimpleRelocator relocator
+
+        relocator = new SimpleRelocator("org.foo", null, null, null)
+        relocator.includeRegex("org/foo/R(\\\$.*)?\$")
+        assertEquals(true, relocator.canRelocatePath("org/foo/R.class"))
+        assertEquals(true, relocator.canRelocatePath("org/foo/R\$string.class"))
+        assertEquals(true, relocator.canRelocatePath("org/foo/R\$layout.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/Recording/R.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/Recording.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/bar/R\$string.class"))
+        assertEquals(false, relocator.canRelocatePath("org/R.class"))
+        assertEquals(false, relocator.canRelocatePath("org/R\$string.class"))
+
+        relocator = new SimpleRelocator("org.foo", null, null, null)
+        relocator.excludeRegex("org/foo/.*Factory[0-9].*")
+        assertEquals(true, relocator.canRelocatePath("org/foo/Factory.class"))
+        assertEquals(true, relocator.canRelocatePath("org/foo/FooFactoryMain.class"))
+        assertEquals(true, relocator.canRelocatePath("org/foo/BarFactory.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/Factory0.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/FooFactory1Main.class"))
+        assertEquals(false, relocator.canRelocatePath("org/foo/BarFactory2.class"))
+    }
+
     void testCanRelocateClass() {
         SimpleRelocator relocator