Closed Bug 1332423 Opened 3 years ago Closed 3 years ago

add BUG_COMPONENT to extensions/* files

Categories

(Core :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: jmaher, Assigned: jmaher)

References

Details

Attachments

(1 file, 1 obsolete file)

$ ./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.
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.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8828485 - Flags: review?(overholt)
gio is for gvfs (bug 402892).  It might be either widget:gtk or XPCOM.
updated with gio/* as Widget: Gtk
Attachment #8828485 - Attachment is obsolete: true
Attachment #8828485 - Flags: review?(overholt)
Attachment #8829208 - Flags: review?(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.
Attachment #8829208 - Flags: review?(overholt)
Attachment #8829208 - Flags: review?(benjamin)
Attachment #8829208 - Flags: review-
> I *think* this should be Core:: Preferences: Backend but bsmedberg would know for sure.

Actually, autoconfig is correct. It has its own component now.
thanks :mkaply, I will wait for bsmedberg's review
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.
Attachment #8829208 - Flags: review?(benjamin)
Attachment #8829208 - Flags: review-
Attachment #8829208 - Flags: review+
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bcba677f9c6d
add BUG_COMPONENT to extensions/* files. r=bsmedberg
https://hg.mozilla.org/mozilla-central/rev/bcba677f9c6d
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.