Skip to content

Switch to using the other style of environment markers#3796

Merged
reaperhulk merged 5 commits into
pyca:masterfrom
alex:other-style-environment-markers
Aug 24, 2017
Merged

Switch to using the other style of environment markers#3796
reaperhulk merged 5 commits into
pyca:masterfrom
alex:other-style-environment-markers

Conversation

@alex

@alex alex commented Jul 19, 2017

Copy link
Copy Markdown
Member

No description provided.

@alex

alex commented Jul 19, 2017

Copy link
Copy Markdown
Member Author

Note to self: branch-free setup.py is a prerequisite for shipping py3-stable-abi wheels.

@alex

alex commented Jul 19, 2017

Copy link
Copy Markdown
Member Author

@dstufft uhhh, any idea why this would result in a package not getting installed under setup.py test?

@dstufft

dstufft commented Jul 19, 2017

Copy link
Copy Markdown
Member

I know very little about setup.py test and how it resolved dependencies.

@alex

alex commented Jul 19, 2017

Copy link
Copy Markdown
Member Author

@jaraco -- @dstufft suggested you might have some idea why setup.py test is not installing things from extra_requires with environment markers?

@alex alex modified the milestone: Twenty first release Jul 23, 2017
@alex

alex commented Aug 1, 2017

Copy link
Copy Markdown
Member Author

Chart of "Percent of cryptography installs with pip that are pip == 1.x by month":
Image

@alex

alex commented Aug 2, 2017

Copy link
Copy Markdown
Member Author

@reaperhulk looking at the first 2 days of the month of august, we're down to 5.9% of installs being with pip 1.x

@alex

alex commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

Now with a week of data, it's down to 5.5%:

SELECT
  STRFTIME_UTC_USEC(timestamp, "%Y-%m") AS yyyymm,
  ROUND(100 * SUM(CASE WHEN REGEXP_MATCH(details.installer.version, r"^1\.") THEN 1 ELSE 0 END) / COUNT(*), 1) AS percent_pip1,
  COUNT(*) as download_count
FROM
  TABLE_DATE_RANGE(
    [the-psf:pypi.downloads],
    DATE_ADD(CURRENT_TIMESTAMP(), -1, "year"),
    CURRENT_TIMESTAMP()
  )
WHERE
  details.installer.name == "pip" AND file.project == "cryptography"
GROUP BY
  yyyymm
ORDER BY
  yyyymm DESC
LIMIT 100

@reaperhulk

Copy link
Copy Markdown
Member

What does this do to users who install via setuptools rather than via pip?

@alex

alex commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

setuptools version mumble mumble and above work fine.

@alex

alex commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

(IIRC, it's version 0.7, incidentally the same version that started verifying shit in TLS, so I don't care about 0.6 at all)

@alex

alex commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

@reaperhulk

Copy link
Copy Markdown
Member

So the setuptools version is actually substantially older than the pip version. 4 years to pip 6's coming-up-on-3

@alex

alex commented Aug 8, 2017

Copy link
Copy Markdown
Member Author

I believe that's correct, yes.

@alex

alex commented Aug 10, 2017

Copy link
Copy Markdown
Member Author

@reaperhulk breaking setuptools 0.6 is arguably an important security measure :-)

@reaperhulk

Copy link
Copy Markdown
Member

Running that query gives 6.4% for August now. I think I'm convinced, especially since this won't affect setuptools unless you're on a ludicrously old version and all that happens in the worst case is some deps don't get installed.

@reaperhulk reaperhulk merged commit 4c41ab0 into pyca:master Aug 24, 2017
@alex alex deleted the other-style-environment-markers branch August 24, 2017 00:33
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants