Open Bug 1699753 (goodbye-py2) Opened 3 years ago Updated 2 months ago

[meta] Unsupport Python 2 in mozilla-central

Categories

(Firefox Build System :: General, task, P3)

task

Tracking

(Not tracked)

People

(Reporter: ahal, Unassigned)

References

(Depends on 4 open bugs, Blocks 1 open bug)

Details

(Keywords: meta)

We're almost there!

A few tasks under this bug:

  1. Delete the py2/py3 linters
  2. Remove "py2commands" logic from mach driver
  3. Remove logic for a Python 2 virtualenv
  4. Remove uses of six and delete the vendored copy of it (this one will likely take a long time, but no rush).

Probably missing a few.

Some of this work (like the mach stuff) can proceed before we finish the ci-py3 metabug as it only impacts things locally.

Bravo :)
It might be an interesting experience to share in a blog post :)

Yes, I'll definitely publish a retrospective blog post when this is all done.

Blocks: py3

Let's make this a meta bug.. I think there will be a long tail of "Remove uses of six from X" bugs to be filed.

Keywords: meta
Summary: Unsupport Python 2 in mozilla-central → (meta) Unsupport Python 2 in mozilla-central
Summary: (meta) Unsupport Python 2 in mozilla-central → [meta] (meta) Unsupport Python 2 in mozilla-central
Summary: [meta] (meta) Unsupport Python 2 in mozilla-central → [meta] Unsupport Python 2 in mozilla-central
Depends on: 1700419
Depends on: 1700423
Depends on: 1700430
Depends on: 1700432
Priority: -- → P3

Callek found https://github.com/asottile/pyupgrade which can automatically remove six compatibility code as well as __future__ imports. It might be worth standing this up as a mozlint linter, but even if not, it'll likely be handy for folks to use manually as part of this bug.

Depends on: 1577908
Depends on: 1726573

The Bugbug bot thinks this bug should belong to the 'Firefox Build System::Mach Core' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → Mach Core
Component: Mach Core → General
Depends on: 1763930
Depends on: 1796839
Depends on: 1797277
Depends on: 1793668
You need to log in before you can comment on or make changes to this bug.