Closed Bug 1753683 Opened 3 years ago Closed 2 years ago

refactor python distutils usage

Categories

(Thunderbird :: Build Config, task)

Tracking

(thunderbird_esr102 fixed)

RESOLVED FIXED
104 Branch
Tracking Status
thunderbird_esr102 --- fixed

People

(Reporter: rjl, Assigned: dandarnell)

References

Details

Attachments

(1 file, 1 obsolete file)

DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives

Used in:

  • taskcluster/comm_taskgraph/init.py
  • python/thirdroc/thirdroc/rnp.py

Both cases are distutils.StrictVersion. The vendored "packaging" package is contains the replacement suggested by the PEP.

Assignee: nobody → daniel
Status: NEW → ASSIGNED
Attachment #9285947 - Attachment description: Bug 1753683 - Replace distutils (deprecated) with setuputils. r=rjl → Bug 1753683 - Replace distutils (deprecated) with packaging. r=rjl
Attachment #9285947 - Attachment is obsolete: true
Target Milestone: --- → 104 Branch

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/8346f3eaf302
Replace distutils (deprecated) with packaging. r=rjl

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

I suggest to uplift this to esr102, it's a dependency for the RNP v0.16.2 uplifts, avoids merge issues.
Any concerns?

Flags: needinfo?(rob)

Comment on attachment 9285954 [details]
Bug 1753683 - Replace distutils (deprecated) with packaging. r=rjl

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky): low

Attachment #9285954 - Flags: approval-comm-esr102?

(In reply to Kai Engert (:KaiE:) from comment #4)

I suggest to uplift this to esr102, it's a dependency for the RNP v0.16.2 uplifts, avoids merge issues.
Any concerns?

Should be okay. The "packaging" Python library is in m-e102:third_party/python/packaging just as it is on m-c.

Flags: needinfo?(rob)

Comment on attachment 9285954 [details]
Bug 1753683 - Replace distutils (deprecated) with packaging. r=rjl

[Triage Comment]
Approved for esr102

Attachment #9285954 - Flags: approval-comm-esr102? → approval-comm-esr102+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: