camera_test crashed with gi._glib.GError in function(): no element "ffmpegcolorspace"

Bug #1100594 reported by Sanjeev Gupta
30
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Undecided
Unassigned
checkbox (Ubuntu)
Fix Released
Medium
Jeff Lane 

Bug Description

This is on an IBM x201 . Camera works in applications normally.

ProblemType: Crash
DistroRelease: Ubuntu 13.04
Package: checkbox 0.15
ProcVersionSignature: Ubuntu 3.8.0-0.4-generic 3.8.0-rc3
Uname: Linux 3.8.0-0-generic x86_64
ApportVersion: 2.8-0ubuntu1
Architecture: amd64
Date: Tue Jan 15 13:18:00 2013
ExecutablePath: /usr/lib/checkbox/bin/camera_test
InstallationDate: Installed on 2012-11-05 (72 days ago)
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Release amd64 (20121017.5)
InterpreterPath: /usr/bin/python3.3
MarkForUpload: True
ProcCmdline: /usr/bin/python3 /usr/share/checkbox/scripts/camera_test still
PythonArgs: ['/usr/share/checkbox/scripts/camera_test', 'still']
SourcePackage: checkbox
Title: camera_test crashed with gi._glib.GError in function(): no element "ffmpegcolorspace"
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo

Related branches

Revision history for this message
Sanjeev Gupta (ghane) wrote :
tags: removed: need-duplicate-check
Changed in checkbox (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Jeff Lane  (bladernr) wrote :

Daniel pointed out part of the problem here... there are three things that need to be sorted out:

1: the requires: for camera/still needs to be cleaned up (the requires for gir1.2 is listed twice)

2: when fswebcam isn't installed we're falling back to gstreamer, but in this case that caused an exception to occur inside the exception handling for the fswebcam call. To fix this, we should just set a fallback-to-gstreamer flag in the exception handling for the fswebcam call then put an if flag=true: bit to do the actual fallback code. This should produce less confusing tracebacks.

3: ffmpegcolorspace was removed from gir1.2 1.0 and replaced with videoconvert... so we need to address this. so that could mean a third fallback.

Better, check for the version and set appropriate variables at the beginning of the scritpt:
<roadmr> bladernr: it's just "videoconverter or ffmpegcolorspace" and "video/x-raw or video/x-raw-yuv". That's the only change needed in local tests
<roadmr> bladernr: this was useful in figuring things out: http://gstreamer.freedesktop.org/data/doc/gstreamer/head/manual/html/chapter-porting-1.0.html
<roadmr> bladernr: right after "from gi.repository import Gst", we can check Gst.version(), it's a tuple

Changed in checkbox (Ubuntu):
status: New → Triaged
Jeff Lane  (bladernr)
Changed in checkbox (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Jeff Lane (bladernr)
Jeff Lane  (bladernr)
Changed in checkbox (Ubuntu):
status: In Progress → Fix Committed
Jeff Lane  (bladernr)
information type: Private → Public
Jeff Lane  (bladernr)
Changed in checkbox:
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (7.8 KiB)

This bug was fixed in the package checkbox - 0.16

---------------
checkbox (0.16) saucy; urgency=low

  * New upstream release (LP: #1178403):

  * Launchpad automated translation updates

  [ Jeff Lane ]
  * scripts/cpu_offlining: Modified script to no longer offline cpu0 to resolve
    a bug on ARM. Modified output so most of it is redirected to stderr for
    fail cases, we don't need that much for success cases. (LP: #1078897)
  * jobs/mediacard.txt.in: Modified test instructions to be less confusing
    (LP: #970857)
  * scripts/cpu_topology: define the cpuinfo nested dicts on creation rather
    than define elements during parsing of /proc/cpuinfo (LP: #1111878)
  * scripts/lsmod_info: Corrected error handling for the check_output() call to
    trap the correct error. (LP: #1103647)
  * jobs/camera.txt.in: removed an extraneous requres line for gir1.2
    scripts/camera_test: added code to determine what version of gst we're
    using and set video type and plugin accordingly. (LP: #1100594)
  * scripts/network_check: added ability to specify custom target URL for
    debugging failures (LP: #1128017)
  * scripts/sleep_test_log_check: added new script to parse fwts logs for
    errors
    jobs/hibernate.txt.in, jobs/stress.txt.in, jobs/suspend.txt.in: added jobs
    to use the sleep_test_log_check script after s3/s4 tests. Modified current
    s3/s4 tests to use the new None fail level for fwts_test
    scripts/fwts_test: no longer fails on fwts errors if "-f none" is chosen.
    Now it should only fail by manual indication if the actual sleep action
    fails. (LP: #1169922)
  * scripts/bluetooth_test: test no longer generates odd Broken Pipe error when
    the dpkg bit verifies obexftp is installed (LP: #1169488)
  * jobs/networking.txt.in: fixed networking/info jobs that were not being run
    because $output was being resolved too early (LP: #1065983)
  * scripts/memory_compare: cleaned up the output to be more explanatory and
    easier to visually parse (LP: #1072666)
  * jobs/miscellanea.txt.in: Added jobs for manual verification of PXE boot and
    remote IPMI to improve server test coverage.
  * jobs/led.txt.in: Modified Jobs: led/power, led/wlan, led/wlan-disabled. New
    jobs: led/power-blink-suspend, led/suspend, led/mute
    jobs/keys.txt.in: modified keys/wireless
  * plugins/lock_prompt.py: added a lock release to cleanly clear lock on stop.
    plugins/persist_prompt.py: promoted save to run before lock release.
    checkbox/lib/fifo.py: trap OSError exception at close when the input/output
    fifo fds disappear before fifo.close() can get to them. (LP: #115561)
  * jobs/info.txt.in: fixed bad driver name in audio_codecs job (LP: #1165215)
  * jobs/mediacard.txt.in: reduce test file size for MMC to 64MB (LP: #1167214)
  * jobs/led.txt.in, jobs/suspend.txt.in: removed bits about WLAN LED flashing
    from the WLAN jobs as that no longer happens on data tx/rx (LP: #1157606)
  * jobs/power-management.txt.in: added manual reboot and shutdown tests for
    server and Xen testing. Took the opportunity to correct punctuation in the
    descriptions or other manual jobs.
    jobs/virtualization.txt.in: added virtualization/x...

Read more...

Changed in checkbox (Ubuntu):
status: Fix Committed → Fix Released
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.