Closed Bug 1539283 Opened 6 months ago Closed 5 months ago

crashinject.exe and crashinjectdll.dll missing from Windows common.tests.tar.gz


(Firefox Build System :: General, enhancement)

Not set


(firefox68 fixed)

Tracking Status
firefox68 --- fixed


(Reporter: gbrown, Assigned: cmanchester)



(Keywords: in-triage)


(1 file)

In bug 1534858 I noticed that crashinject.exe and crashinjectdll.dll are missing from the common.tests archive, because they are missing from the list at

I think crashinject should be included in the tests archive because test automation code expects it:

However, adding a dll to that list also accidentally includes the dll in the archive, which is clearly wrong, and resulted in backout of my proposed change:

:glandium noted:

Be is a an artifact builds, and artifact builds install *.dll :( (see
Part of the problem is that dist/bin is kind of polluted with non-Firefox things.
Part of the problem is that wildcard for artifact builds.

I'm not sure how to address that.

Chris do you have any ideas how to address this issue?

Flags: needinfo?(cmanchester)

I think we need to be smarter about figuring out which msvc dlls the build needs, so an improvement to this:

I can take a look at this later in the week if need be.

Flags: needinfo?(cmanchester)
Keywords: in-triage

gbrown, is this something you can take a look at based on comment 2, or should I find some time to take a look?

Thanks Chris, but I'm not very comfortable with build changes and I'm pretty busy. It would be great if you could take this.

Assignee: nobody → cmanchester
Pushed by
Use less general wildcard patterns when packaging dlls in an artifact build. r=nalexander
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Regressions: 1557019
You need to log in before you can comment on or make changes to this bug.