From noreply at sourceforge.net Sat Jul 7 01:57:35 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 16:57:35 -0700 Subject: [ mailman-Bugs-1749431 ] Missing Japanese Message-ID: Bugs item #1749431, was opened at 2007-07-06 19:57 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing Japanese Initial Comment: I am attempting to install the latest Mailman from Source Forge. I have not specified any language but am hoping to get English. The "make install" failed with an error message similar to "cannot import japanise.SOMETHING" in the /usr/local/mailman/bin/paths.py file. I commented that line of misc/paths.py out (and also lines relating to korean anticipating problems there) and repeated "make install", which succeeded. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 From noreply at sourceforge.net Sat Jul 7 02:00:15 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 17:00:15 -0700 Subject: [ mailman-Bugs-1749432 ] Missing steps Message-ID: Bugs item #1749432, was opened at 2007-07-06 20:00 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing steps Initial Comment: The Installation Guide's step 4: O Run bin/check_perms -f is broken! It needs to be changed to: O cd /usr/local/mailman O Run bin/check_perms -f You also need to add the following missing step as per check_perms' complaint just after the new "cd" command above to satisfy check_perms: O chmod o-x archives/private ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 From noreply at sourceforge.net Sat Jul 7 02:32:20 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 17:32:20 -0700 Subject: [ mailman-Bugs-1749432 ] Missing steps Message-ID: Bugs item #1749432, was opened at 2007-07-06 17:00 Message generated for change (Comment added) made by msapiro You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None >Status: Pending Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing steps Initial Comment: The Installation Guide's step 4: O Run bin/check_perms -f is broken! It needs to be changed to: O cd /usr/local/mailman O Run bin/check_perms -f You also need to add the following missing step as per check_perms' complaint just after the new "cd" command above to satisfy check_perms: O chmod o-x archives/private ---------------------------------------------------------------------- >Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 17:32 Message: Logged In: YES user_id=1123998 Originator: NO It says "First change to the installation (i.e. $prefix) directory, then run the bin/check_perms program." That seems sufficient. Regarding "chmod o-x archives/private", you may or may not want to do that. I know check_perms says Warning: Private archive directory is other-executable (o+x). This could allow other users on your system to read private archives. If you're on a shared multiuser system, you should consult the installation manual on how to fix this. And it means what it says, i.e. see the "Warning" box in section 4 of the installation manual which tells you what you need to do if you want to protect private archives from access by other users on your system. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 From noreply at sourceforge.net Sat Jul 7 02:51:46 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 17:51:46 -0700 Subject: [ mailman-Bugs-1749432 ] Missing steps Message-ID: Bugs item #1749432, was opened at 2007-07-06 20:00 Message generated for change (Comment added) made by bob_toxen You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None >Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing steps Initial Comment: The Installation Guide's step 4: O Run bin/check_perms -f is broken! It needs to be changed to: O cd /usr/local/mailman O Run bin/check_perms -f You also need to add the following missing step as per check_perms' complaint just after the new "cd" command above to satisfy check_perms: O chmod o-x archives/private ---------------------------------------------------------------------- >Comment By: Bob_Toxen (bob_toxen) Date: 2007-07-06 20:51 Message: Logged In: YES user_id=1838295 Originator: YES May I humbly suggest that CRITICAL commands not be buried in text where they are easily missed by overworked SysAdmins. Thus, to improve the document add the following prior to the "Run bin/check_perms -f" line: O cd /usr/local/mailman ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 20:32 Message: Logged In: YES user_id=1123998 Originator: NO It says "First change to the installation (i.e. $prefix) directory, then run the bin/check_perms program." That seems sufficient. Regarding "chmod o-x archives/private", you may or may not want to do that. I know check_perms says Warning: Private archive directory is other-executable (o+x). This could allow other users on your system to read private archives. If you're on a shared multiuser system, you should consult the installation manual on how to fix this. And it means what it says, i.e. see the "Warning" box in section 4 of the installation manual which tells you what you need to do if you want to protect private archives from access by other users on your system. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749432&group_id=103 From noreply at sourceforge.net Sat Jul 7 02:55:40 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 17:55:40 -0700 Subject: [ mailman-Bugs-1749431 ] Missing Japanese Message-ID: Bugs item #1749431, was opened at 2007-07-06 16:57 Message generated for change (Comment added) made by msapiro You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing Japanese Initial Comment: I am attempting to install the latest Mailman from Source Forge. I have not specified any language but am hoping to get English. The "make install" failed with an error message similar to "cannot import japanise.SOMETHING" in the /usr/local/mailman/bin/paths.py file. I commented that line of misc/paths.py out (and also lines relating to korean anticipating problems there) and repeated "make install", which succeeded. ---------------------------------------------------------------------- >Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 17:55 Message: Logged In: YES user_id=1123998 Originator: NO If you tell us what your OS is and what your Python version is and exactly what error occurred and where, we may be able to help debug this. If you wish to follow up, the best place is the mailman-users at python.org list . I can tell you that configure and make install work for most people, but there have been reports of problems with some systems, but there is not enough information in this report for us to figure out what the problem is in this case. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 From noreply at sourceforge.net Sat Jul 7 03:08:56 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 18:08:56 -0700 Subject: [ mailman-Bugs-1749431 ] Missing Japanese Message-ID: Bugs item #1749431, was opened at 2007-07-06 19:57 Message generated for change (Comment added) made by bob_toxen You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing Japanese Initial Comment: I am attempting to install the latest Mailman from Source Forge. I have not specified any language but am hoping to get English. The "make install" failed with an error message similar to "cannot import japanise.SOMETHING" in the /usr/local/mailman/bin/paths.py file. I commented that line of misc/paths.py out (and also lines relating to korean anticipating problems there) and repeated "make install", which succeeded. ---------------------------------------------------------------------- >Comment By: Bob_Toxen (bob_toxen) Date: 2007-07-06 21:08 Message: Logged In: YES user_id=1838295 Originator: YES OS: GNU Linux (Mandreva 2007) with nothing special or unique Python 2.4.1 The exact error message is the one that Python generates when the line: import japanese is encountered on line 56 of file /usr/local/mailman/bin/paths.py because there is no japanese.* file. ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 20:55 Message: Logged In: YES user_id=1123998 Originator: NO If you tell us what your OS is and what your Python version is and exactly what error occurred and where, we may be able to help debug this. If you wish to follow up, the best place is the mailman-users at python.org list . I can tell you that configure and make install work for most people, but there have been reports of problems with some systems, but there is not enough information in this report for us to figure out what the problem is in this case. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 From noreply at sourceforge.net Sat Jul 7 03:46:58 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2007 18:46:58 -0700 Subject: [ mailman-Bugs-1749431 ] Missing Japanese Message-ID: Bugs item #1749431, was opened at 2007-07-06 16:57 Message generated for change (Comment added) made by msapiro You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: configuring/installing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Bob_Toxen (bob_toxen) Assigned to: Nobody/Anonymous (nobody) Summary: Missing Japanese Initial Comment: I am attempting to install the latest Mailman from Source Forge. I have not specified any language but am hoping to get English. The "make install" failed with an error message similar to "cannot import japanise.SOMETHING" in the /usr/local/mailman/bin/paths.py file. I commented that line of misc/paths.py out (and also lines relating to korean anticipating problems there) and repeated "make install", which succeeded. ---------------------------------------------------------------------- >Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 18:46 Message: Logged In: YES user_id=1123998 Originator: NO I understand that it comes from "import japanese" in paths.py, and I'm guessing that this came from the 'import paths' in bin/update, but I'd really like to know if you have a $prefix/pythonlib (where $prefix appears in your case from #1749432 to be /usr/local/mailman), and if you do, does it contain directories japanese/ and korean/, and if not, what did 'make install' say when it tried to build these? ---------------------------------------------------------------------- Comment By: Bob_Toxen (bob_toxen) Date: 2007-07-06 18:08 Message: Logged In: YES user_id=1838295 Originator: YES OS: GNU Linux (Mandreva 2007) with nothing special or unique Python 2.4.1 The exact error message is the one that Python generates when the line: import japanese is encountered on line 56 of file /usr/local/mailman/bin/paths.py because there is no japanese.* file. ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2007-07-06 17:55 Message: Logged In: YES user_id=1123998 Originator: NO If you tell us what your OS is and what your Python version is and exactly what error occurred and where, we may be able to help debug this. If you wish to follow up, the best place is the mailman-users at python.org list . I can tell you that configure and make install work for most people, but there have been reports of problems with some systems, but there is not enough information in this report for us to figure out what the problem is in this case. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1749431&group_id=103 From noreply at sourceforge.net Fri Jul 27 08:56:23 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Thu, 26 Jul 2007 23:56:23 -0700 Subject: [ mailman-Patches-1727084 ] invite_members command line Message-ID: Patches item #1727084, was opened at 2007-05-28 12:48 Message generated for change (Comment added) made by mylogon You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1727084&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: command line scripts Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alexis (alexistr) Assigned to: Nobody/Anonymous (nobody) Summary: invite_members command line Initial Comment: A script to invite new members from command line. Derived from add_members. I add "-l" option to choose the language of the invited user. To have de mail sent in the proper language small change are necessary in MailList.py function InviteNewMember: if userdesc.language: lang = userdesc.language else: lang = self.preferred_language ----------------- Utils.maketext(.............lang=lang) ----------------- Message.UserNotification(...............lang=lang) Could be nice to add this as an option in add_members. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-26 23:56 Message: Logged In: YES user_id=1042734 Originator: NO Would be real nice if could prepend a welcome message - so would be a one time thing - as on the web form. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1727084&group_id=103 From noreply at sourceforge.net Fri Jul 27 09:30:36 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 27 Jul 2007 00:30:36 -0700 Subject: [ mailman-Patches-1727084 ] invite_members command line Message-ID: Patches item #1727084, was opened at 2007-05-28 12:48 Message generated for change (Comment added) made by mylogon You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1727084&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: command line scripts Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alexis (alexistr) Assigned to: Nobody/Anonymous (nobody) Summary: invite_members command line Initial Comment: A script to invite new members from command line. Derived from add_members. I add "-l" option to choose the language of the invited user. To have de mail sent in the proper language small change are necessary in MailList.py function InviteNewMember: if userdesc.language: lang = userdesc.language else: lang = self.preferred_language ----------------- Utils.maketext(.............lang=lang) ----------------- Message.UserNotification(...............lang=lang) Could be nice to add this as an option in add_members. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-27 00:30 Message: Logged In: YES user_id=1042734 Originator: NO I see someone else already did. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-26 23:56 Message: Logged In: YES user_id=1042734 Originator: NO Would be real nice if could prepend a welcome message - so would be a one time thing - as on the web form. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1727084&group_id=103 From noreply at sourceforge.net Fri Jul 27 10:15:45 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 27 Jul 2007 01:15:45 -0700 Subject: [ mailman-Patches-1611855 ] Add invite option to add_members script Message-ID: Patches item #1611855, was opened at 2006-12-08 13:31 Message generated for change (Comment added) made by mylogon You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: command line scripts Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Abram Thielke (athielke) Assigned to: Nobody/Anonymous (nobody) Summary: Add invite option to add_members script Initial Comment: This will add two options to the add_members script: --invite-msg-file=file -m file This will prepend the message in the file to the invite email that gets generated. --invite= -i Set to yes if you only want to invite the users to a list, instead of subscribing them. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-27 01:15 Message: Logged In: YES user_id=1042734 Originator: NO It invites everyone - including people who are members for me. It totally ignores the except Errors.MMAlreadyAMember: ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103 From noreply at sourceforge.net Fri Jul 27 14:50:56 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 27 Jul 2007 05:50:56 -0700 Subject: [ mailman-Patches-1740331 ] New translation: Slovak Message-ID: Patches item #1740331, was opened at 2007-06-20 15:06 Message generated for change (Comment added) made by rebum You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: internationalization Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Martin Matuska (rebum) Assigned to: Nobody/Anonymous (nobody) Summary: New translation: Slovak Initial Comment: The attached patch adds the slovak translation to Mailman 2.1.9. The included .po file is from SVN trunk. I would like to become the language champion for the Slovak language. Thank you, Martin Matuska ---------------------------------------------------------------------- >Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:50 Message: Logged In: YES user_id=41220 Originator: YES File Added: mailman.po.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-06-20 18:07 Message: Logged In: YES user_id=41220 Originator: YES Updated translation and single mailman.po + templates can be found under: http://www.matuska.org/martin/mailman ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 From noreply at sourceforge.net Fri Jul 27 14:52:27 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 27 Jul 2007 05:52:27 -0700 Subject: [ mailman-Patches-1740331 ] New translation: Slovak Message-ID: Patches item #1740331, was opened at 2007-06-20 15:06 Message generated for change (Comment added) made by rebum You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: internationalization Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Martin Matuska (rebum) Assigned to: Nobody/Anonymous (nobody) Summary: New translation: Slovak Initial Comment: The attached patch adds the slovak translation to Mailman 2.1.9. The included .po file is from SVN trunk. I would like to become the language champion for the Slovak language. Thank you, Martin Matuska ---------------------------------------------------------------------- >Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:52 Message: Logged In: YES user_id=41220 Originator: YES File Added: templates.tar.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:50 Message: Logged In: YES user_id=41220 Originator: YES File Added: mailman.po.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-06-20 18:07 Message: Logged In: YES user_id=41220 Originator: YES Updated translation and single mailman.po + templates can be found under: http://www.matuska.org/martin/mailman ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 From noreply at sourceforge.net Fri Jul 27 14:53:05 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 27 Jul 2007 05:53:05 -0700 Subject: [ mailman-Patches-1740331 ] New translation: Slovak Message-ID: Patches item #1740331, was opened at 2007-06-20 15:06 Message generated for change (Comment added) made by rebum You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: internationalization Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Martin Matuska (rebum) Assigned to: Nobody/Anonymous (nobody) Summary: New translation: Slovak Initial Comment: The attached patch adds the slovak translation to Mailman 2.1.9. The included .po file is from SVN trunk. I would like to become the language champion for the Slovak language. Thank you, Martin Matuska ---------------------------------------------------------------------- >Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:53 Message: Logged In: YES user_id=41220 Originator: YES File Added: sk-2.1.9-0.1.patch.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:52 Message: Logged In: YES user_id=41220 Originator: YES File Added: templates.tar.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-07-27 14:50 Message: Logged In: YES user_id=41220 Originator: YES File Added: mailman.po.gz ---------------------------------------------------------------------- Comment By: Martin Matuska (rebum) Date: 2007-06-20 18:07 Message: Logged In: YES user_id=41220 Originator: YES Updated translation and single mailman.po + templates can be found under: http://www.matuska.org/martin/mailman ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1740331&group_id=103 From noreply at sourceforge.net Sun Jul 29 03:32:30 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Sat, 28 Jul 2007 18:32:30 -0700 Subject: [ mailman-Patches-1611855 ] Add invite option to add_members script Message-ID: Patches item #1611855, was opened at 2006-12-08 13:31 Message generated for change (Comment added) made by msapiro You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: command line scripts Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Abram Thielke (athielke) >Assigned to: Mark Sapiro (msapiro) Summary: Add invite option to add_members script Initial Comment: This will add two options to the add_members script: --invite-msg-file=file -m file This will prepend the message in the file to the invite email that gets generated. --invite= -i Set to yes if you only want to invite the users to a list, instead of subscribing them. ---------------------------------------------------------------------- >Comment By: Mark Sapiro (msapiro) Date: 2007-07-28 18:32 Message: Logged In: YES user_id=1123998 Originator: NO Actually, the patch looks quite good. There are some problems but they aren't problems with the patch. The problem with inviting someone who is already a member is because MailList.InviteNewMember() does not test for membership before inviting. The patch would catch Errors.MMAlreadyAMember just as is done for adding, but the exception isn't thrown. The web Mailman/Cgi/admin.py gets around this by doing the test itself before inviting, but I think the proper thing is for InviteNewMember() to do it. There is another problem in that InviteNewMember() can throw an Errors.MembershipIsBanned exception which is not caught, but this problem is in the base add_members, not the patch. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-27 01:15 Message: Logged In: YES user_id=1042734 Originator: NO It invites everyone - including people who are members for me. It totally ignores the except Errors.MMAlreadyAMember: ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103 From noreply at sourceforge.net Sun Jul 29 05:47:08 2007 From: noreply at sourceforge.net (SourceForge.net) Date: Sat, 28 Jul 2007 20:47:08 -0700 Subject: [ mailman-Patches-1611855 ] Add invite option to add_members script Message-ID: Patches item #1611855, was opened at 2006-12-08 13:31 Message generated for change (Comment added) made by mylogon You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: command line scripts Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Abram Thielke (athielke) Assigned to: Mark Sapiro (msapiro) Summary: Add invite option to add_members script Initial Comment: This will add two options to the add_members script: --invite-msg-file=file -m file This will prepend the message in the file to the invite email that gets generated. --invite= -i Set to yes if you only want to invite the users to a list, instead of subscribing them. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-28 20:47 Message: Logged In: YES user_id=1042734 Originator: NO As a quick change to that section: try: if invite: if mlist.isMember(member): raise Errors.MMAlreadyAMember print >> tee, _('Already a member: %(member)s') else: mlist.InviteNewMember(userdesc, invite_msg) else: mlist.ApprovedAddMember(userdesc, ack, 0) This works - but still not for banned. ---------------------------------------------------------------------- Comment By: Mark Sapiro (msapiro) Date: 2007-07-28 18:32 Message: Logged In: YES user_id=1123998 Originator: NO Actually, the patch looks quite good. There are some problems but they aren't problems with the patch. The problem with inviting someone who is already a member is because MailList.InviteNewMember() does not test for membership before inviting. The patch would catch Errors.MMAlreadyAMember just as is done for adding, but the exception isn't thrown. The web Mailman/Cgi/admin.py gets around this by doing the test itself before inviting, but I think the proper thing is for InviteNewMember() to do it. There is another problem in that InviteNewMember() can throw an Errors.MembershipIsBanned exception which is not caught, but this problem is in the base add_members, not the patch. ---------------------------------------------------------------------- Comment By: mylogon (mylogon) Date: 2007-07-27 01:15 Message: Logged In: YES user_id=1042734 Originator: NO It invites everyone - including people who are members for me. It totally ignores the except Errors.MMAlreadyAMember: ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1611855&group_id=103