Closed Bug 1139679 Opened 9 years ago Closed 9 years ago

Make use of BUG_COMPONENT metadata

Categories

(Calendar :: Build Config, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

References

Details

Attachments

(1 file, 2 obsolete files)

Attached patch Fix - v1 (obsolete) β€” β€” Splinter Review
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8572925 - Flags: review?(Pidgeot18)
See bug 1139341 for why we need the empty moz.build
Comment on attachment 8572925 [details] [diff] [review]
Fix - v1

>+++ b/calendar/lightning/moz.build
>@@ -31,3 +31,17 @@
>     'content/lightning.js',
> ]
> 
>+with Files('**'):
>+    BUG_COMPONENT = ('Calendar', 'Lightning Only')
>+
>+with Files('content/suite-*'):
>+    BUG_COMPONENT = ('Calendar', 'Lightning: SeaMonkey Integration')
>+
>+with Files('content/suite-*'):
>+    BUG_COMPONENT = ('Calendar', 'Lightning: SeaMonkey Integration')
>+
Not sure if my eyes are playing tricks, but this appears to be a duplication.
Attached patch Fix - v2 (obsolete) β€” β€” Splinter Review
Oh thanks for spotting that, not sure where that came from.
Attachment #8572925 - Attachment is obsolete: true
Attachment #8572925 - Flags: review?(Pidgeot18)
Attachment #8574831 - Flags: review?(Pidgeot18)
Comment on attachment 8574831 [details] [diff] [review]
Fix - v2

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

I only vaguely looked over the lists of bug component metadata, since I'm not familiar with the Calendar side of things.

::: calendar/moz.build
@@ +23,5 @@
> +with Files('**/*.mk'):
> +    BUG_COMPONENT = ('Calendar', 'Build Config')
> +    FINAL = True
> +
> +with Files('**/*.manifest'):

I'm not too sure about this one being true...

::: moz.build
@@ +2,5 @@
> +# This Source Code Form is subject to the terms of the Mozilla Public
> +# License, v. 2.0. If a copy of the MPL was not distributed with this
> +# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> +
> +# This file needs to stay here even if empty so that mach will work

You should mention that it's needed specifically for commands like mach file-info.
Attachment #8574831 - Flags: review?(Pidgeot18) → review+
Attached patch Fix - v3 β€” β€” Splinter Review
(In reply to Joshua Cranmer [:jcranmer] from comment #5)
> Comment on attachment 8574831 [details] [diff] [review]
> Fix - v2
> 
> Review of attachment 8574831 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I only vaguely looked over the lists of bug component metadata, since I'm
> not familiar with the Calendar side of things.
> 
> ::: calendar/moz.build
> @@ +23,5 @@
> > +with Files('**/*.mk'):
> > +    BUG_COMPONENT = ('Calendar', 'Build Config')
> > +    FINAL = True
> > +
> > +with Files('**/*.manifest'):
> 
> I'm not too sure about this one being true...
Given bug 1143163 I should probably change it to *manifest. It would cover these files:
http://mxr.mozilla.org/comm-central/find?string=manifest%24&tree=comm-central&hint=calendar

I guess they could be seen as belonging to the respective component, I don't think it makes a big difference though. Leaving as build config for now.


> 
> ::: moz.build
> @@ +2,5 @@
> > +# This Source Code Form is subject to the terms of the Mozilla Public
> > +# License, v. 2.0. If a copy of the MPL was not distributed with this
> > +# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> > +
> > +# This file needs to stay here even if empty so that mach will work
> 
> You should mention that it's needed specifically for commands like mach
> file-info.
Thanks, I've changed the comment.
Attachment #8574831 - Attachment is obsolete: true
Attachment #8578609 - Flags: review+
Pushed to comm-central changeset b924dcfdae95
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.1
Blocks: 1175735
You need to log in before you can comment on or make changes to this bug.