setup sub-command without any argument error
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Buildout |
Fix Committed
|
Undecided
|
Baiju Muthukadan |
Bug Description
"buildout setup" command raise an error like this:
An internal error occured due to a bug in either zc.buildout or in a
recipe being used:
IndexError:
pop from empty list
-------
This error message should be improved. Here is a patch (please change message, if required):
Index: src/zc/
=======
--- src/zc/
+++ src/zc/
@@ -725,6 +725,10 @@
def setup(self, args):
+ if not args:
+ raise zc.buildout.
+ "setup command expects one argument.\n"
+ )
setup = args.pop(0)
if os.path.
setup = os.path.join(setup, 'setup.py')
On Oct 5, 2007, at 5:54 AM, Baiju Muthukadan wrote:
> Public bug reported: ------- ------- ----- buildout/ buildout. py ======= ======= ======= ======= ======= ======= ======= ======= ==== buildout/ buildout. py (revision 80625) buildout/ buildout. py (working copy) UserError(
>
> "buildout setup" command raise an error like this:
>
> An internal error occured due to a bug in either zc.buildout or in a
> recipe being used:
>
> IndexError:
> pop from empty list
>
> -------
>
> This error message should be improved. Here is a patch (please change
> message, if required):
>
>
> Index: src/zc/
> =======
> --- src/zc/
> +++ src/zc/
> @@ -725,6 +725,10 @@
> ep.load()(self)
>
> def setup(self, args):
> + if not args:
> + raise zc.buildout.
> + "setup command expects one argument.\n"
"one or more arguments".
Please feel free to go ahead and apply this. :)
Thanks.
Jim
--
Jim Fulton
Zope Corporation