Log file Viewer

[14:07:28] <xafer> hello, regarding https://github.com/pypa/pip/pull/3088, any preference for https://packaging.python.org/en/latest/distributing.html#console-scripts or http://pythonhosted.org/setuptools/setuptools.html#automatic-script-creation ?

[17:10:09] <ErikRose> OTOH, I guess I'd have to write my own tests anyway, so...

[20:11:31] <xafer> What is usually done when there are two similar PRs (https://github.com/pypa/pip/pull/2812 and https://github.com/pypa/pip/pull/3084) ?

[20:12:00] <xafer> merge one and close the other or merge the two ?

[21:15:10] <xafer> dstufft, did you delete your comment on https://github.com/pypa/pip/pull/3085 ? cant find it anymore ^^

[21:21:39] <dstufft> xafer: it's on one of the collapsed things

[21:22:36] <xafer> found :)

[21:30:10] <xafer> btw, regarding https://github.com/pypa/pip/pull/3088, any preference between https://packaging.python.org/en/latest/distributing.html#console-scripts and http://pythonhosted.org/setuptools/setuptools.html#automatic-script-creation ?

[21:33:18] <xafer> thanks qwcode :)

[21:35:16] <xafer> good to merge ?

[21:41:02] <dstufft> xafer: pip_downloads is kinda ugly, idk maybe just "." would be better as a default (I hate the pip wheel default too)

[21:41:07] <dstufft> idk

[21:41:12] <dstufft> I'm happy either way

[21:41:13] <dstufft> just a nit

[21:47:39] <dstufft> ErikRose: random thought, consider --hash sha256=... instead of --sha256 ...., since this is primarily for the requirements.txt a little bit of extra verbosity should be ok, with --hash sha256=.... they'll be able to copy the line straight off of PyPI's /simple/ page and it doesn't pollute the CLI with a bunch of extra cli flags, just one

[21:50:50] <xafer> the issue with "." is that pip download can quickly create a bunch of files and I rarely want to dump dozens of files in my current directory

[21:51:22] <dstufft> heh

[21:51:31] <dstufft> I've never used pip install -d with anything but . I think

[21:51:37] <dstufft> maybe I'm the weirdo idk

[21:52:17] <dstufft> just thinking that most commands on the CLI default to "." or require an explict argument

[21:52:49] <ErikRose> dstufft: Thanks! I'll think about that in a bit.

[21:52:51] <dstufft> REALLY REALLY GOES NOW

[21:53:45] <xafer> I've never used install -d for other things than testing but with pip wheel I always had my wheelhouse configured in pip.conf

[21:53:53] <ErikRose> I think I'm going to get away without redoing all of download.py, so yay.

[21:54:56] <xafer> so maybe a middle ground is to remove the default and require a download-dir (either in CLI or pip.conf)

[21:55:09] <xafer> or just default to "." I dont know either ^^