1332423 - add BUG_COMPONENT to extensions/* files

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

$ ./mach file-info bugzilla-component extensions/**
UNKNOWN
  extensions/auth
  extensions/build.mk
  extensions/confvars.sh
  extensions/cookie
  extensions/gio
  extensions/moz.build
  extensions/moz.configure
  extensions/permissions
  extensions/pref
  extensions/spellcheck
  extensions/universalchardet


adding BUG_COMPONENT helps with research on files and other tools which interface with the source.  My reason is to ensure when I am working on an intermittent test case that I can find a proper bug component for the test case or root cause if it is obvious.

From some digging in dxr, it looks like each of these subdirs belong to different components.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: add BUG_COMPONENT to extensions/*

Here is what this patch ends up with:

$ ./mach file-info bugzilla-component extensions/**
Core :: AutoConfig (Mission Control Desktop)
  extensions/pref
Core :: DOM
  extensions/permissions
Core :: General
  extensions/build.mk
  extensions/confvars.sh
  extensions/gio
  extensions/moz.build
  extensions/moz.configure
Core :: Internationalization
  extensions/universalchardet
Core :: Networking
  extensions/auth
Core :: Networking: Cookies
  extensions/cookie
Core :: Spelling checker
  extensions/spellcheck


I am not sure about the extensions/gio/* folder.  I took guesses on many of the files here, some have no specific history that could indicate what component they belong to.

Comment 2

[Makoto Kato [:m_kato]]

gio is for gvfs (bug 402892).  It might be either widget:gtk or XPCOM.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: add BUG_COMPONENT to extensions/*

updated with gio/* as Widget: Gtk

Comment 4

[Andrew Overholt [:overholt]]

Comment on attachment 8829208 [details] [diff] [review]
add BUG_COMPONENT to extensions/*

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

Everything looks good except for the pref/ one. I don't know if this should be an r- or an r+ for that, sorry :)

::: extensions/pref/moz.build
@@ +7,5 @@
>  
>  DIRS += ['autoconfig']
> +
> +with Files('**'):
> +    BUG_COMPONENT = ('Core', 'AutoConfig (Mission Control Desktop)')

I *think* this should be Core:: Preferences: Backend but bsmedberg would know for sure.

Comment 5

[Mike Kaply [:mkaply]]

> I *think* this should be Core:: Preferences: Backend but bsmedberg would know for sure.

Actually, autoconfig is correct. It has its own component now.

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

thanks :mkaply, I will wait for bsmedberg's review

Comment 7

[Andrew Overholt [:overholt]]

Comment on attachment 8829208 [details] [diff] [review]
add BUG_COMPONENT to extensions/*

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

Given mkaply's confirmation, r+ from me. I don't think we need to bother Benjamin.

::: extensions/pref/moz.build
@@ +7,5 @@
>  
>  DIRS += ['autoconfig']
> +
> +with Files('**'):
> +    BUG_COMPONENT = ('Core', 'AutoConfig (Mission Control Desktop)')

mkaply confirmed this is the correct component.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bcba677f9c6d
add BUG_COMPONENT to extensions/* files. r=bsmedberg

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/bcba677f9c6d