524216 - Support 'gre-resources' as an alias on 1.9.2

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[Serge Gautherie (:sgautherie)]

... to ease non m-c applications to migrate from m-1.9.1 to m-1.9.2/m-c.

Comment 1

[(dormant account)]

(In reply to comment #0)
> ... to ease non m-c applications to migrate from m-1.9.1 to m-1.9.2/m-c.

why not try land the gre-resources patches on 192?

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Just alias it [Checkin: Comment 12]

Per your bug 521293 comment 7.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)
> why not try land the gre-resources patches on 192?

Obviously ... You'd better ask people involved in those bugs and branch...

In any case, this bug could land as a temporary workaround (so I can move my c-c patches forward) in the meantime.

Comment 4

[(dormant account)]

(In reply to comment #3)
> (In reply to comment #1)
> > why not try land the gre-resources patches on 192?
> 
> Obviously ... You'd better ask people involved in those bugs and branch...
> 
> In any case, this bug could land as a temporary workaround (so I can move my
> c-c patches forward) in the meantime.

I am just a little worried about this fallback approach leading to perf problems in the long term. I'd be ok if this alias had an expiration date or something that would gurantee that we would use it forever.

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 408127 [details] [diff] [review]
(Av1) Just alias it
[Checkin: Comment 12]

"approval1.9.2=?":
No risk, will help comm-central.

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to comment #4)

> I am just a little worried about this fallback approach leading to perf
> problems in the long term.

Hopefully, the perf impact is very low.
Moreover, one can continue to use the old/native syntax if one prefers.

> I'd be ok if this alias had an expiration date or
> something that would gurantee that we would use it forever.

It's m-1.9.2 specific, thus guaranteed to expire when that branch does.
(And m-1.9.2 is not aimed to be long-lived for all we know now.)

Comment 7

[Mark Banner (:standard8)]

IMHO the much better solution would be to have done the alias in a comm-central jar.mn.

The *only* reason it is difficult to patch for 1.9.2 for comm-central apps is the fact that we haven't got a branch. Non-comm-central apps are likely to be able to cope with it in a much better way and then we're also not polluting mozilla-central with unnecessary items.

(In reply to comment #3)
> (In reply to comment #1)
> > why not try land the gre-resources patches on 192?
> 
> Obviously ... You'd better ask people involved in those bugs and branch...

It would perhaps have been nice to at least investigate the possibilities with a quick question first.

Comment 8

[Serge Gautherie (:sgautherie)]

(In reply to comment #7)

> IMHO the much better solution would be to have done the alias in a comm-central
> jar.mn.

Iiuc, this would mean duplicating it in each c-c app.

> Non-comm-central apps are likely to be
> able to cope with it in a much better way and then we're also not polluting
> mozilla-central with unnecessary items.

I assumed some other apps might just be in the same case...
I thought it best to fix it at the source, at least to ask for that first.

> (In reply to comment #3)
> > Obviously ... You'd better ask people involved in those bugs and branch...
> 
> It would perhaps have been nice to at least investigate the possibilities with
> a quick question first.

(My previous answer stands :-|)

Comment 9

[Serge Gautherie (:sgautherie)]

Taras, ping for approval1.9.2:
my synchronization work on c-c depends on having (or not) this alias in m-1.9.2.

Comment 10

[(dormant account)]

(In reply to comment #9)
> Taras, ping for approval1.9.2:
> my synchronization work on c-c depends on having (or not) this alias in
> m-1.9.2.

I don't think I have approval powers for this. I agree with this change, but I think it's up to 192 drivers to approve this.

Comment 11

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 408127 [details] [diff] [review]
(Av1) Just alias it
[Checkin: Comment 12]

a192=beltzner

Comment 12

[Serge Gautherie (:sgautherie)]

Comment on attachment 408127 [details] [diff] [review]
(Av1) Just alias it
[Checkin: Comment 12]


http://hg.mozilla.org/releases/mozilla-1.9.2/rev/8641b5d60989