Closed Bug 1145171 Opened 7 years ago Closed 7 years ago

Include the beta version in about:support


(Toolkit :: General, defect)

Not set



Tracking Status
firefox41 --- fixed
firefox42 --- fixed


(Reporter: Sylvestre, Assigned: Sylvestre)


(Blocks 1 open bug)



(1 file, 2 obsolete files)

In the release management process (and also for QE and UA), having the beta number displayed in about:support would help a lot.

The buildid is a too technical information for most of our users (and, usually, even us, we sometimes don't know).
No longer depends on: 638769
Keywords: helpwanted
Does the beta number get baked into the build?
Depends on: 1145184
Duplicate of this bug: 1175152
Attached patch betasupport.diff (obsolete) — Splinter Review
Blocks: 1145184
No longer depends on: 1145184
Comment on attachment 8623607 [details] [diff] [review]

Review of attachment 8623607 [details] [diff] [review]:

This patch requires to |hg add browser/config/version_about.txt| after copying from version.txt. Setting its content to 39.0b6 throws a config error.

Furthermore, this patch has been bitrotted by a AppConstants.jsm refactoring.
Thanks for the feedback but this is an ongoing work. That is why I didn't ask for a review yet. :)
Assignee: nobody → sledru
Attached patch aboutsupport.diff (obsolete) — Splinter Review
Should be fine now
Attachment #8623607 - Attachment is obsolete: true
Archaeopteryx, what was the error you had?
Blocks: 1176533
It is not clear who can review that... Maybe you can (or help me).
Attachment #8625095 - Attachment is obsolete: true
Attachment #8625961 - Flags: review?(dolske)
Comment on attachment 8625961 [details] [diff] [review]

Review of attachment 8625961 [details] [diff] [review]:

::: toolkit/modules/Troubleshoot.jsm
@@ +171,5 @@
>    application: function application(done) {
>      let data = {
>        name:,
> +      version: AppConstants.MOZ_APP_VERSION_ABOUT,

The data gets exposed to certain Mozilla sites (see bug 1079563), have you checked with who's currently using this to make sure getting the extra version info isn't going to break anything? If so, r+.
Attachment #8625961 - Flags: review?(dolske) → review+
I started a try job to confirm that:

Mark, do you see any potential issue with this change? thanks
Flags: needinfo?(markh)
remote troubleshooting means sumo could potentially/surprisingly be affected, but I don't see any in-tree issues.
Flags: needinfo?(markh)
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Comment on attachment 8625961 [details] [diff] [review]

Approval Request Comment
[Feature/regressing bug #]: bug 1145184 - getting the beta information more available
[User impact if declined]: a few weeks more without the beta information available in about:support. This patch depends on bug 1145171
[Describe test coverage new/current, TreeHerder]: in m-c + in inbound. 
[Risks and why]: Should be low risk. However, this might have an impact on sumo as Mark mentioned. This will only be a potential issue when 41 moves to beta.
[String/UUID change made/needed]: None
Attachment #8625961 - Flags: approval-mozilla-aurora?
Comment on attachment 8625961 [details] [diff] [review]

Let's land this in Aurora. Try server results look good, should be safe.
Attachment #8625961 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Depends on: 1196373
You need to log in before you can comment on or make changes to this bug.