depends: use MacOS friendly sed syntax in qt.mk #13750

pull Sjors wants to merge 1 commits into bitcoin:master from Sjors:2018/07/depends-qt-macos changing 1 files +1 −1
  1. Sjors commented at 10:46 am on July 24, 2018: member

    I just got this error on master:

    0make clean-all
    1make -j5
    2[...]
    3/bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
    4Preprocessing qt...
    5sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
    6make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
    
  2. fanquake added the label Build system on Jul 24, 2018
  3. Fuzzbawls commented at 10:50 am on July 24, 2018: contributor
    With all these MacOS specific exceptions popping up recently, I almost wonder if it would not be more beneficial to “expect” (or even enforce) brew GNU versions of tools to be installed rather than the archaic pseudo-BSD MacOS versions?
  4. Sjors commented at 10:51 am on July 24, 2018: member
    This line goes away again in #13696 by the way.
  5. Sjors commented at 10:53 am on July 24, 2018: member

    archaic pseudo-BSD MacOS versions

    Maybe, but apparently OpenBSB often encounters the same issues (cc @laanwj), so macOS devs help catch problems there this way :-)

  6. fanquake commented at 11:03 am on July 24, 2018: member

    This is not the first time sed has been used in depends, there are already multiple uses in the same file that all work fine on macOS. Something like:

    0sed -i.old "s/arm-linux-gnueabi-/$(host)-/g" qtbase/mkspecs/bitcoin-linux-g++/qmake.conf && \
    

    should work fine?

  7. Fuzzbawls commented at 11:07 am on July 24, 2018: contributor
    @fanquake yes, my comment stems from knowing that (in general) there are some differences between MacOS sed and GNU sed…irregardless of wither those differences have ever surfaced here.
  8. Sjors commented at 11:08 am on July 24, 2018: member
    @fanquake I’ll try sed -i.old for consistency.
  9. fanquake commented at 11:12 am on July 24, 2018: member
    @Fuzzbawls Sure, sorry if I sounded dismissive. There definitely are some differences, i.e something like #13514 which I stumbled on when playing around with iOS depends builds.
  10. depends: use MacOS friendly sed syntax in qt.mk 12c93cb85b
  11. Sjors force-pushed on Jul 24, 2018
  12. Fuzzbawls commented at 1:09 pm on July 24, 2018: contributor

    @fanquake hey no worries mate! I myself don’t even know all the (seemingly stupid) differences between MacOS tools and GNU (or even more recent BSD).

    I just know there are differences…and those differences may ore may not ever apply here…and if they do…lets make sure they are exclusive to MacOS and exclude them for being archaic (require brew GNU equivilent in this case) rather than cater to such exclusions.

  13. fanquake commented at 10:57 am on July 25, 2018: member
    utACK 12c93cb
  14. MarcoFalke commented at 10:59 am on July 25, 2018: member
    utACK 12c93cb85b6cb075d22323ff02acebbe05e42375
  15. MarcoFalke merged this on Jul 25, 2018
  16. MarcoFalke closed this on Jul 25, 2018

  17. MarcoFalke referenced this in commit c883653202 on Jul 25, 2018
  18. Sjors deleted the branch on Jul 25, 2018
  19. UdjinM6 referenced this in commit e1d8a8c918 on Jul 26, 2020
  20. UdjinM6 referenced this in commit 3f5284b0e4 on Jul 27, 2020
  21. UdjinM6 referenced this in commit 8172157b13 on Jul 27, 2020
  22. PastaPastaPasta referenced this in commit eb578984d3 on Jul 27, 2020
  23. PastaPastaPasta referenced this in commit f8c309ca54 on Jul 28, 2020
  24. CryptoCentric referenced this in commit 75d74afdad on Jul 2, 2021
  25. DrahtBot locked this on Sep 8, 2021

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: 2024-12-24 18:12 UTC

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