Merge lp://qastaging/~roadmr/ubuntu/precise/checkbox/0.13.1 into lp://qastaging/ubuntu/precise/checkbox

Proposed by Daniel Manrique
Status: Superseded
Proposed branch: lp://qastaging/~roadmr/ubuntu/precise/checkbox/0.13.1
Merge into: lp://qastaging/ubuntu/precise/checkbox
Diff against target: 471232 lines (+252651/-169867)
158 files modified
bin/checkbox-cli (+1/-1)
bin/checkbox-gtk (+1/-1)
bin/checkbox-urwid (+1/-1)
checkbox/lib/config.py (+3/-1)
checkbox/lib/template_i18n.py (+2/-1)
checkbox/parsers/submission.py (+8/-4)
checkbox/user_interface.py (+1/-1)
checkbox_gtk/gtk_interface.py (+8/-3)
data/whitelists/default.whitelist (+47/-21)
debian/changelog (+89/-1)
debian/checkbox-cli.postinst (+1/-0)
debian/checkbox-gtk.postinst (+1/-0)
debian/checkbox-urwid.postinst (+1/-0)
debian/control (+14/-36)
debian/hwtest-cli.postinst (+0/-6)
debian/hwtest-gtk.postinst (+0/-6)
debian/hwtest.postinst (+0/-6)
debian/hwtest.postrm (+0/-26)
debian/po/de.po (+3/-4)
debian/po/en_AU.po (+3/-4)
debian/po/en_GB.po (+4/-5)
debian/po/es.po (+5/-5)
debian/po/fr.po (+3/-4)
debian/po/he.po (+3/-4)
debian/po/hu.po (+3/-4)
debian/po/it.po (+3/-4)
debian/po/ja.po (+6/-10)
debian/po/nl.po (+3/-4)
debian/po/pl.po (+3/-4)
debian/po/pt_BR.po (+3/-4)
debian/po/ro.po (+3/-4)
debian/po/ru.po (+3/-4)
debian/po/zh_TW.po (+4/-7)
debian/rules (+3/-7)
install/config (+4/-5)
install/postinst (+17/-21)
jobs/audio.txt.in (+9/-12)
jobs/bluetooth.txt.in (+6/-14)
jobs/camera.txt.in (+1/-17)
jobs/firewire.txt.in (+32/-7)
jobs/graphics.txt.in (+18/-26)
jobs/keys.txt.in (+5/-9)
jobs/local.txt.in (+1/-1)
jobs/mediacard.txt.in (+318/-158)
jobs/memory.txt.in (+5/-10)
jobs/miscellanea.txt.in (+4/-5)
jobs/monitor.txt.in (+6/-0)
jobs/networking.txt.in (+11/-2)
jobs/optical.txt.in (+1/-2)
jobs/panel_clock_test.txt.in (+1/-1)
jobs/resource.txt.in (+9/-0)
jobs/suspend.txt.in (+69/-26)
jobs/usb.txt.in (+34/-17)
jobs/user_apps.txt.in (+2/-2)
jobs/wireless.txt.in (+62/-0)
plugins/apport_prompt.py (+4/-3)
plugins/backend_info.py (+10/-0)
plugins/intro_prompt.py (+7/-5)
plugins/jobs_prompt.py (+17/-5)
plugins/launchpad_report.py (+2/-1)
plugins/recover_prompt.py (+4/-2)
plugins/shell_test.py (+1/-1)
po/POTFILES.in (+2/-0)
po/ace.po (+2170/-1770)
po/af.po (+2164/-1770)
po/am.po (+2185/-1770)
po/ar.po (+2472/-2020)
po/ast.po (+3969/-2264)
po/be.po (+3927/-1931)
po/bg.po (+2819/-2045)
po/bn.po (+2942/-1956)
po/bo.po (+2245/-1778)
po/br.po (+2339/-1806)
po/bs.po (+3856/-2116)
po/ca.po (+3234/-2211)
po/ca@valencia.po (+2964/-1997)
po/checkbox.pot (+2147/-1763)
po/ckb.po (+2192/-1774)
po/cs.po (+3938/-2404)
po/cy.po (+2156/-1771)
po/da.po (+3571/-2266)
po/de.po (+5006/-2478)
po/dv.po (+2151/-1769)
po/el.po (+4319/-2105)
po/en_AU.po (+4874/-2289)
po/en_CA.po (+2560/-1864)
po/en_GB.po (+4974/-2336)
po/eo.po (+3109/-2165)
po/es.po (+4248/-2325)
po/et.po (+2236/-1791)
po/eu.po (+2323/-1796)
po/fa.po (+2151/-1769)
po/fi.po (+3351/-2252)
po/fr.po (+4114/-2246)
po/gd.po (+2125/-1739)
po/gl.po (+3802/-2056)
po/he.po (+2902/-2070)
po/hi.po (+2659/-1837)
po/hr.po (+2330/-1797)
po/hu.po (+3822/-2253)
po/hy.po (+2151/-1769)
po/id.po (+2295/-1892)
po/is.po (+2208/-1871)
po/it.po (+3970/-2446)
po/ja.po (+3387/-1984)
po/jbo.po (+2151/-1769)
po/ka.po (+2172/-1796)
po/kk.po (+2236/-1778)
po/km.po (+2151/-1769)
po/kn.po (+2176/-1770)
po/ko.po (+2712/-1858)
po/ku.po (+2167/-1770)
po/ky.po (+2151/-1769)
po/lt.po (+2333/-1776)
po/lv.po (+3318/-1899)
po/mk.po (+2250/-1891)
po/ml.po (+2160/-1769)
po/mr.po (+2163/-1769)
po/ms.po (+4978/-2213)
po/my.po (+2243/-1791)
po/nb.po (+2525/-1996)
po/nds.po (+2151/-1769)
po/ne.po (+2226/-1778)
po/nl.po (+4474/-2368)
po/nn.po (+2173/-1770)
po/oc.po (+2934/-1897)
po/pl.po (+3634/-2303)
po/pt.po (+3426/-2151)
po/pt_BR.po (+4988/-2313)
po/ro.po (+2809/-2049)
po/ru.po (+5108/-2399)
po/si.po (+2163/-1769)
po/sk.po (+2668/-2068)
po/sl.po (+3771/-2194)
po/sq.po (+3610/-1890)
po/sr.po (+3716/-2188)
po/sv.po (+3762/-2302)
po/ta.po (+2161/-1770)
po/te.po (+2160/-1769)
po/th.po (+2359/-1938)
po/tr.po (+3467/-2112)
po/ug.po (+3820/-2051)
po/uk.po (+2656/-1998)
po/ur.po (+2151/-1769)
po/vi.po (+2629/-1875)
po/zh_CN.po (+2970/-1990)
po/zh_HK.po (+2731/-1958)
po/zh_TW.po (+2718/-1901)
scripts/camera_test (+4/-30)
scripts/compiz-check (+0/-968)
scripts/connect_wireless (+1/-1)
scripts/cpu_offlining (+2/-2)
scripts/cpu_topology (+3/-3)
scripts/removable_storage_test (+60/-59)
scripts/run_compiz_check (+0/-27)
scripts/sleep_test (+4/-37)
scripts/watch_command (+0/-159)
setup.py (+4/-3)
To merge this branch: bzr merge lp://qastaging/~roadmr/ubuntu/precise/checkbox/0.13.1
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Needs Fixing
Review via email: mp+91169@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2012-02-07.

Description of the change

With a sincere apology for the previous, botched merge request, all traces of which have hopefully been either deleted or forgotten, here it is: Checkbox 0.13.1, contains a lot of bugfixes and enhancements.

Detailed changelog is available in the candidate revision request bug 925090.

Thanks!

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

 - I see there are extra changes for a checkbox-sru script; they should either be added to changelog or dropped.

- There are also extra changes in install/postinst for update_configuration() (third hunk in the diff) that weren't immediately obvious to me what they were for besides the dropped hwtest packages. They should be better described in changelog (if not already there).

- there is no mention of the changes to jobs/local.txt.in for the command; changing from .../peripheral.txt to .../phoronix.txt looks rather unusual; and should be mentioned in syslog.

- changes to plugins/jobs_prompt.py aren't changelog, afaict; please add them :)

- there appears to be extra changes to scripts/cpu_offlining; again not in changelog when they should probably be.

review: Needs Fixing
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

I also found that the check_is_laptop script is probably missing, it might have not been added to version control w/ bzr add. ;)

In general, I found the changelog entry quality to be a little low -- given the extreme size of the changelog, please try as a group to be extra careful about writing clear and precise changelog entries in order to simplify review. There was a good UDW session about this for reference (https://wiki.ubuntu.com/MeetingLogs/devweek1201/WritingCrispChangelogs).

review: Needs Fixing

Unmerged revisions

37. By Daniel Manrique

New upstream release (LP: #925090):
* Fixed the cpu_topology script so that it doesn't mistake the word
  'processor' in the value of another field for the field 'processor'
  (LP: #882161)
* Added create_connection script and jobs to automatically create/test a
  wireless network connection.
* Updated wireless job dependencies.
* Add wireless performance data collecting tests.
* Changed is_laptop test to a shell test and implemented a check_is_laptop
  script to check automatically for a systems 'laptopness' (LP: #886668)
* Fixed connect_wireless script which continued failing to correctly
  identify wireless connections.
* Don't fail the sleep_test if the wake alarm is still set (LP: #911161)
* Add requirement for mem sleep state to be supported to the
  suspend_advanced_auto job (LP: #804190)
* Fixed the camera/display test and removed the camera/video one.
* Added display resource and matching requirements to external video
  output tests.
* Added removable_storage_watcher script to replace watch_command to make
  testing USB, FireWire and MMC devices easier and more cohesive.
* Added memory_compare script to automate the memory/info job
* Switch audio settings to correct device before running audio tests
  (LP: #916859)
* Fixed last two remaining strings with backslashes (LP: #868571)
* Fix misplaced parentheses, so translation can work (LP: #904876)
* Refactored install script to be agnostic of variant name.
* Using title defined in user_interface plugin in GTK interface.
* Updated default.whitelist to reflect renamed jobs.
* Removed files with non-printable characters from submission.xml.
* Fixed parser for submission files with empty question comments
  and context info (LP: #912546)
* Added support for skipping tests when the depends don't pass
  (LP: #509598)
* Removed extraneous code from the sleep_test.
* Refactored logic to check for network after suspend.
* Removed deprecated hwtest package.
* Update control files under debian/ to eliminate (most) lintian warnings
  (LP: #352986)
* Environment variables specified with environ: in a job description will be
  passed to the backend for it to add to its environment. (LP: #897889)
* Handle malformed LANGUAGE environment variable values (LP: #912946)
* Added interactive media_keys_test script.
* Make creation of wireless connection files more robust (LP: #923836)
* Recommend gstreamer-gconf to enable media tests on kubuntu (LP: #898641)
* Add bluetooth device requirement to obex jobs (LP: #921128)
* Add a plugin conf variable for the welcome string (shown on the first
  screen when checkbox runs), so it can be changed without much effort.
* Remove superflous bluetooth/detect job
* Fixed spelling mistakes in user_apps job file. (LP: #904209)
* Created automated network info test to get some config info during automated
  runs. (LP: #912038)
* Added requires to suspend wireless jobs so they won't run if wireless isn't
  present (LP: #907150)
* Fixed issue in usb_test with unwritable filesystems (LP: #912522)
* Fixed USB tests so that insert, storage, remove run in proper order
* Removed usb_storage_after_suspend since it's superfluous, all other USB
  tests already run after suspend.
* Modifed usb_test to handle firewire drives as well, renamed script to
  removable_storage_test
* Improvements to Recover dialog and show_info method.
* Error while creating binary package fixed (LP: #921576)
* Replaced xrandr_display_modes with automated check for VESA driver
* Refactored Unity compatibility tests
* Nixed graphics/xorg-version-output job and updated other job dependencies,
  since it is redundant with graphics/xorg-version. (LP: #671144)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches