network_device_info test driver info could potentially be blank
Bug #1042395 reported by
Matt Fischer
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox |
Fix Released
|
Medium
|
Matt Fischer | ||
Bug Description
Some device drivers do not set a Version field. I found this out when working on the audio driver check. For those we should fall back to using vermagic. I've not seen a networking driver that fails yet, but my testing has been limited, so I'd rather just use the safer code from the audio_driver_info script.
Related branches
lp://staging/~mfisch/checkbox/network_device_info_enhance
- Jeff Lane : Approve
- Matt Fischer (community): Needs Resubmitting
-
Diff: 217 lines (+96/-44)2 files modifiedjobs/networking.txt.in (+1/-0)
scripts/network_device_info (+95/-44)
lp://staging/~roadmr/ubuntu/quantal/checkbox/0.14.7
- Ubuntu branches: Pending requested
-
Diff: 114866 lines (+22292/-18992)125 files modifiedcheckbox_qt/qt_interface.py (+7/-2)
debian/changelog (+51/-0)
debian/po/ast.po (+2/-2)
debian/po/cs.po (+2/-2)
debian/po/de.po (+2/-2)
debian/po/en_AU.po (+2/-2)
debian/po/en_GB.po (+2/-2)
debian/po/es.po (+2/-2)
debian/po/fr.po (+2/-2)
debian/po/he.po (+2/-2)
debian/po/hu.po (+2/-2)
debian/po/id.po (+2/-2)
debian/po/it.po (+2/-2)
debian/po/ja.po (+2/-2)
debian/po/nl.po (+2/-2)
debian/po/oc.po (+2/-2)
debian/po/pl.po (+2/-2)
debian/po/pt_BR.po (+2/-2)
debian/po/ro.po (+2/-2)
debian/po/ru.po (+2/-2)
debian/po/tr.po (+2/-2)
debian/po/uk.po (+2/-2)
debian/po/zh_CN.po (+2/-2)
debian/po/zh_TW.po (+2/-2)
jobs/audio.txt.in (+6/-6)
jobs/networking.txt.in (+4/-1)
jobs/suspend.txt.in (+5/-4)
patches/0.14.2 (+2/-1)
po/ace.po (+199/-198)
po/af.po (+199/-198)
po/am.po (+199/-198)
po/ar.po (+219/-203)
po/ast.po (+199/-198)
po/az.po (+199/-198)
po/be.po (+199/-198)
po/bg.po (+199/-198)
po/bn.po (+199/-198)
po/bo.po (+199/-198)
po/br.po (+199/-198)
po/bs.po (+199/-198)
po/ca.po (+199/-198)
po/ca@valencia.po (+199/-198)
po/ckb.po (+199/-198)
po/cs.po (+199/-198)
po/cy.po (+199/-198)
po/da.po (+199/-198)
po/de.po (+179/-178)
po/dv.po (+199/-198)
po/el.po (+199/-198)
po/en_AU.po (+415/-414)
po/en_CA.po (+199/-198)
po/en_GB.po (+891/-504)
po/eo.po (+202/-201)
po/es.po (+462/-216)
po/et.po (+199/-198)
po/eu.po (+199/-198)
po/fa.po (+199/-198)
po/fi.po (+202/-201)
po/fr.po (+1576/-489)
po/ga.po (+199/-198)
po/gd.po (+199/-198)
po/gl.po (+261/-208)
po/he.po (+199/-198)
po/hi.po (+199/-198)
po/hr.po (+199/-198)
po/hu.po (+199/-198)
po/hy.po (+199/-198)
po/id.po (+199/-198)
po/is.po (+199/-198)
po/it.po (+199/-198)
po/ja.po (+199/-198)
po/jbo.po (+199/-198)
po/ka.po (+199/-198)
po/kk.po (+199/-198)
po/km.po (+199/-198)
po/kn.po (+199/-198)
po/ko.po (+199/-198)
po/ku.po (+199/-198)
po/ky.po (+199/-198)
po/lt.po (+199/-198)
po/lv.po (+199/-198)
po/mk.po (+199/-198)
po/ml.po (+199/-198)
po/mr.po (+199/-198)
po/ms.po (+666/-251)
po/my.po (+199/-198)
po/nb.po (+199/-198)
po/nds.po (+199/-198)
po/ne.po (+199/-198)
po/nl.po (+199/-198)
po/nn.po (+199/-198)
po/oc.po (+204/-199)
po/pl.po (+199/-198)
po/ps.po (+199/-198)
po/pt.po (+266/-257)
po/pt_BR.po (+626/-236)
po/ro.po (+199/-198)
po/ru.po (+629/-222)
po/sd.po (+199/-198)
po/shn.po (+199/-198)
po/si.po (+199/-198)
po/sk.po (+199/-198)
po/sl.po (+288/-225)
po/sq.po (+217/-198)
po/sr.po (+199/-198)
po/sv.po (+199/-198)
po/ta.po (+199/-198)
po/te.po (+199/-198)
po/th.po (+199/-198)
po/tr.po (+204/-203)
po/ug.po (+199/-198)
po/uk.po (+218/-217)
po/ur.po (+199/-198)
po/uz.po (+199/-198)
po/vi.po (+199/-198)
po/zh_CN.po (+199/-198)
po/zh_HK.po (+199/-198)
po/zh_TW.po (+199/-198)
qt/frontend/qtfront.ui (+2/-2)
scripts/audio_settings (+12/-2)
scripts/color_depth_info (+1/-1)
scripts/frequency_governors_test (+5/-2)
scripts/graphics_driver (+4/-4)
scripts/graphics_modes_info (+1/-1)
scripts/network_device_info (+95/-44)
| Changed in checkbox: | |
| assignee: | nobody → Matt Fischer (mfisch) |
| status: | New → In Progress |
| Changed in checkbox: | |
| importance: | Undecided → Medium |
| Changed in checkbox: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
