Serge Gautherie (:sgautherie) Reporter
	Description • 13 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1306735320.1306747954.31186.gz&fulltext=1
WINNT 5.2 comm-central-trunk build on 2011/05/29 23:02:00
{
Warning: package error or possible missing or unnecessary file: bin/libEGL.dll (package-manifest, 40).
Warning: package error or possible missing or unnecessary file: bin/libGLESv2.dll (package-manifest, 41).
Warning: package error or possible missing or unnecessary file: bin/d3dx9_42.dll (package-manifest, 42).
Warning: package error or possible missing or unnecessary file: bin/D3DCompiler_42.dll (package-manifest, 43).
}

***

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1305926698.1305934700.17622.gz&fulltext=1
WINNT 5.2 comm-central-trunk build on 2011/05/20 14:24:58
comm-central/9f2029d7b5ff, mozilla-central/2117e6f26155

No bug.

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1306260414.1306278203.20994.gz&fulltext=1
WINNT 5.2 comm-central-trunk leak test build on 2011/05/24 11:06:54
comm-central/0d1c875f3997, mozilla-central/0fad9fa9fa3f

Bug.

(4 days) Regression timeframe:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2117e6f26155&tochange=0fad9fa9fa3f
Lots of changesets... :-/

Joe, could this be a fallout of bug 648911 (which I found at near random), or would you have a clue on what could have triggered these warnings ?

	Justin Wood (:Callek) Assignee
	Comment 1 • 13 years ago

(In reply to comment #0)
> Warning: package error or possible missing or unnecessary file:
> bin/libEGL.dll (package-manifest, 40).

...

> Joe, could this be a fallout of bug 648911 (which I found at near random),
> or would you have a clue on what could have triggered these warnings ?

Since this dll itself is built with angle, I would be more apt to suspect Bug 657748 which is in the regression range you cite.

Bjacob?


And actually serge, can you please look at the log and see if we have the message at: (which looks more suspect)
|http://hg.mozilla.org/mozilla-central/rev/478015023b8a#l1.33|

If that is so, please file a blocking (2.2) releng bug, and assign to me, and mark it as a dependancy of this bug.

	Justin Wood (:Callek) Assignee
	Comment 2 • 13 years ago

(In reply to comment #0)
> Warning: package error or possible missing or unnecessary file:
> bin/libEGL.dll (package-manifest, 40).

...

> Joe, could this be a fallout of bug 648911 (which I found at near random),
> or would you have a clue on what could have triggered these warnings ?

Since this dll itself is built with angle, I would be more apt to suspect Bug 657748 which is in the regression range you cite.

Bjacob?


And actually serge, can you please look at the log and see if we have the message at: (which looks more suspect)
|http://hg.mozilla.org/mozilla-central/rev/478015023b8a#l1.33|

If that is so, please file a blocking (2.2) releng bug, and assign to me, and mark it as a dependancy of this bug.

	Benoit Jacob [:bjacob] (mostly away)
	Comment 3 • 13 years ago

In bug 648804, we upgraded the DXSDK on the builders. Before that, libEGL.dll (and libGLESv2.dll) was linking to d3dx9_42.dll, and after that, it links to d3dx9_43.dll. Could that be the cause of this warning?

For sure bug 657748 was a big change affecting libEGL.dll, but this file existed before and exists after this change so I don't see the problem.

CC'ing people who might know more.

	Justin Wood (:Callek) Assignee
	Comment 4 • 13 years ago

(In reply to comment #3)
> In bug 648804, we upgraded the DXSDK on the builders.

But not on SeaMonkey machines (which this bug is hit on, for the record, sorry if that was not clear)

Which could be causing us to hit the change in http://hg.mozilla.org/mozilla-central/rev/478015023b8a#l1.12

> Before that,
> libEGL.dll (and libGLESv2.dll) was linking to d3dx9_42.dll, and after that,
> it links to d3dx9_43.dll. Could that be the cause of this warning?
> 
> For sure bug 657748 was a big change affecting libEGL.dll, but this file
> existed before and exists after this change so I don't see the problem.
> 
> CC'ing people who might know more.

	Joe Drew (not getting mail)
	Comment 5 • 13 years ago

So do the SeaMonkey builders have the older DirectX SDK installed? No DirectX SDK installed at all?

(aside: Benoit, on builders with the new DX SDK installed, we copy d3dx9_43.dll - and not the older version - into our installation package, right?)

	Justin Wood (:Callek) Assignee
	Comment 6 • 13 years ago

(In reply to comment #5)
> So do the SeaMonkey builders have the older DirectX SDK installed? No
> DirectX SDK installed at all?

In Bug 632325 SeaMonkey installed the Feb SDK onto our builders.

	Joe Drew (not getting mail)
	Comment 7 • 13 years ago

Does something in the SeaMonkey manifest file need to be updated for this new ANGLE? It's not clear to me what is outputting those error messages.

	Serge Gautherie (:sgautherie) Reporter
	Comment 8 • 13 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=ThunderbirdTrunk/1306259947.1306277940.19711.gz&fulltext=1
WINNT 5.2 comm-central leak test build on 2011/05/24 10:59:07
comm-central/ab3ace782235, mozilla-central/0fad9fa9fa3f

No bug.

http://tinderbox.mozilla.org/showlog.cgi?log=ThunderbirdTrunk/1306269119.1306279735.27161.gz&fulltext=1
WINNT 5.2 comm-central build on 2011/05/24 13:31:59
comm-central/8b7cdb6dd357, mozilla-central/0fad9fa9fa3f

Bug.

Regression timeframe, from Thunderbird:
Same mozilla-central changeset :-|
http://hg.mozilla.org/comm-central/pushloghtml?fromchange=ab3ace782235&tochange=8b7cdb6dd357
3 totally unrelated changesets :-<
http://tinderbox.mozilla.org/showbuilds.cgi?tree=ThunderbirdTrunk&maxdate=1306341228&hours=24&legend=0&norules=1
It looks like this could have been revealed after a clobber.!. :-/

Ftr, (4 days) comm-central Regression timeframe, from SeaMonkey:
http://hg.mozilla.org/comm-central/pushloghtml?fromchange=9f2029d7b5ff&tochange=0d1c875f3997
Not too many changesets, (but I didn't check them).

***

(In reply to comment #2)

> And actually serge, can you please look at the log and see if we have the
> message at: (which looks more suspect)
> |http://hg.mozilla.org/mozilla-central/rev/478015023b8a#l1.33|

Yes, first failing SM build and both TB builds report:
{
configure: warning: Found the February 2010 DirectX SDK. This is too old. We now require the June 2010 DirectX SDK, or newer.
configure: warning: Couldn't find the DirectX SDK, needed for ANGLE. Please install it (June 2010 or newer). To explicitly build without ANGLE, reconfigure with --disable-angle.
configure: warning: This will become an error in the future.
}

> If that is so, please file a blocking (2.2) releng bug, and assign to me,
> and mark it as a dependancy of this bug.

Bug 660745.
I see no 'blocking-seamonkey2.2' flag (or the like)... (same with 2.3 ... and 2.4)

	Serge Gautherie (:sgautherie) Reporter
	Comment 9 • 13 years ago

(In reply to comment #3)
> In bug 648804, we upgraded the DXSDK on the builders. Before that,
> libEGL.dll (and libGLESv2.dll) was linking to d3dx9_42.dll, and after that,
> it links to d3dx9_43.dll. Could that be the cause of this warning?

Yes, I assume this is the cause of these warnings: SeaMonkey is trying to package files which are either not installed yet or not built anymore (= atm)...

	Serge Gautherie (:sgautherie) Reporter
	Comment 10 • 13 years ago

(In reply to comment #9)
> not installed yet

Arf, I meant "doesn't yet know that d3dx9 dll name will have to be updated"...

	Serge Gautherie (:sgautherie) Reporter
	Comment 11 • 13 years ago

V.Fixed, by bug 660745,
per seamonkey-2.3[.3].zip, seamonkey-2.4[.1].zip and seamonkey-2.5.zip,
and current 2.9a1 build logs.