Skip to content

Update vendored dependencies (requests, certifi, tomli)#13882

Merged
sbidoul merged 3 commits intopypa:mainfrom
sbidoul:vendoring-update-2026-04-04-sbi
Apr 5, 2026
Merged

Update vendored dependencies (requests, certifi, tomli)#13882
sbidoul merged 3 commits intopypa:mainfrom
sbidoul:vendoring-update-2026-04-04-sbi

Conversation

@sbidoul
Copy link
Copy Markdown
Member

@sbidoul sbidoul commented Apr 4, 2026

No description provided.

@sbidoul sbidoul added this to the 26.1 milestone Apr 4, 2026
@sbidoul sbidoul force-pushed the vendoring-update-2026-04-04-sbi branch from 490294d to b4c5e65 Compare April 4, 2026 10:16
@sbidoul sbidoul force-pushed the vendoring-update-2026-04-04-sbi branch from b4c5e65 to 8128ce6 Compare April 4, 2026 10:48
@sbidoul
Copy link
Copy Markdown
Member Author

sbidoul commented Apr 4, 2026

Here I manually limited tomli to 2.3.1 (see also #13756)

@notatallshaw
Copy link
Copy Markdown
Member

I don't think we need to vendor in rich's vendoring of old Unicode databases, I skimmed over the issue a few weeks ago but didn't have time to verify.

@ichard26
Copy link
Copy Markdown
Member

ichard26 commented Apr 4, 2026

I'd much rather not upgrade rich than include whatever copy of some Unicode database rich has TBH.

@notatallshaw
Copy link
Copy Markdown
Member

I'd much rather not upgrade rich than include whatever copy of some Unicode database rich has TBH.

I don't think that's tenable long term, I think we should do two things: Break out rich vendoring from this PR, and open an issue with rich to make sure it makes sense for us to just not vendor the full old unicode databases and that won't cause issues long term.

@pradyunsg pradyunsg changed the title Update vendored depdendencies Update vendored dependencies Apr 4, 2026
@ichard26
Copy link
Copy Markdown
Member

ichard26 commented Apr 5, 2026

Well, yeah. I just don't think it's worth pushing to include an update to rich for pip 26.1.

@pfmoore
Copy link
Copy Markdown
Member

pfmoore commented Apr 5, 2026

I agree with @ichard26 here. I’m increasingly frustrated with the amount of bloat we are bringing in with rich, largely for features we do not use or do not care about. Maybe the rich project would be amenable to making it possible to get some sort of “minimal” version that excluded the expensive dependencies and features that depended on them?

But in the short term I think delaying the update while we consider our options is reasonable.

@sbidoul sbidoul force-pushed the vendoring-update-2026-04-04-sbi branch from 4ab1c07 to 434b6ac Compare April 5, 2026 09:03
@pradyunsg
Copy link
Copy Markdown
Member

pradyunsg commented Apr 5, 2026

Maybe the rich project would be amenable to making it possible to get some sort of “minimal” version that excluded the expensive dependencies and features that depended on them?

When I'd brought this up a few years ago, Will had mentioned that he would be open to it; if the standards supported something along the lines of default extras (so that he could have a "core" and non-core stuff around it).

@pfmoore
Copy link
Copy Markdown
Member

pfmoore commented Apr 5, 2026

Will had mentioned that he would be open to it; if the standards supported something along the lines of default extras

Yeah, that proposal appears to have stalled...

@dstufft
Copy link
Copy Markdown
Member

dstufft commented Apr 5, 2026

You can mimic that with a rich-core that has the minimal stuff and a rich that doesn't (and depends on rich-core), but I dunno how amenable he'd be to that.

@pfmoore
Copy link
Copy Markdown
Member

pfmoore commented Apr 5, 2026

You can mimic that with a rich-core that has the minimal stuff and a rich that doesn't

This is one of the alternative approaches that were suggested in the discussions around default extras. I don't think1 there were that many good arguments against it beyond various forms of "I don't want to". Maybe rich will be another project that simply doesn't want to. It's worth asking, though.

Footnotes

  1. I'd need to re-read the discussion to be sure, it's been a long time...

@notatallshaw
Copy link
Copy Markdown
Member

notatallshaw commented Apr 5, 2026

Well, yeah. I just don't think it's worth pushing to include an update to rich for pip 26.1.

Agreed, let's skip it for 26.1.

I'm increasingly frustrated with the amount of bloat we are bringing in with rich, largely for features we do not use or do not care about.

Two things I think we should be cognizant of: Rich is essentially a terminal rendering library, every rending engine-like tool I've followed (browsers, graphic drivers etc.) balloon in size over time as edge-case fixes and optimizations win out over disk savings. And commercial Textualize project is now defunct, so large scale refactors might be much less likely.

I'm not saying we shouldn't ask for changes, but I think bloat is a bit of an opinionated term, everything will have been added for a reason.

@pfmoore
Copy link
Copy Markdown
Member

pfmoore commented Apr 5, 2026

Agreed, sorry. It's bloat for us, not for rich. And that's at least in part because Python doesn't have good ways to only install parts of libraries that are needed (unless the library makes the effort to make that happen).

@sbidoul sbidoul force-pushed the vendoring-update-2026-04-04-sbi branch from 434b6ac to f2639aa Compare April 5, 2026 15:30
@sbidoul sbidoul changed the title Update vendored dependencies Update vendored dependencies (requests, certifi, tomli) Apr 5, 2026
@sbidoul sbidoul enabled auto-merge April 5, 2026 15:31
@sbidoul
Copy link
Copy Markdown
Member Author

sbidoul commented Apr 5, 2026

I've removed rich and pygments from this PR.

@sbidoul sbidoul merged commit 8df7b66 into pypa:main Apr 5, 2026
36 checks passed
@sbidoul sbidoul deleted the vendoring-update-2026-04-04-sbi branch April 5, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants