OS sniffing should detect Windows 10 from "Windows NT 6.4" instead of detecting Windows NT

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Gijs, Assigned: Gijs)

Tracking

Production

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Pretty much as per summary.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment on attachment 8521407 [details] [diff] [review]
3.patch

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

r=dylan
Attachment #8521407 - Flags: review?(dylan) → review+
thanks gijs!

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   e15f335..ecc04a6  master -> master
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Component: Extensions: GuidedBugEntry → General
Why hasn't this patch been pushed upstream?
(In reply to Frédéric Buclin from comment #4)
> Why hasn't this patch been pushed upstream?

Because I have no idea how to? If you tell me what you need from me to push this upstream, I'd be happy to oblige. Keep in mind that it does seem like MS will change the UA bits and exposed version number once more before final release.
(In reply to :Gijs Kruitbosch from comment #5)
> (In reply to Frédéric Buclin from comment #4)
> > Why hasn't this patch been pushed upstream?
> 
> Because I have no idea how to? If you tell me what you need from me to push
> this upstream, I'd be happy to oblige. Keep in mind that it does seem like
> MS will change the UA bits and exposed version number once more before final
> release.

File a similar bug to this one under the product Bugzilla with component "Creating Bugs". The fork the upstream Bugzilla git repo[1] and create a similar pull request and place the pull request link in the attachment form here using the URL feature. The ask for review? from one of the suggested reviewers.

Once it is r+ and then approved, the reviewer can move it over from github like we did here and commit it to the upstream code base.

There is a docker container for upstream Bugzilla[2] as well that you can use to test your patches before review.

Thanks
dkl

[1] https://github.com/bugzilla/bugzilla
[2] https://github.com/dklawren/docker-bugzilla
You need to log in before you can comment on or make changes to this bug.