-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release 19.0 -- New year, New release #6106
Comments
This one is a release with some heavy weight issues given that PEP 517 and manylinux2010 support are a part of it. There's also a bunch of "nicer error message" and VCS improvements here. There's enough to justify a release here, clearly. I'm willing to be the RM for this release. A quick question to @pfmoore, do you reckon PEP 517 should spend some time in a "beta" phase before being released? |
If you mean do we need a prerelease, I don't think so. It's not something we've done that often in the past, and the PEP 517 changes should default back to legacy processing, so I'd be OK just going for a normal release. In my experience, very few people actually test the betas anyway. I'm not against a beta (I think it would be better if we did have a standard process of doing prereleases) but I don't think the PEP 517 changes warrant one on their own. |
That's what I meant. We do have a standard process in terms of pre-releases in release cycles - if there's a pre-release, the main release is pushed ahead a month. It's documented in our release cadence docs. |
What I meant was that I think we should routinely do prereleases, but thinking more about it, with releases every 3 months, prereleases are too much additional burden (and it's not like we'd get that much additional testing if we did do prereleases...) So just ignore that aside of mine. |
While I think about it, it would be nice if a new packaging version could be released and vendored with pypa/packaging@eb02438 |
I can do a release of packaging - assuming @dstufft doesn't mind that, and merging a PR or giving me push access. :) |
And we might want to add a deprecation warning for Python 3.4 support ? |
I don't think we need a deprecation warning for that. We usually drop versions when the usage is low on PyPI. I haven't run the bigquery SQL query to check what the numbers are, to facilitate us deciding that. |
According to PyPI Stats, Python 3.4 represents ~3% of downloads. If we plan to drop support for 3.4 in 19.1, we'll want a deprecation warning in 19.0 (à la 90b3db4#diff-84e592bd8e56a3c31ef7f6908753e4feL201). |
+1 for dropping 3.4 in 19.1 and warning now. |
Cool; let's do that. |
In #6123, @pradyunsg makes a good point on our usual 6 months deprecation period (#6123 (comment)) so we might want to drop support in 19.2 ? cc @pfmoore |
+1 that's a very good point |
Super quick update -- there's 2 release blockers right now:
I intend to make a release on 22nd/23rd Jan - of course conditional to not having any blockers by then. If there are any concerns regarding what all is in the release, please flag them before/during the weekend. |
@pradyunsg I noticed that a number of NEWS entries use single backticks to mark monospace when they should be double. So the NEWS entries should be reviewed before release. |
Thanks @cjerdonek! I'll take care of that when cutting the release. :) |
Got this. Will do today or tomorrow. |
#6147 is a new thing -- making it a part of this release. Other than that and the packaging release/vendoring, if there are any changes, please ping here before merging PRs into master. |
@pradyunsg, may I merge PR #6142? I thoroughly reviewed it a number of times, and it is well-tested. /cc @fischman |
Yeah sure, let's get that in as well. :) |
Alrighty then. That should be all for now; I'll let things stay settled for a couple of days just in case there's something else that might come up -- plus I have "mid" semester examinations -- so I intend to stick to the 22/23 Jan release date. |
@pradyunsg I added PR #6153 to the release milestone. It's a follow-up to the broken stdout pipe PR #5907. I noticed a tiny issue that occurs when a broken pipe happens when |
Made the release. Pushed change to |
Reopened master for active development of 19.1. Thanks @pypa/pip-committers for letting me lock down master multiple times in this release cycle to avoid cherry-picking commits. :) I don't think there's any need for a 19.0.3, unless someone files a show stopper bug in the next 3-4 days. Keeping this issue open for that case only. Everything else can go into pip 19.1 development. |
https://pypi.org/project/virtualenv/16.4.0/ adopted it. |
Any major-ish bugs reported for 19.0.2? |
There's at least one regression from 19.0.1 to 19.0.2 (#6252), but I'm not sure how widely it will affect people. (Note: I think I might want to make one tiny tweak to my fix that was committed, which I can discuss separately.) There's also #6255 which is PEP 517-related, but it looks like it can be fixed strictly in setuptools? |
I think that's pretty small impact bug and it's a case of better error messaging -- something I think we should aim to do for more cases related to PEP 517 in 19.1. This doesn't warrant a 19.0.3 IMO.
Yea, that's a setuptools bug. |
No, it's not if you read the opening comment. Their build works in 19.0.1 but not in 19.0.2. It's not just better messaging because in 19.0.1 the error is caught, but in 19.0.2 it bubbles up and crashes pip. |
Hmm... Fair enough. I think I read that wrong. I thought pip printed an IndexError and continued. Alrighty then, that'll be a 19.0.3. I don't think there any other new issues so I'll probably make that release too; later tonight. |
Ah well, that didn't happen because a RL issue came up. I'll do it in the next 24 hours. |
@pradyunsg so what do we think about 19.0.3? |
Someone needs to diagnose #6262 at some point. |
I thought have commented here already - There's more time needed to figure out existing bugs filed for regressions from 19.0.2. Putting off the 19.0.3 release until we figure that out. Sorry for the false alarm on the release plan. |
That doesn't seem like a pip issue to me tbh. |
@pradyunsg Are there any remaining potential 19.x bugs you know of? I haven't seen any myself, and I've been keeping an eye out. |
Nope. I've mostly been busy with some prototyping lately. I think we're golden for 19.0.3 now. |
Cool, cutting a 19.0.3 in the next 24 hours, from the current master because I'm lucky and no one merged any non 19.0.3 bug fixes. ;) |
I've started with the release engineering for 19.0.3, from current master. |
Made the release. Updated get-pip.py.
s/19.0.3/19.0.4/ |
Alrighty. That closes this release cycle. I'll file the CPython PR in a bit. Thanks to everyone who helped out in the form of PRs, discussion or any other contributions in this release cycle! :D |
### Update [pip](https://pypi.org/project/pip) from **18.1** to **19.0.1**. <details> <summary>Changelog</summary> ### 19.0 ``` ================= Deprecations and Removals ------------------------- - Deprecate support for Python 3.4 (`6106 <https://github.com/pypa/pip/issues/6106>`_) - Start printing a warning for Python 2.7 to warn of impending Python 2.7 End-of-life and prompt users to start migrating to Python 3. (`6148 <https://github.com/pypa/pip/issues/6148>`_) - Remove the deprecated ``--process-dependency-links`` option. (`6060 <https://github.com/pypa/pip/issues/6060>`_) - Remove the deprecated SVN editable detection based on dependency links during freeze. (`5866 <https://github.com/pypa/pip/issues/5866>`_) Features -------- - Implement PEP 517 (allow projects to specify a build backend via pyproject.toml). (`5743 <https://github.com/pypa/pip/issues/5743>`_) - Implement manylinux2010 platform tag support. manylinux2010 is the successor to manylinux1. It allows carefully compiled binary wheels to be installed on compatible Linux platforms. (`5008 <https://github.com/pypa/pip/issues/5008>`_) - Improve build isolation: handle ``.pth`` files, so namespace packages are correctly supported under Python 3.2 and earlier. (`5656 <https://github.com/pypa/pip/issues/5656>`_) - Include the package name in a freeze warning if the package is not installed. (`5943 <https://github.com/pypa/pip/issues/5943>`_) - Warn when dropping an ``--[extra-]index-url`` value that points to an existing local directory. (`5827 <https://github.com/pypa/pip/issues/5827>`_) - Prefix pip's ``--log`` file lines with their timestamp. (`6141 <https://github.com/pypa/pip/issues/6141>`_) Bug Fixes --------- - Avoid creating excessively long temporary paths when uninstalling packages. (`3055 <https://github.com/pypa/pip/issues/3055>`_) - Redact the password from the URL in various log messages. (`4746 <https://github.com/pypa/pip/issues/4746>`_, `6124 <https://github.com/pypa/pip/issues/6124>`_) - Avoid creating excessively long temporary paths when uninstalling packages. (`3055 <https://github.com/pypa/pip/issues/3055>`_) - Avoid printing a stack trace when given an invalid requirement. (`5147 <https://github.com/pypa/pip/issues/5147>`_) - Present 401 warning if username/password do not work for URL (`4833 <https://github.com/pypa/pip/issues/4833>`_) - Handle ``requests.exceptions.RetryError`` raised in ``PackageFinder`` that was causing pip to fail silently when some indexes were unreachable. (`5270 <https://github.com/pypa/pip/issues/5270>`_, `5483 <https://github.com/pypa/pip/issues/5483>`_) - Handle a broken stdout pipe more gracefully (e.g. when running ``pip list | head``). (`4170 <https://github.com/pypa/pip/issues/4170>`_) - Fix crash from setting ``PIP_NO_CACHE_DIR=yes``. (`5385 <https://github.com/pypa/pip/issues/5385>`_) - Fix crash from unparseable requirements when checking installed packages. (`5839 <https://github.com/pypa/pip/issues/5839>`_) - Fix content type detection if a directory named like an archive is used as a package source. (`5838 <https://github.com/pypa/pip/issues/5838>`_) - Fix listing of outdated packages that are not dependencies of installed packages in ``pip list --outdated --not-required`` (`5737 <https://github.com/pypa/pip/issues/5737>`_) - Fix sorting ``TypeError`` in ``move_wheel_files()`` when installing some packages. (`5868 <https://github.com/pypa/pip/issues/5868>`_) - Fix support for invoking pip using ``python src/pip ...``. (`5841 <https://github.com/pypa/pip/issues/5841>`_) - Greatly reduce memory usage when installing wheels containing large files. (`5848 <https://github.com/pypa/pip/issues/5848>`_) - Editable non-VCS installs now freeze as editable. (`5031 <https://github.com/pypa/pip/issues/5031>`_) - Editable Git installs without a remote now freeze as editable. (`4759 <https://github.com/pypa/pip/issues/4759>`_) - Canonicalize sdist file names so they can be matched to a canonicalized package name passed to ``pip install``. (`5870 <https://github.com/pypa/pip/issues/5870>`_) - Properly decode special characters in SVN URL credentials. (`5968 <https://github.com/pypa/pip/issues/5968>`_) - Make ``PIP_NO_CACHE_DIR`` disable the cache also for truthy values like ``"true"``, ``"yes"``, ``"1"``, etc. (`5735 <https://github.com/pypa/pip/issues/5735>`_) Vendored Libraries ------------------ - Include license text of vendored 3rd party libraries. (`5213 <https://github.com/pypa/pip/issues/5213>`_) - Update certifi to 2018.11.29 - Update colorama to 0.4.1 - Update distlib to 0.2.8 - Update idna to 2.8 - Update packaging to 19.0 - Update pep517 to 0.5.0 - Update pkg_resources to 40.6.3 (via setuptools) - Update pyparsing to 2.3.1 - Update pytoml to 0.1.20 - Update requests to 2.21.0 - Update six to 1.12.0 - Update urllib3 to 1.24.1 Improved Documentation ---------------------- - Include the Vendoring Policy in the documentation. (`5958 <https://github.com/pypa/pip/issues/5958>`_) - Add instructions for running pip from source to Development documentation. (`5949 <https://github.com/pypa/pip/issues/5949>`_) - Remove references to removed ``egg=<name>-<version>`` functionality (`5888 <https://github.com/pypa/pip/issues/5888>`_) - Fix omission of command name in HTML usage documentation (`5984 <https://github.com/pypa/pip/issues/5984>`_) ``` </details> <details> <summary>Links</summary> - PyPI: https://pypi.org/project/pip - Changelog: https://pyup.io/changelogs/pip/ - Homepage: https://pip.pypa.io/ </details> ### Update [tox](https://pypi.org/project/tox) from **3.6.1** to **3.7.0**. *The bot wasn't able to find a changelog for this release. [Got an idea?](https://github.com/pyupio/changelogs/issues/new)* <details> <summary>Links</summary> - PyPI: https://pypi.org/project/tox - Docs: https://tox.readthedocs.org/ </details> ### Update [PyYAML](https://pypi.org/project/PyYAML) from **4.2b1** to **4.2b4**. *The bot wasn't able to find a changelog for this release. [Got an idea?](https://github.com/pyupio/changelogs/issues/new)* <details> <summary>Links</summary> - PyPI: https://pypi.org/project/pyyaml - Homepage: http://pyyaml.org/wiki/PyYAML </details>
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Yet another issue for discussing yet another release. :)
The text was updated successfully, but these errors were encountered: