Overview
Request 636631 superseded
- Add reproducible-qrc-time.patch to use SOURCE_DATE_EPOCH
for qrc embedded timestamps to make bitcoin-qt build reproducible
(https://bugreports.qt.io/browse/QTBUG-62511 , boo#1047218)
Fixes ~40 Factory packages:
bitcoin breeze5-icons calamares chessx equalx falkon focuswriter gcompris-qt goldendict kreport krita libqt5-qtscript libqt5-qtwebkit librepilot linphoneqt lyx mathgl nitroshare notepadqq paraview photoqt plasma5-desktop qbittorrent qmmp-plugin-pack qt5ct qtox rbutil rosegarden rssguard sigil spectator sqlitebrowser swift-im syntax-highlighting tomahawk umbrello vokoscreen wireshark yubikey-manager-qt yubioath-desktop
- Created by bmwiedemann
- In state superseded
- Supersedes 636629
- Superseded by 643992
Request History
bmwiedemann created request
- Add reproducible-qrc-time.patch to use SOURCE_DATE_EPOCH
for qrc embedded timestamps to make bitcoin-qt build reproducible
(https://bugreports.qt.io/browse/QTBUG-62511 , boo#1047218)
Fixes ~40 Factory packages:
bitcoin breeze5-icons calamares chessx equalx falkon focuswriter gcompris-qt goldendict kreport krita libqt5-qtscript libqt5-qtwebkit librepilot linphoneqt lyx mathgl nitroshare notepadqq paraview photoqt plasma5-desktop qbittorrent qmmp-plugin-pack qt5ct qtox rbutil rosegarden rssguard sigil spectator sqlitebrowser swift-im syntax-highlighting tomahawk umbrello vokoscreen wireshark yubikey-manager-qt yubioath-desktop
bmwiedemann superseded request
superseded by 643992
Can this go upstream instead? If not, I'd prefer if QT_RCC_SOURCE_DATE_OVERRIDE was set in the environment.
not that easy. We inject most other variables through https://github.com/openSUSE/post-build-checks/pull/20/files but when it runs, SOURCE_DATE_EPOCH is not yet set by rpmbuild. And I heard, QT has a policy of requiring env-variable names to be prefixed by QT_ . The original proposed patch there used SOURCE_DATE_EPOCH but was stuck in review for ages. We could try again...
Trying again is probably the best option - it's a standard env variable so Qt needs to follow that.
done: https://codereview.qt-project.org/243636