From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Sun, 21 Dec 2014 23:53:38 +0100 Subject: [Buildroot] [PATCH 1/3] pinentry: pinentry-qt4 - make the accessibility part optional In-Reply-To: <1418399947-12517-1-git-send-email-Vincent.Riera@imgtec.com> References: <1418399947-12517-1-git-send-email-Vincent.Riera@imgtec.com> Message-ID: <20141221225338.GD4207@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Vicente, All, On 2014-12-12 15:59 +0000, Vicente Olivert Riera spake thusly: > Backporting an upstream patch to check if the Qt libraries have support > for Qt Accessibility before using it. Otherwise it will raise errors > like these one: > > main.cpp: In function 'int qt_cmd_handler(pinentry_t)': > main.cpp:220:51: error: 'class QAbstractButton' has no member named > 'setAccessibleDescription' > > Backported upstream patch: > http://git.gnupg.org/cgi-bin/gitweb.cgi?p=pinentry.git;a=commit;h=484465338ea6f5f12eb1341ca0ef84aa79224ed4 > > Fixes: > http://autobuild.buildroot.net/results/62c/62ca977a7a44d1ad2bbf9e5bc2df716d469484de/ > > Signed-off-by: Vicente Olivert Riera I independently came to submit this same patch, as well as the next one, and they have been applied by Thomas: http://git.buildroot.org/buildroot/commit/package/pinentry?id=0ec5167d0cccd8628537ac98222cfc528b08022b http://git.buildroot.org/buildroot/commit/package/pinentry?id=4d58a27da458c438d375250c74fbc4444c9b5d3d Sorry for the mishap, I did not see your patches before submitting. I've marked them Superseded in Patchwork. Regards, Yann E. MORIN. > --- > ...-qt4-make-the-accessibility-part-optional.patch | 112 ++++++++++++++++++++ > 1 files changed, 112 insertions(+), 0 deletions(-) > create mode 100644 package/pinentry/0001-pinentry-qt4-make-the-accessibility-part-optional.patch > > diff --git a/package/pinentry/0001-pinentry-qt4-make-the-accessibility-part-optional.patch b/package/pinentry/0001-pinentry-qt4-make-the-accessibility-part-optional.patch > new file mode 100644 > index 0000000..1b49edd > --- /dev/null > +++ b/package/pinentry/0001-pinentry-qt4-make-the-accessibility-part-optional.patch > @@ -0,0 +1,112 @@ > +pinentry-qt4: make the accessibility part optional > + > +Backported upstream patch: > + http://git.gnupg.org/cgi-bin/gitweb.cgi?p=pinentry.git;a=commit;h=484465338ea6f5f12eb1341ca0ef84aa79224ed4 > + > +Signed-off-by: Vicente Olivert Riera > + > +From 484465338ea6f5f12eb1341ca0ef84aa79224ed4 Mon Sep 17 00:00:00 2001 > +From: Vicente Olivert Riera > +Date: Mon, 8 Dec 2014 16:42:42 +0000 > +Subject: [PATCH] pinentry-qt4: make the accessibility part optional > + > +Check if the Qt libraries have support for QT Accessibility before using > +it. Otherwise it will raise error like these one: > + > +main.cpp: In function 'int qt_cmd_handler(pinentry_t)': > +main.cpp:220:51: error: 'class QAbstractButton' has no member named > +'setAccessibleDescription' > + > +Signed-off-by: Vicente Olivert Riera > +--- > + qt4/main.cpp | 3 ++- > + qt4/pinentryconfirm.cpp | 2 ++ > + qt4/pinentrydialog.cpp | 10 ++++++++++ > + 3 files changed, 14 insertions(+), 1 deletions(-) > + > +diff --git a/qt4/main.cpp b/qt4/main.cpp > +index 106999e..b2a69f2 100644 > +--- a/qt4/main.cpp > ++++ b/qt4/main.cpp > +@@ -217,8 +217,9 @@ qt_cmd_handler (pinentry_t pe) > + for ( size_t i = 0 ; i < sizeof buttonLabels / sizeof *buttonLabels ; ++i ) > + if ( (buttons & buttonLabels[i].button) && !buttonLabels[i].label.isEmpty() ) { > + box.button( buttonLabels[i].button )->setText( buttonLabels[i].label ); > ++#ifndef QT_NO_ACCESSIBILITY > + box.button( buttonLabels[i].button )->setAccessibleDescription ( buttonLabels[i].label ); > +- > ++#endif > + } > + > + box.setIconPixmap( icon() ); > +diff --git a/qt4/pinentryconfirm.cpp b/qt4/pinentryconfirm.cpp > +index dfbd19f..6b3d545 100644 > +--- a/qt4/pinentryconfirm.cpp > ++++ b/qt4/pinentryconfirm.cpp > +@@ -30,8 +30,10 @@ PinentryConfirm::PinentryConfirm(Icon icon, int timeout, const QString &title, > + connect(_timer, SIGNAL(timeout()), this, SLOT(slotTimeout())); > + _timer->start(timeout*1000); > + } > ++#ifndef QT_NO_ACCESSIBILITY > + setAccessibleDescription (desc); > + setAccessibleName (title); > ++#endif > + raiseWindow (this); > + } > + > +diff --git a/qt4/pinentrydialog.cpp b/qt4/pinentrydialog.cpp > +index 3a6dacc..456f022 100644 > +--- a/qt4/pinentrydialog.cpp > ++++ b/qt4/pinentrydialog.cpp > +@@ -217,7 +217,9 @@ void PinEntryDialog::setDescription( const QString& txt ) > + { > + _desc->setVisible( !txt.isEmpty() ); > + _desc->setText( txt ); > ++#ifndef QT_NO_ACCESSIBILITY > + _desc->setAccessibleDescription ( txt ); > ++#endif > + _icon->setPixmap( icon() ); > + setError( QString::null ); > + } > +@@ -231,7 +233,9 @@ void PinEntryDialog::setError( const QString& txt ) > + { > + if( !txt.isNull() )_icon->setPixmap( icon( QStyle::SP_MessageBoxCritical ) ); > + _error->setText( txt ); > ++#ifndef QT_NO_ACCESSIBILITY > + _error->setAccessibleDescription ( txt ); > ++#endif > + _error->setVisible( !txt.isEmpty() ); > + } > + > +@@ -264,14 +268,18 @@ QString PinEntryDialog::prompt() const > + void PinEntryDialog::setOkText( const QString& txt ) > + { > + _ok->setText( txt ); > ++#ifndef QT_NO_ACCESSIBILITY > + _ok->setAccessibleDescription ( txt ); > ++#endif > + _ok->setVisible( !txt.isEmpty() ); > + } > + > + void PinEntryDialog::setCancelText( const QString& txt ) > + { > + _cancel->setText( txt ); > ++#ifndef QT_NO_ACCESSIBILITY > + _cancel->setAccessibleDescription ( txt ); > ++#endif > + _cancel->setVisible( !txt.isEmpty() ); > + } > + > +@@ -279,7 +287,9 @@ void PinEntryDialog::setQualityBar( const QString& txt ) > + { > + if (_have_quality_bar) { > + _quality_bar_label->setText( txt ); > ++#ifndef QT_NO_ACCESSIBILITY > + _quality_bar_label->setAccessibleDescription ( txt ); > ++#endif > + } > + } > + > +-- > +1.7.1 > + > -- > 1.7.1 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'