1361370 - It is not possible to override the values of MOZ_SOURCE_CHANGESET and MOZ_SOURCE-REPO in Seamonkey builds

Status: RESOLVED FIXED

(SeaMonkey :: General, defect)

Description

[Bill Gianopoulos [:WG9s]]

For a Frefox build it si possible to override the values for MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO for a custom build by defining these vaues in system environment variables.  In fact the Seamonkey build makes use of this feature to ensure that about:buildconfig reflects the Seamonkey repository and changeset rather than the gecko ones.  Unfortunately, it does this in an unconditional manner.  It should only be defining these environment variables if they are not already defined, to permit the builder to override these values.

Comment 1

[Bill Gianopoulos [:WG9s]]

Attachment: Allow overrride of MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO.

Comment 2

[Bill Gianopoulos [:WG9s]]

Attachment: Allow overrride of MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO.

Ooops had wrong reviewer and not really a checkin ready comment.

Comment 3

[Bill Gianopoulos [:WG9s]]

To make it clear why I do that, with this fix I can make about:buildconfig show the comm-central changeset my build was based from and also have a link to my build page which documents the additional patches I includue in my builds.

Comment 4

[Bill Gianopoulos [:WG9s]]

I accomplish this by including the following in my build script:

export MOZ_INCLUDE_SOURCE_INFO=1
export MOZ_SOURCE_CHANGESET=`cat .changeset`
export MOZ_SOURCE_REPO=https://hg.mozilla.org/comm-central

the .changset file contains the changset from before my patches were applied.

Comment 5

[Bill Gianopoulos [:WG9s]]

I suppose maybe is should have also updated the comment but looking for a rview first.

I was thinking perhaps changing the comment to:

# Ensure the comm-* values are used unless overridden via environment variables.].

Comment 6

[Bill Gianopoulos [:WG9s]]

I suppose maybe is should have also updated the comment but looking for a rview first.

I was thinking perhaps changing the comment to:
# Ensure the comm-* values are used unless overridden via environment variables.(In reply to Bill Gianopoulos [:WG9s] from comment #5)
> I suppose maybe is should have also updated the comment but looking for a
> rview first.
> 
> I was thinking perhaps changing the comment to:
> 
> # Ensure the comm-* values are used unless overridden via environment
> variables.].

Comment 7

[Bill Gianopoulos [:WG9s]]

Attachment: Allow overrride of MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO.

Comment 8

[Bill Gianopoulos [:WG9s]]

I decided to fix the comments before review.

Comment 9

[Bill Gianopoulos [:WG9s]]

Attachment: Allow overrride of MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO.

Same patch just a typo in the comment corrected.

Comment 10

[Mike Hommey [:glandium]]

Comment on attachment 8864811 [details] [diff] [review]
Allow overrride of MOZ_SOURCE_CHANGESET and MOZ_SOURCE_REPO.

Review of attachment 8864811 [details] [diff] [review]:
-----------------------------------------------------------------

Beware that as of bug 1279048, those variables are used for debug symbol files.

Comment 11

[Bill Gianopoulos [:WG9s]]

OK so I don;t know, does adding checkin-needed as a keyword here help to get it checked in, or do I need to find my own friend with check-in privs to do this for me?

Comment 12

[Frank-Rainer Grahl (:frg)]

Jorg, the patch got r+ from Mike. Should I just check it in or do you want to do it?

Comment 13

[Jorg K (CEST = GMT+2)]

I'll check it in when I next need a patch, so within the next 24 hours. I'll leave the NI to remind me.

Comment 14

[Bill Gianopoulos [:WG9s]]

(In reply to Jorg K (GMT+2) from comment #13)
> I'll check it in when I next need a patch, so within the next 24 hours. I'll
> leave the NI to remind me.

Thanks!

Comment 15

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/comm-central/rev/bd34d80df5ff4929617aaede1ec44230f54d65f7

I quite aggressively manage C-C these days, that means, I do an C-C push after every M-C merge to detect bustage. Small patches like this one help me with that. However, at times a small patch has to wait until I need to do a push again and don't have more important patches.