Confirmed users
1,567
edits
Compatibility/Mobile/WipeOutUAOverides: Difference between revisions
Jump to navigation
Jump to search
Compatibility/Mobile/WipeOutUAOverides (view source)
Revision as of 15:13, 23 July 2013
, 23 July 2013→Updating the UA override list
| Line 86: | Line 86: | ||
# Synchronize your repo with the mother repo<br/> <code>git fetch upstream</code> | # Synchronize your repo with the mother repo<br/> <code>git fetch upstream</code> | ||
# Modify the <code>gaia/blob/master/build/ua-override-prefs.js</code> | # Modify the <code>gaia/blob/master/build/ua-override-prefs.js</code> | ||
# Do a | # Do a commit for the modification<br/> <code>commit -m "bug_number- bug label" ua-override-prefs.js</code> | ||
# You can create a diff file for a patch if you want. Not mandatory.<br/> <code>git diff -U8 > ~/ua-override-prefs-YYYYMMDD.patch</code> you might want to add it to the bug you will create later on. | |||
# One finished, push your modification to your github repo.<br/> <code>git push</code> | # One finished, push your modification to your github repo.<br/> <code>git push</code> | ||
# Create a pull request in your github repo for the gaia project with your modifications | |||
# Create a pull request | # Create a new bug in bugzilla with the title "Remove UA override for example.org". Replace example.org by the domain. | ||
# Create a new bug in bugzilla with the title "Remove UA override for example.org". Replace example.org by the domain | # Product: Boot2Gecko, Component: Gaia, Platform: ARM, Gonk (Firefox OS) | ||
# Add the Patch as an attachment. | |||
# Add a comment on this bug with the link to the Pull Request. | # Add a comment on this bug with the link to the Pull Request. | ||
# Ask for review in bugzilla and put lmandel for reviewer | # Ask for review in bugzilla and put lmandel for reviewer | ||