Skip to content

Conversation

Sjors
Copy link
Member

@Sjors Sjors commented Jul 24, 2018

I just got this error on master:

make clean-all
make -j5
[...]
/bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
Preprocessing qt...
sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1

@Fuzzbawls
Copy link
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?

@Sjors
Copy link
Member Author

Sjors commented Jul 24, 2018

This line goes away again in #13696 by the way.

@Sjors
Copy link
Member Author

Sjors commented Jul 24, 2018

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 :-)

@fanquake
Copy link
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:

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

should work fine?

@Fuzzbawls
Copy link
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.

@Sjors
Copy link
Member Author

Sjors commented Jul 24, 2018

@fanquake I'll try sed -i.old for consistency.

@fanquake
Copy link
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.

@Fuzzbawls
Copy link
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.

@fanquake
Copy link
Member

utACK 12c93cb

@maflcko
Copy link
Member

maflcko commented Jul 25, 2018

utACK 12c93cb

@maflcko maflcko merged commit 12c93cb into bitcoin:master Jul 25, 2018
maflcko pushed a commit that referenced this pull request Jul 25, 2018
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
@Sjors Sjors deleted the 2018/07/depends-qt-macos branch July 25, 2018 14:00
UdjinM6 pushed a commit to UdjinM6/dash that referenced this pull request Jul 26, 2020
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
UdjinM6 pushed a commit to UdjinM6/dash that referenced this pull request Jul 27, 2020
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
UdjinM6 pushed a commit to UdjinM6/dash that referenced this pull request Jul 27, 2020
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 27, 2020
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 28, 2020
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Jul 2, 2021
12c93cb depends: use MacOS friendly sed syntax in qt.mk (Sjors Provoost)

Pull request description:

  I just got this error on master:
  ```
  make clean-all
  make -j5
  [...]
  /bitcoin/depends/sources/qttools-opensource-src-5.9.6.tar.xz: OK
  Preprocessing qt...
  sed: 1: "qtbase/mkspecs/bitcoin- ...": extra characters at the end of q command
  make: *** /bitcoin/depends/work/build/x86_64-apple-darwin17.7.0/qt/5.9.6-b48df1b490b/.stamp_preprocessed] Error 1
  ```

Tree-SHA512: 1eaee535c0d637a67801c7dd48e9cd5a037823f4e162a6f50e375787045fdb95929f409a88c7d000d8781a4edc497b5ccb1fd23a24e29324ec633157fadd371e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants