Patches item #1422385, was opened at 2006-02-02 03:26 Message generated for change (Comment added) made by nnorwitz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1422385&group_id=5470
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: Modules Group: Python 2.4 Status: Closed Resolution: Accepted Priority: 5 Submitted By: Ben Bell (csuwi) Assigned to: Nobody/Anonymous (nobody) Summary: Changes to nis module to support multiple NIS domains Initial Comment: The nis module currently has no way to access nis domains other than the default NIS domain. This is fairly easy to achieve via the C API and so I've patched it. Summary of changes: I've added a new get_default_domain() function and exposed it to python. I've added an optional "domain" keyword arg to the cat, maps and match arguments but defaulted it to the default domain to ensure backwards compatibility. ---------------------------------------------------------------------- >Comment By: Neal Norwitz (nnorwitz) Date: 2006-02-04 14:45 Message: Logged In: YES user_id=33168 Ben, could you also update Lib/test/test_nis.py to test the new code? ---------------------------------------------------------------------- Comment By: Martin v. Löwis (loewis) Date: 2006-02-04 11:14 Message: Logged In: YES user_id=21627 Thanks for the patch, committed as revision 42235. In the future, please remember the following details: - use PyDoc_STRVAR for documentation variables - add changes to the tex documentation as well ---------------------------------------------------------------------- Comment By: Ben Bell (csuwi) Date: 2006-02-04 07:32 Message: Logged In: YES user_id=145982 Sorry, uploaded this time ---------------------------------------------------------------------- Comment By: Martin v. Löwis (loewis) Date: 2006-02-04 03:34 Message: Logged In: YES user_id=21627 Can you please attach the new patch to the report? ---------------------------------------------------------------------- Comment By: Ben Bell (csuwi) Date: 2006-02-02 04:02 Message: Logged In: YES user_id=145982 The previous patch was a little lazy. I've now taken the opportunity to add module doc and fix the two compile time warnings as well. New diff attached. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1422385&group_id=5470 _______________________________________________ Patches mailing list [email protected] http://mail.python.org/mailman/listinfo/patches
