contrib: remove no longer used contrib/bitcoin-qt.pro #21695

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:210415-pro changing 2 files +0 −24
  1. hebasto commented at 12:46 PM on April 15, 2021: member

    From IRC:

    <hebasto> wumpus: I cannot see any way how the contrib/bitcoin-qt.pro is used in the translation process, neither in the main repo nor in https://github.com/bitcoin-core/bitcoin-maintainer-tools. Besides it looks outdated and unmaintained. May I ask you to confirm/deny my assumption? <wumpus> hebasto: it is not used for anything, it exists to be able to edit the qt forms in qt designer nothing more <wumpus> i'm not sure if it is even necessary for that, but it is why it is there <hebasto> wumpus: thanks, qt designer does not need *.pro file at all <hebasto> maybe qt creator does <wumpus> feel free to create a PR to remove it, best way to find out if someone wants to keep it, you are right it hasn't been updated in a long time <hebasto> ok <wumpus> fwiw, the only question i get about it ever is why it exists <hebasto> it was in use with qmake years ago (what I found digging into the repo history) <wumpus> yes, that was the original reason, but when we switched to automake it was kept around for use w/ qt's GUI tools <hebasto> I've noticed it in https://github.com/bitcoin/bitcoin/blame/master/doc/translation_process.md#L25 <wumpus> what it says there is definitely not true anymore

  2. Remove no longer used contrib/bitcoin-qt.pro from the repo 5f2be6e71e
  3. laanwj commented at 12:47 PM on April 15, 2021: member

    ACK 5f2be6e71e6130b58ebfbf81aaf48ce90dd9d179

  4. hebasto commented at 12:52 PM on April 15, 2021: member

    @jarolrod

    As you was the last person who checked Using Qt Creator as an IDE, could your confirm/deny the necessary of the contrib/bitcoin-qt.pro for such setup?

  5. DrahtBot added the label Scripts and tools on Apr 15, 2021
  6. jarolrod commented at 4:07 PM on April 15, 2021: member

    ACK on removing the line about bitcoin-qt.pro from doc/translation_process.md.

    The bitcoin-qt.pro file is a convenient way to quickly open up the GUI ui files and start editing them. The flow is (Using Qt Creator) Open Project -> Select pro file -> Edit UI Files. This can be the recommended flow to suggest to designers who'd like to make UI changes. I think it should stay for this reason, and I can add a note to the Qt readme about it.

    The pro file is a bit outdated. To bring the pro file up to speed, we'd need to add the following missing .ui files under FORMS +=:

    • modeloverlay.ui
    • psbtoperationsdialog.ui
  7. hebasto commented at 4:13 PM on April 15, 2021: member

    @jarolrod Does Qt Creator allow to open *.ui file directly (as Qt Designer does)?

  8. hebasto commented at 4:17 PM on April 15, 2021: member

    The pro file is a bit outdated. To bring the pro file up to speed, we'd need to add the following missing .ui files under FORMS +=:

    * modeloverlay.ui
    
    * psbtoperationsdialog.ui

    ... and remove aboutdialog.ui...

  9. jarolrod commented at 4:23 PM on April 15, 2021: member

    @hebasto yeah you can open up UI files individually. This pro file opens up all of them for you as well as the resources folder. I think it's more convenient to have all of them open. I don't have a strong opinion on this

    It could be that no one uses it (i dont) and it could be wasting time to maintain this.

  10. hebasto commented at 4:24 PM on April 15, 2021: member

    ... it could be wasting time to maintain this.

    That is my concern.

  11. Bosch-0 commented at 2:13 AM on April 16, 2021: none

    I personally do not use this as a designer. Qt Creator has its uses for working with Qt widgets to understand what can and can't be modified on the design side but outside that it's use as a design tool isn't great

    ACK from me if this files requires resources to maintain

  12. jarolrod commented at 2:46 AM on April 16, 2021: member

    ACK 5f2be6e71e6130b58ebfbf81aaf48ce90dd9d179

    ACK because I don't have a strong opinion, the file is not used very much, and the resulting conversation would be more man-hours than is used to maintain this file.

    Qt Creator has its uses for working with Qt widgets to understand what can and can't be modified on the design side but outside that it's use as a design tool isn't great

    There's a difference between designing and implementing a design by changing the UI files. Qt Creator, which packages Qt Designer, is the best tool for performing design work that changes the GUI as it's WYSIWYG.

    we could add the two lines and remove the one line, that's the cost of maintenance for now. Or just remove the file. Again no strong opinion.

  13. fanquake renamed this:
    Remove no longer used contrib/bitcoin-qt.pro from the repo
    contrib: remove no longer used contrib/bitcoin-qt.pro
    on Apr 16, 2021
  14. fanquake merged this on Apr 16, 2021
  15. fanquake closed this on Apr 16, 2021

  16. sidhujag referenced this in commit 331480793c on Apr 16, 2021
  17. hebasto deleted the branch on Apr 16, 2021
  18. DrahtBot locked this on Aug 16, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 21:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me