Closed Bug 1422848 Opened 7 years ago Closed 7 years ago

NSS taskcluster should run at least one "make optimized" build

Categories

(NSS :: Test, enhancement)

3.34
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(1 file, 1 obsolete file)

Currently, taskcluster runs "make" builds only in debug mode, apparently.

It would be good to test optimized make builds, too.
Assignee: nobody → kaie
Attached patch 1422848-v1.patch (obsolete) — Splinter Review
Attachment #8934190 - Flags: review?(franziskuskiefer)
Comment on attachment 8934190 [details] [diff] [review]
1422848-v1.patch

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

::: automation/taskcluster/graph/src/extend.js
@@ +213,5 @@
>      collection: "make",
>      env: {USE_64: "1"}
>    }, "build.sh");
>  
> +  await scheduleWindows("Windows 2012 64 (opt, make)", {

I'd like to avoid adding too many platforms. I don't think that we need Windows opt make builds. The number of Windows NSS users is rather limited.
Attachment #8934190 - Flags: review?(franziskuskiefer)
That's fine with me.

I'd also be OK with adding Linux 64bit optimized, only. My primary motiviation is to be able to detect failures in that build configuration, so one architecture should be sufficient.
Summary: NSS taskcluster should run "make optimized" builds → NSS taskcluster should run at least one "make optimized" build
Attached patch 1422848-v2.patchSplinter Review
Attachment #8934190 - Attachment is obsolete: true
Attachment #8934264 - Flags: review?(franziskuskiefer)
Comment on attachment 8934264 [details] [diff] [review]
1422848-v2.patch

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

thanks
Attachment #8934264 - Flags: review?(franziskuskiefer) → review+
https://hg.mozilla.org/projects/nss/rev/e3c27c38cb10
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: