Merge lp:~schwann/qtubuntu-camera/camera-torch-support into lp:qtubuntu-camera

Proposed by Günter Schwann
Status: Merged
Approved by: Günter Schwann
Approved revision: 54
Merged at revision: 62
Proposed branch: lp:~schwann/qtubuntu-camera/camera-torch-support
Merge into: lp:qtubuntu-camera
Diff against target: 30 lines (+4/-2)
1 file modified
src/aalcameraflashcontrol.cpp (+4/-2)
To merge this branch: bzr merge lp:~schwann/qtubuntu-camera/camera-torch-support
Reviewer Review Type Date Requested Status
Victor Tuson Palau (community) Approve
PS Jenkins bot continuous-integration Approve
Ubuntu Phablet Team Pending
Review via email: mp+168468@code.launchpad.net

Commit message

Support FlashTorch

Description of the change

Support FlashTorch
This enable support for the torch component as well.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

I try the deb but broke the cam and the app running via QTcreator gives the following error
/usr/lib/arm-linux-gnueabihf/qt5/bin/qmlscene: symbol lookup error: /usr/lib/arm-linux-gnueabihf/qt5/plugins/mediaservice/libaalcamera.so: undefined symbol: _ZN12SharedSignal8instanceEv

Revision history for this message
Günter Schwann (schwann) wrote :

Ok - so we need to wait for an image including the new libhybris (with video support)

Revision history for this message
Victor Tuson Palau (vtuson) wrote :

> Ok - so we need to wait for an image including the new libhybris (with video
> support)
please ping me in irc when this us ready to test. thanks

53. By Günter Schwann

Trunk merged

Revision history for this message
Günter Schwann (schwann) wrote :

Victor - it's fixed now. You can test it again (didn't see you on IRC ;)

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Günter Schwann (schwann) wrote :

The debian package of the last build here on the saucy image-13 works fine for me.

54. By Günter Schwann

Trunk merged

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

using the latest build Camera.FlashTorch works fine.

Revision history for this message
Victor Tuson Palau (vtuson) wrote :

WFM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/aalcameraflashcontrol.cpp'
2--- src/aalcameraflashcontrol.cpp 2013-06-05 16:11:12 +0000
3+++ src/aalcameraflashcontrol.cpp 2013-07-02 13:37:08 +0000
4@@ -39,7 +39,8 @@
5 {
6 if (m_service->isBackCameraUsed()) {
7 if (mode==QCameraExposure::FlashAuto || mode==QCameraExposure::FlashOff ||
8- mode==QCameraExposure::FlashOn || mode==QCameraExposure::FlashVideoLight) {
9+ mode==QCameraExposure::FlashOn || mode==QCameraExposure::FlashVideoLight ||
10+ mode==QCameraExposure::FlashTorch) {
11 return true;
12 }
13 } else {
14@@ -98,6 +99,7 @@
15 case QCameraExposure::FlashOn:
16 return FLASH_MODE_ON;
17 case QCameraExposure::FlashVideoLight:
18+ case QCameraExposure::FlashTorch:
19 return FLASH_MODE_TORCH;
20 case QCameraExposure::FlashAuto:
21 default:
22@@ -113,7 +115,7 @@
23 case FLASH_MODE_ON:
24 return QCameraExposure::FlashOn;
25 case FLASH_MODE_TORCH:
26- return QCameraExposure::FlashVideoLight;
27+ return QCameraExposure::FlashTorch;
28 case FLASH_MODE_AUTO:
29 default:
30 return QCameraExposure::FlashAuto;

Subscribers

People subscribed via source and target branches