541657 - Port |Bug 540038 - add warning if configure or config.status are out of date| to comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Serge Gautherie (:sgautherie)]

Callek, could you do that one? I'm not very confident with it.

Ted, while there, I wonder what the "all" target is:
Could it be removed on m-c?
Should it be added to c-c?

Comment 1

[Justin Wood (:Callek)]

Attachment: v1 -- just copy it

This does 2 things:
* Adds an |all| target that we just forward to m-c (we do nothing with it here, I can pull it back out if you want)
* Just copies the code from Bug 540038.

Comment 2

[Robert Kaiser]

Comment on attachment 423199 [details] [diff] [review]
v1 -- just copy it

Sounds good to me, even if the diff looked strange at first until I realized I was thinking differently than the diff tool ;-)

Comment 3

[Justin Wood (:Callek)]

Pushed as http://hg.mozilla.org/comm-central/rev/142e1e5b4f0b

Comment 4

[Mark Banner (:standard8)]

Re status-thunderbird3.0: Not sure I really care either way about this. It seems to be a build config issue that afaik no-one has ever hit and the build system should re-generate out of date configure anyway... or is there some other weird case?

Either way, this doesn't feel enough to set "wanted", but "wontfix" is probably not appropriate either, so I'll just go for cancelling the flag.

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #4)

I'll let it be for now.
I'll ask again if bug 540038 eventually makes it to m-1.9.1.