539184 - [meta] ship xforms extension for Firefox 3.6 (Gecko 1.9.2)

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[alexander :surkov (:asurkov)]

We need to build xforms extension for Firefox 3.7 (Gecko 1.9.2) which is targeted to Q1 and put an extension to admo. I hope Aaron will help to do this.

I think the primary goal of the new version should be getting an xforms extension working on 1.9.2. However this doesn't mean all regressions must be fixed, but any of them which can be fixed in Q1 timeframe should be fixed. In this light I suggest to bump the version number from 0.8.7pre to 0.8.7.

Please comment any bugs you think should be fixed and shipped within new version.

Comment 1

[aaronr]

Philipp is doing all of the hard work (the builds).  Putting out a build isn't much more than updating the status page and pushing the builds to addons, which I can certainly help with.

Last I heard XForms wasn't building on Mac, though.  Is that true?  I don't know that we should put out a build until that is fixed.

Comment 2

[alexander :surkov (:asurkov)]

(In reply to comment #1)
> Philipp is doing all of the hard work (the builds).  Putting out a build isn't
> much more than updating the status page and pushing the builds to addons, which
> I can certainly help with.

That's great. Thanks.

> Last I heard XForms wasn't building on Mac, though.  Is that true?  I don't
> know that we should put out a build until that is fixed.

This make sense I think.

Comment 3

[Philipp Wagner [:imphil]]

(In reply to comment #1)
> Last I heard XForms wasn't building on Mac, though.  Is that true?  I don't
> know that we should put out a build until that is fixed.

I haven't heard that yet. But I cannot check either as I have no Mac hardware around.

Comment 4

[Philipp Wagner [:imphil]]

The target for this release should be Firefox 3.6, not 3.7 btw (I guess developers always think one more release into the future) ;)

Comment 5

[alexander :surkov (:asurkov)]

(In reply to comment #4)
> The target for this release should be Firefox 3.6, not 3.7 btw (I guess
> developers always think one more release into the future) ;)

Yeah, you're right. 3.6 is something in the past for me :) Thank you for fixing this.

Comment 6

[alexander :surkov (:asurkov)]

I would like to get your opinion whether we should ship xforms extension right now and do not wait other bugs marked blocking. It sounds fixing these bugs might take long time so it's probably worth to drop them from release expectation list. What do you think?

Comment 7

[Sergey Reymerov]

Peter, could you look at the 469717? Is it XForms bug? This bug blocks new XForms release...

Comment 8

[Philipp Wagner [:imphil]]

(In reply to comment #6)
> I would like to get your opinion whether we should ship xforms extension right
> now and do not wait other bugs marked blocking. It sounds fixing these bugs
> might take long time so it's probably worth to drop them from release
> expectation list. What do you think?

I have close to no time at the moment, I'd say ship it now as-is, which gives us at least a first release for Firefox 3.6 and do a follow-up release as soon as some of us have more time to get some of the most annoying bugs fixed (I hope by the end of April).

Comment 9

[Sergey Reymerov]

This is list of bugs that fixed after 2008-10-07.
Is it right list? :
487310, 488330, 556318, 384950, 399567, 466712, 467815, 494337, 494848, 497237, 326413, 455772, 478105, 485734, 490983, 501987, 506915, 515067, 526376, 539275, 
539525, 436866, 521246, 525730, 525735, 530488, 531297, 537881, 539533

Comment 10

[Sergey Reymerov]

I want you this list for changeset for XForms release 0.8.7ff36.

Comment 11

[Sergey Reymerov]

Do exist another patches (that put into repo) that not in this list but make XForms more suitable?

Comment 12

[Sergey Reymerov]

Attachment: patch for version

Patch that changes version of XForms.

Comment 13

[Philipp Wagner [:imphil]]

Comment on attachment 437277 [details] [diff] [review]
patch for version

>diff --git a/version.txt b/version.txt
>--- a/version.txt
>+++ b/version.txt
>@@ -1,1 +1,1 @@
>-0.8.7pre
>+0.8.8ff363

version 0.8.7 was never released, so I'd suggest using this version. There is no need to append ff363 as we support only this Firefox version with this release.

Comment 14

[Sergey Reymerov]

Attachment: patch for version

Comment 15

[aaronr]

At some point before we release we should make sure that xforms installs just fine with Seamonkey, too.

Comment 16

[Sergey Reymerov]

I couldn't compile it for Seamonkey.
It breaks with error:
> e. .t/o. .b/odoils t'/tirnucel'u doer/ c'ofnatlesnet'\ >n(spIePrrfiovramtaenDcOeM Ewvaernnti.nhg()6
>4) : see declaration of 'nsIPrivateDOMEvent::SetCurrentTarget'
>        'nsresult nsIPrivateDOMEvent::SetOriginalTarget(nsIDOMEventTarget *)' >: is abstract
>        ../../dist/include/content\nsIPrivateDOMEvent.h(65) : see declaration >of 'nsIPrivateDOMEvent::SetOriginalTarget'
>
>        'PRBool nsIPrivateDOMEvent::HasOriginalTarget(void)' : is abstract
>        ../../dist/include/content\nsIPrivateDOMEvent.h(68) : see declaration >of 'nsIPrivateDOMEvent::HasOriginalTarget'
>
>nsXFormsOutputElement.cpp
>make[6]: *** [nsXFormsUtils.obj] Error 2

But I made xpi for Firefox 3.6.3 win32. Also I can make this for Linux.

Comment 17

[Sergey Reymerov]

Attachment: xpi for Firefox 3.6.3 on win32

Comment 18

[Alexey Gaidukov]

(In reply to comment #16)
> I couldn't compile it for Seamonkey.
> It breaks with error:
> > e. .t/o. .b/odoils t'/tirnucel'u doer/ c'ofnatlesnet'\ >n(spIePrrfiovramtaenDcOeM Ewvaernnti.nhg()6
> >4) : see declaration of 'nsIPrivateDOMEvent::SetCurrentTarget'
> >        'nsresult nsIPrivateDOMEvent::SetOriginalTarget(nsIDOMEventTarget *)' >: is abstract
> >        ../../dist/include/content\nsIPrivateDOMEvent.h(65) : see declaration >of 'nsIPrivateDOMEvent::SetOriginalTarget'
> >
> >        'PRBool nsIPrivateDOMEvent::HasOriginalTarget(void)' : is abstract
> >        ../../dist/include/content\nsIPrivateDOMEvent.h(68) : see declaration >of 'nsIPrivateDOMEvent::HasOriginalTarget'
> >
> >nsXFormsOutputElement.cpp
> >make[6]: *** [nsXFormsUtils.obj] Error 2
> 
> But I made xpi for Firefox 3.6.3 win32. Also I can make this for Linux.

With which version of seamonkey you built XForms? Stable version of seamonkey is based on Gecko 1.9.1.

Comment 19

[Alexey Gaidukov]

The current verion of xforms is for gecko 1.9.2. Seamonkey 2.1 is based on trunk as I figured out and there is no seamonkey with gecko 1.9.2. My propose is try to support next version of seamonkey in future releases of xforms.

Comment 20

[alexander :surkov (:asurkov)]

bump version patch landed - http://hg.mozilla.org/xforms/rev/f21111260427

Comment 21

[alexander :surkov (:asurkov)]

So I'd suggest to wait for bug 457075 and ship the extension, move all blocking bugs to next version. Are we all on the same page?

Comment 22

[aaronr]

Alex, can you check in the fix for bug 469717 before it ships?  Nice to have another crasher fixed.

Comment 23

[alexander :surkov (:asurkov)]

(In reply to comment #22)
> Alex, can you check in the fix for bug 469717 before it ships?  Nice to have
> another crasher fixed.

I could but bug 469717 patch is Gecko's patch and it doesn't affect on XForms release until it's landed on 1.9.2.x and proper Firefox version is shipped.

Comment 24

[alexander :surkov (:asurkov)]

(In reply to comment #21)
> So I'd suggest to wait for bug 457075 and ship the extension, move all blocking
> bugs to next version. Are we all on the same page?

I landed bug 457075. So, let's start shipping? Any objections?

Comment 25

[aaronr]

ok with me

Comment 26

[alexander :surkov (:asurkov)]

Do we want to ship next xforms version for Firefox 3.6.x or Firefox 3.7? I wonder to know wether we need to deal with hg branches or we can just focus on trunk development?

Comment 27

[Sergey Reymerov]

I think that it's time to ship XForms for Firefox 3.6.x.

Comment 28

[aaronr]

since we don't depend on any browser patches (except for the delete & send crash) I don't see a reason not to put this out for 3.6 first

Comment 29

[alexander :surkov (:asurkov)]

(In reply to comment #28)
> since we don't depend on any browser patches (except for the delete & send
> crash) I don't see a reason not to put this out for 3.6 first

Sure. Is next xforms version expected for Firefox 3.7 (or later)?

Comment 30

[aaronr]

It can be whenever we need it to be if we've got patches that need to get out to people.  I haven't been following Firefox for a while, any changes expected in 3.7 such that the 3.6 xforms extension wouldn't work on it?

Comment 31

[Sergey Reymerov]

Last XForms doesn't compile on trunk FF.

Comment 32

[alexander :surkov (:asurkov)]

(In reply to comment #30)
> It can be whenever we need it to be if we've got patches that need to get out
> to people.  I haven't been following Firefox for a while, any changes expected
> in 3.7 such that the 3.6 xforms extension wouldn't work on it?

Current xforms version isn't compiled for Firefox 3.7. So if we're sure we want to keep an ability to ship xforms for 3.6 while the primary work happens on Firefox 3.7 then we need to try to get hg branches working or have separate hg repo for 3.6. Any way we should care about patches for two xforms versions which can't make happy anybody. On the other hand we could stay on Firefox 3.6 and don't try to get xforms working on 3.7 until it's shipped.

Comment 33

[aaronr]

I'd say we don't branch and just support 3.6 until 3.7 reaches beta.  Then it'd be worth getting us compiling for 3.7 so that we are ready as quickly as we can after 3.7 goes out.  It might be worth branching if compiling for 3.7 looks to be a huge issue (involving multiple patches) or if we encounter a bug that will require a change in the base so that we can test and make sure the issue is resolved.  But if it looks like routine maintenance, I'd think we could just keep things on the trunk like we always did.

Comment 34

[alexander :surkov (:asurkov)]

Ok, fine with me.

Aaron, do you need a help to make xforms.xpi so that all you need is to publish them on amo?

Personally I could do a mac version.

Comment 35

[aaronr]

If you can do that mac version and send it to me and if philipp can do the windows and linux versions (which he already does, right?) and send them to me, then I'll get them posted and update the documentation.

Comment 36

[alexander :surkov (:asurkov)]

Great. Probably philipp should rebuild because bug 457075 was landed. Also I think it's worth to set hg tag.

Comment 37

[alexander :surkov (:asurkov)]

(In reply to comment #36)
>  Also I
> think it's worth to set hg tag.

Is XFORMS_0_8_7_GECKO_1_9_2 ok as a tag name (both for xforms and schema-validation repos)?

Comment 38

[Philipp Wagner [:imphil]]

fine with me. I'll do a manual rebuild (linux and windows) from the tag as soon as it's set.

Comment 39

[alexander :surkov (:asurkov)]

(In reply to comment #38)
> fine with me. I'll do a manual rebuild (linux and windows) from the tag as soon
> as it's set.

done:
http://hg.mozilla.org/xforms/rev/90618060e59f
http://hg.mozilla.org/schema-validation/rev/a4bc587332b5

Comment 40

[alexander :surkov (:asurkov)]

Could we consider to include crash fix from bug 556492 into release? It sounds very important.

Comment 41

[Philipp Wagner [:imphil]]

let's include it. I need to redo the builds anyway as they're crashing for some reason.

Comment 42

[Alexey Gaidukov]

I guess the bug is important and it's neccessary to include it into release.

Comment 43

[alexander :surkov (:asurkov)]

Bug 556492 was landed. Tag was added XFORMS_0_8_7_GECKO_1_9_2_3 - http://hg.mozilla.org/xforms/rev/f51a343e8228

Comment 44

[alexander :surkov (:asurkov)]

Attachment: os x xpi

Comment 45

[Alexey Gaidukov]

I guess bug 563190 is Gecko problem and it is not a reason to delay releasing XForms 0.8.7.

Comment 46

[Alexey Gaidukov]

Does anybody have informatin about 0.8.7 shipping?

Comment 47

[aaronr]

(In reply to comment #46)
> Does anybody have informatin about 0.8.7 shipping?

It has been submitted to addons.  Waiting for it to be approved.

Comment 48

[Alexey Gaidukov]

(In reply to comment #47)
> (In reply to comment #46)
> > Does anybody have informatin about 0.8.7 shipping?
> 
> It has been submitted to addons.  Waiting for it to be approved.

I've found it

https://addons.mozilla.org/ru/firefox/addons/versions/824#version-0.8.7

Comment 49

[Alexey Gaidukov]

(In reply to comment #9)
> This is list of bugs that fixed after 2008-10-07.
> Is it right list? :
> 487310, 488330, 556318, 384950, 399567, 466712, 467815, 494337, 494848, 497237,
> 326413, 455772, 478105, 485734, 490983, 501987, 506915, 515067, 526376, 539275, 
> 539525, 436866, 521246, 525730, 525735, 530488, 531297, 537881, 539533

+  457075, 556492, 556318

Comment 50

[alexander :surkov (:asurkov)]

Move bugs blocking 532963 563190 to next release (bug 567310). Since the extension is on amo expecting a review I think we can mark this bug fixed. Nothing is left to be done. Please reopen if you have objections.

Comment 51

[aaronr]

(In reply to comment #49)
> (In reply to comment #9)
> > This is list of bugs that fixed after 2008-10-07.
> > Is it right list? :
> > 487310, 488330, 556318, 384950, 399567, 466712, 467815, 494337, 494848, 497237,
> > 326413, 455772, 478105, 485734, 490983, 501987, 506915, 515067, 526376, 539275, 
> > 539525, 436866, 521246, 525730, 525735, 530488, 531297, 537881, 539533
> 
> +  457075, 556492, 556318

I'm also including 372127, 391926, 392645, 337428, 366886 in the list of bugs in the announcement since these bugs are no longer re-creatable since 0.8.6 went out (probably due to core changes in 3.5+).

Comment 52

[Alexey Gaidukov]

XForms 0.8.7 has been approved in amo.

Comment 53

[Alexey Gaidukov]

Is it possible to add release notes into page http://www.mozilla.org/projects/xforms/? 

Aaron could you do this?

Comment 54

[aaronr]

Attachment: documentation update

Here is a patch to update the documentation with.  I had cvs access to do it before but now that mozilla.org has switched to svn access I'm re-requesting permission to do documentation checkins using svn.  I'll check this in once I get access.

Alexey, please look at the patch and let me know if you had any other changes in mind.

Comment 55

[Alexey Gaidukov]

I have no ideas about other changes of patch to update the documentation.

Comment 56

[aaronr]

Attachment: status detail xml patch

Comment 57

[Intermittent Failures Robot]

1 failures in 4705 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 1

Platform breakdown:

• windows10-64-shippable-qr: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=539184&startday=2019-05-06&endday=2019-05-12&tree=all

Comment 58

[Intermittent Failures Robot]

1 failures in 4658 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 1

Platform breakdown:

• android-hw-g5-7-0-arm7-api-16: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=539184&startday=2019-05-13&endday=2019-05-19&tree=all