Closed Bug 1344891 Opened 7 years ago Closed 7 years ago

add BUG_COMPONENT to python/* files

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox55 fixed)

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jmaher, Assigned: jmaher)

References

Details

Attachments

(1 file)

choose build config as a root for this since it is the largest consumer of these packages.
here is what we have with this patch applied:
$ ./mach file-info bugzilla-component python/**
Core :: Build Config
  python/README
  python/blessings
  python/configobj
  python/dlmanager
  python/futures
  python/gdbpp
  python/mach_commands.py
  python/mach_commands.pyc
  python/mock-1.0.0
  python/moz.build
  python/mozboot
  python/mozbuild
  python/psutil
  python/py
  python/pytest
  python/pytoml
  python/redo
  python/requests
  python/virtualenv
  python/which
Core :: General
  python/lldbutils
Core :: Security: PSM
  python/PyECC
  python/pyasn1-modules
  python/rsa
Core :: mach
  python/mach
  python/mozversioncontrol
Firefox :: Developer Tools
  python/devtools
Firefox for Android :: Build Config & IDE Support
  python/jsmin
Localization Infrastructure and Tools :: compare-locales
  python/compare-locales
Release Engineering :: General Automation
  python/altgraph
  python/bitstring
  python/eme
  python/macholib
  python/pyasn1
Taskcluster :: General
  python/pystache
  python/pyyaml
Taskcluster :: Platform Libraries
  python/slugid
Taskcluster :: Task Configuration
  python/voluptuous
Testing :: Lint
  python/mozlint
mozilla.org :: MozillaBuild
  python/pylru
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8844154 - Flags: review?(ted)
Comment on attachment 8844154 [details] [diff] [review]
add BUG_COMPONENT to python/*

Review of attachment 8844154 [details] [diff] [review]:
-----------------------------------------------------------------

Kinda arbitrary, but OK.

::: python/moz.build
@@ +65,5 @@
> +with Files('mozlint/**'):
> +    BUG_COMPONENT = ('Testing', 'Lint')
> +
> +with Files('mozversioncontrol/**'):
> +    BUG_COMPONENT = ('Core', 'mach')

I think Core :: Build Config would be better.

@@ +86,5 @@
> +with Files('pystache/**'):
> +    BUG_COMPONENT = ('Taskcluster', 'General')
> +
> +with Files('pytest/**'):
> +    BUG_COMPONENT = ('Core', 'Build Config')

Maybe Testing :: General instead?
Attachment #8844154 - Flags: review?(ted) → review+
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d63722ec2a6
add BUG_COMPONENT to python/* files, r=ted
https://hg.mozilla.org/mozilla-central/rev/7d63722ec2a6
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: