Closed Bug 1589087 Opened 5 years ago Closed 5 years ago

Latest Firefox Preview Nightly doesn't have a $version in Remote Debugging

Categories

(DevTools :: about:debugging, defect, P3)

defect

Tracking

(firefox71 fixed)

RESOLVED FIXED
Firefox 71
Tracking Status
firefox71 --- fixed

People

(Reporter: Harald, Assigned: jdescottes)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Attached image image.png (deleted) —

STR: Connext Firefox Preview Nightly in Remote Debugging

AR: Title is Firefox Preview Nightly ({$version})

ER: Version shown

Thanks for filing, mentioned it in https://github.com/mozilla-mobile/fenix/issues/2016

Initial workaround was to use a version retrieved from ADB as explained in https://bugzilla.mozilla.org/show_bug.cgi?id=1542286#c9

The exact code we are using at the moment to retrieve the version is

  async init() {
    const packageName = this._packageName();
    const query = `dumpsys package ${packageName} | grep versionName`;
    const versionNameString = await shell(this._adbDevice.id, query);
    const matches = versionNameString.match(/versionName=([\d.]+)/);
    if (matches && matches[1]) {
      this._versionName = matches[1];
    }
  }

https://searchfox.org/mozilla-central/rev/171109434c6f2fe086af3b2322839b346a112a99/devtools/shared/adb/adb-runtime.js#19-23

It looks like the version number is now Nightly 191016 06:01 on Nightly builds of Firefox Preview, so it no longer matches our regexp. We can update the regexp to be a bit more flexible, but I mentioned it in #fission on slack because this seems weird as a version number.

Attached image fixed_screenshot.png (deleted) —

Fixed it but it's not looking great :)

The format of the Fenix versionName on Nightly no longer matches our regular expression.
Updating the regular expression to accommodate both versions such as "2.1.0" and "Nightly 191016 06:01"

Fixed it but it's not looking great :)

Mh, but probably good enough for Nightly for the moment.

For some more context, the version format changed in https://github.com/mozilla-mobile/fenix/pull/3742
The goal was to have more granularity and avoid clashes when several builds occur on the same day.
The actual format is not set in stone and might be updated later if the fenix team decides to do so.

The release version format remains X.Y.Z

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f042cb2f447c Update regular expression used in ADB to extract Fenix version r=ladybenko
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: