[whimsy] branch master updated: self.add/remove no longer in use so drop them

[sebb]

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

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 74493df  self.add/remove no longer in use so drop them
74493df is described below

commit 74493df8459af64fbd5d883f469552b2bb696200
Author: Sebb <s...@apache.org>
AuthorDate: Sat Feb 9 14:38:56 2019 +0000

    self.add/remove no longer in use so drop them
---
 lib/whimsy/asf/ldap.rb | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)

diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 259cfb4..9ef2d24 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -1224,7 +1224,7 @@ module ASF
 
   class Committee < Base
     # TODO what to do about this? Change to ou=project or drop?
-    # It's used by the methods: self.list, self.preload, member[id]s, 
self.add, self.remove
+    # It's used by the methods: self.list, self.preload, member[id]s
     @base = 'ou=pmc,ou=committees,ou=groups,dc=apache,dc=org'
 
     # return a list of committees, from LDAP.
@@ -1356,21 +1356,6 @@ module ASF
       @members = nil
     end
 
-    # add a new committee to LDAP
-    def self.add(name, people)
-      entry = [
-        mod_add('objectClass', ['groupOfNames', 'top']),
-        mod_add('cn', name),
-        mod_add('member', Array(people).map(&:dn))
-      ]
-
-      ASF::LDAP.add("cn=#{name},#{base}", entry)
-    end
-
-    # remove a committee from LDAP
-    def self.remove(name)
-      ASF::LDAP.delete("cn=#{name},#{base}")
-    end
   end
 
   #