Petri Lehtinen <pe...@digip.org> added the comment:

> > In the Doc/includes/sqlite3 directory there are still some scripts that 
> > require the
> > createdb.py, so I didn't nuke the file.
>
> Do these scripts (or a README file) explain about createdb.py?

No. There's no mention of createdb.py anywhere.

> > I also removed the strange "The arguments to :meth:`executescript` must be 
> > :func:`bytes`
> > objects" sentence from the patch. I assume it was a mistake, as it's wrong.
>
> Why not change it so that it’s correct?  (Maybe looking at the file history 
> would
> show where the mistake comes from.)

It's just not in the history, it was only in OP's patch. And it's wrong, as 
executescript() takes an str argument, not bytes.

Having slept over this, I think execute_1.py and execute_2.py should be merged 
as one file, as after my patch, execute_2.py would not be runnable by itself.

This would also be a good chance to look at all the examples and have them 
cleaned up. That could also be a separate patch.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue13491>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to