Include a new Internal Screenshot job

Bug #1348539 reported by Ara Pulido
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Wishlist
Daniel Manrique

Bug Description

When we created the external screenshot job (using an external USB webcam), we removed the one we had for Internal Screenshots:

plugin: shell
name: screenshot
requires:
 package.name == 'xorg'
 package.name == 'imagemagick'
command: import -window root ${CHECKBOX_DATA}/screenshot.png
_description:
 Captures a screenshot.

plugin: attachment
name: screenshot.png
depends: screenshot
command: cat ${CHECKBOX_DATA}/screenshot.png

We would need to add back something similar to the checkbox provider, as it is an useful test for external testing.

Daniel comments:

"If we want to bring an "internal screenshot" test back, I'd suggest
looking at using something other than imagemagick, which sometimes was
unreliable. We can use gnome-screenshot, use the dbus
org.Gnome.Screenshot stuff (what happens when you press PrtScr), or
look into another option that may have better compatibility with
tablet/phone; maybe some qt-based thing."

Related branches

Ara Pulido (ara)
Changed in plainbox-provider-checkbox:
importance: Undecided → Wishlist
description: updated
description: updated
Revision history for this message
Daniel Manrique (roadmr) wrote :

It took me 15 minutes (watch in hand) to write the proposed set of gnome-screenshot-based jobs which solve this requirement for desktop installations of Ubuntu; that way, if we need to refactor or outright trash and start anew, it won't be a lot of wasted work :)

- We can refine this to something using Qt that will also work on Ubuntu touch.
- I only do a simple screengrab. I can easily turn this into a generator to get a screenshot for each card, and either an attacher for each card or a global attachment processor to create a screenshots tarball.

But those two are more effort so I suggest looking at them if/when we think we'll need them.

Changed in plainbox-provider-checkbox:
status: New → In Progress
milestone: none → 0.9
assignee: nobody → Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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