Bug 1255054 - qdoc valgrind wrapper ending up in installed files
Summary: qdoc valgrind wrapper ending up in installed files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: qt5-qtbase
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1256192
TreeView+ depends on / blocked
 
Reported: 2015-08-19 13:59 UTC by Sandro Mani
Modified: 2015-08-24 15:03 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-08-24 15:03:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Sandro Mani 2015-08-19 13:59:48 UTC
Description of problem:
These lines in qt5-qtbase.spec

make html_docs || \
  mv bin/qdoc bin/qdoc.orig && install %{SOURCE2} bin/qdoc && \
  make html_docs && \
  mv bin/qdoc.orig bin/qdoc -f

fail to correctly restore qdoc with qdoc.orig if also the second make html_docs fails.

This is causing qt-creator to fail to build:
https://koji.fedoraproject.org/koji/getfile?taskID=10754160&name=build.log&offset=-4000

Version-Release number of selected component (if applicable):
qt5-qtbase-5.5.0-16.fc24.x86_64

Comment 1 Rex Dieter 2015-08-19 14:04:08 UTC
Interesting, I'd expect the build to fail of 'make html_docs' fails though

Comment 2 Rex Dieter 2015-08-19 14:06:48 UTC
%changelog
* Wed Aug 19 2015 Rex Dieter <rdieter> 5.5.0-17
- unconditionally undo valgrind hack when done (#1255054)

Comment 3 Sandro Mani 2015-08-19 14:07:36 UTC
Thanks.

Comment 4 Sandro Mani 2015-08-20 22:01:43 UTC
The build failed, something like this should work though:

diff --git a/qt5-qtbase.spec b/qt5-qtbase.spec
index 8d8335e..bba391a 100644
--- a/qt5-qtbase.spec
+++ b/qt5-qtbase.spec
@@ -484,10 +484,10 @@ pushd src/xml; ../../bin/qmake; popd
 # see also https://bugreports.qt-project.org/browse/QTBUG-42071
 QT_HASH_SEED=0; export QT_HASH_SEED
 %if 0%{?valgrind}
-make html_docs || \
+make html_docs || (\
   mv bin/qdoc bin/qdoc.orig && install %{SOURCE2} bin/qdoc && \
-  make html_docs
-  mv bin/qdoc.orig bin/qdoc -f
+  make html_docs)
+  [ -e bin/qdoc.orig ] && mv -f bin/qdoc.orig bin/qdoc
 %else
 make html_docs
 %endif


http://koji.fedoraproject.org/koji/taskinfo?taskID=10769302

Comment 5 Sandro Mani 2015-08-24 15:03:05 UTC
Build succeeded, thanks.


Note You need to log in before you can comment on or make changes to this bug.