From acb53d0baa435b2f17be79fcfd0fa4c15d7cfc0c Mon Sep 17 00:00:00 2001 From: Mark Pilgrim Date: Thu, 30 Jul 2009 12:05:33 -0400 Subject: [PATCH] do you have a good reason? --- packaging.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packaging.html b/packaging.html index 8f248c9..a762e64 100644 --- a/packaging.html +++ b/packaging.html @@ -159,7 +159,7 @@ chardet/
from distutils.core import setup
-

This imports the setup() function, which is the main entry point into Distutils. 95% of all Distutils setup scripts consist of a single call to setup() and nothing else. (I totally just made up that statistic, but if your Distutils setup script is doing more than calling the Distutils setup() function, you should have a good reason.) +

This imports the setup() function, which is the main entry point into Distutils. 95% of all Distutils setup scripts consist of a single call to setup() and nothing else. (I totally just made up that statistic, but if your Distutils setup script is doing more than calling the Distutils setup() function, you should have a good reason. Do you have a good reason? I didn’t think so.)

The setup() function can take dozens of parameters. For the sanity of everyone involved, you must use named arguments for every parameter. This is not merely a convention; it’s a hard requirement. Your setup script will crash if you try to call the setup() function with non-named arguments.