449735 - Fix splashos2.rc for comm-central static builds

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Peter Weilbacher]

Static builds from comm-central are broken, because splashos2.rc assumes that the widget dir is two directories up from mail/app and suite/app, and the pointer resources are in ..\..\widget\src\os2\res. With the standard (?) comm-central setup where mozilla-central is located in the mozilla directory below comm-central, this is now  ..\..\mozilla\widget\src\os2\res. 

I wonder if the Windows version (mail/app/splash.rc) is similarly broken, there I see the same relative paths that will probably be just as wrong for comm-central builds...

Hmm, there is no "Build Config" component in MailNews Core...

Comment 1

[Peter Weilbacher]

Attachment: fix

This should do it for OS/2.

Also asking for sr to make sure that I understood the comm-central directory setup correctly. Should I fix this in Windows' splash.rc, too?

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 332906 [details] [diff] [review]
fix

Dan probably doesn't know the build config for comm-central as well as me and Robert, so I'll still this from him.

This looks fine.

It does indeed look like the splash.rc are also broken, though I'm not sure what effect they will have. Could you file a bug please?

fwiw, there's an open bug about getting a Build Config component set up within MailNews Core.

Comment 3

[Peter Weilbacher]

Comment on attachment 332906 [details] [diff] [review]
fix

Thanks, that was really fast.

Actually, I already verified for OS/2 that this works, so I don't think we need mkaply's additional review.

Comment 4

[Peter Weilbacher]

My first push to comm-central, seems to have worked fine (changeset 68:60eef23d349e or http://hg.mozilla.org/comm-central/index.cgi/rev/60eef23d349e).