gh-122631 Add video/3gpp and video/3gpp2 to mimetypes

[robotmlg]

These are officially recognized by IANA

• video/3gpp https://www.iana.org/assignments/media-types/video/3gpp
• video/3gpp2 https://www.iana.org/assignments/media-types/video/3gpp2

• Issue: Add MIME types for video/3gpp and video/3gpp2 #122631

[ghost]

All commit authors signed the Contributor License Agreement.

[bedevere-app]

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

[AA-Turner]

As you note, the audio/ versions already exist:

cpython/Lib/mimetypes.py

Lines 527 to 530 in efcd65c

	'.3gp' : 'audio/3gpp',
	'.3gpp' : 'audio/3gpp',
	'.3g2' : 'audio/3gpp2',
	'.3gpp2' : 'audio/3gpp2',

The suffixes are the same, though, so this would overwrite as they appear later in the dict. Can you think of a way to fix that?

A

[bedevere-app]

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

[robotmlg]

@AA-Turner I have made the requested changes; please review again.

I think ideally types_map would also be a dict[str, list[str]] like types_map_inv is, so that every extension would map to a list of types. But that would be a pretty big breaking change for my first PR here (since types_map gets exported), lemme know if you think I should tackle that instead of my more band-aid solution here.

(Also I'm not sure what's going on with my NEWS build errors, would appreciate any pointers there.)

EDIT: I put up a PR on my fork of how I would do the breaking version. I do think it's cleaner, and more future-proof, but obviously it's a breaking change on types_map and presumably we'd have to go through channels for that. Lemme know your thoughts. robotmlg#1

[bedevere-app]

Thanks for making the requested changes!

@AA-Turner: please review the changes made to this pull request.