Move unconditional LOCAL_INCLUDES into moz.build

RESOLVED FIXED in mozilla30

Status

Firefox Build System
General
RESOLVED FIXED
4 years ago
2 months ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

(Blocks: 1 bug)

Trunk
mozilla30
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 8371550 [details] [diff] [review]
Part 1: automatic moves

Drops us from 81 to 46.
Attachment #8371550 - Flags: review?(mshal)
(Assignee)

Comment 2

4 years ago
Created attachment 8371551 [details] [diff] [review]
Part 2: fixups
Attachment #8371551 - Flags: review?(mshal)
Comment on attachment 8371550 [details] [diff] [review]
Part 1: automatic moves

>diff --git a/modules/libmar/sign/moz.build b/modules/libmar/sign/moz.build
>--- a/modules/libmar/sign/moz.build
>+++ b/modules/libmar/sign/moz.build
>@@ -7,8 +7,15 @@
> LIBRARY_NAME = 'signmar'
> 
> UNIFIED_SOURCES += [
>     'mar_sign.c',
>     'nss_secutil.c',
> ]
> 
> FORCE_STATIC_LIB = True
>+
>+LOCAL_INCLUDES += [
>+    '../src',
>+    '../verify',
>+    '/dist/include',
>+]

/dist/include can go away since we get it from config/config.mk, right?
Attachment #8371550 - Flags: review?(mshal) → review+

Updated

4 years ago
Attachment #8371551 - Flags: review?(mshal) → review+
https://hg.mozilla.org/mozilla-central/rev/20d7a9037429
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30

Updated

2 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.