333172 - minVersion should be a fixed number, don't link against layout for DOM Inspector

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, defect)

Description

[Mark Banner (:standard8)]

This is separated out from the review of bug 331758.

minVersion should be a fixed and not floating number. To do this, we need to make sure that DOMI doesn't link against layout so that it will be able to be used against more than one version of Mozilla application(s).

Comment 1

[Mark Banner (:standard8)]

Looks like bug 325100 has just removed the linking against layout for DOMI. Therefore I believe we should be able to implement this bug?

Comment 2

[Shawn Wilsher :sdwilsh]

So, what should this be set to?  On trunk we'd have to set the minVersion to something like 3.0a1 maybe?

Should we set the max version to 3.0.* (or 3.0.0.*)?

Comment 3

[Nickolay_Ponomarev]

I think you still support old versions in trunk DOMi. So minVersion should be whatever minimum version you support. The maxVersion should be as it is now (i.e. 3.0a2 (iirc) for trunk). Do not use 3.0 and its variations since you can't promise to support something that's not released.

Comment 4

[Shawn Wilsher :sdwilsh]

(In reply to comment #3)
> I think you still support old versions in trunk DOMi. So minVersion should be
> whatever minimum version you support. The maxVersion should be as it is now
> (i.e. 3.0a2 (iirc) for trunk). Do not use 3.0 and its variations since you
> can't promise to support something that's not released.

That's a good point, but I am pretty sure that we've made changes to the c++ components that didn't land on branch, so it would only be trunk.

It should also be noted that my previous numbers were for browser, and I have no idea what the others should be set to.

Comment 5

[Mark Banner (:standard8)]

(In reply to comment #4)
> It should also be noted that my previous numbers were for browser, and I have
> no idea what the others should be set to.

Check the version.txt files for a starting point - typically in mozilla/<app specific dir>/config/version.txt. SeaMonkey should be set to 1.5a minimum, as that's SM's current trunk version - and like you say, I think the c++ changes only made it on trunk.

Comment 6

[[e-mail]]

AFAIK DOMi on 1.8 branch doesn't link against layout either - this broke the OS/2 installer builds which didn't remove the old DOMi library.

Comment 7

[Shawn Wilsher :sdwilsh]

Attachment: v1.0

should do it.
Neil r for Suite, Mail
Gavin r for Browser
Joey r for Calendar

Comment 8

[Shawn Wilsher :sdwilsh]

checked in by timeless