Skip to content

github.com/oneapi-src/oneTBB#3278

Merged
jhheider merged 27 commits intopkgxdev:mainfrom
and-ri:new/github.com/oneapi-src/oneTBB
Oct 6, 2023
Merged

github.com/oneapi-src/oneTBB#3278
jhheider merged 27 commits intopkgxdev:mainfrom
and-ri:new/github.com/oneapi-src/oneTBB

Conversation

@and-ri
Copy link
Contributor

@and-ri and-ri commented Sep 10, 2023

closes #351

and-ri and others added 6 commits September 10, 2023 19:23
new file:   projects/github.com/oneapi-src/oneTBB/package.yml
new file:   projects/github.com/oneapi-src/oneTBB/sum1-100.cpp
@and-ri
Copy link
Contributor Author

and-ri commented Sep 25, 2023

Hi @jhheider
Please tell me, did I do something wrong? Or is this a bug?
https://github.com/teaxyz/pantry/actions/runs/6213833490/job/16865228066?pr=3278

fix-macho: writing /opt/github.com/oneapi-src/oneTBB/v2021.10.0/lib/python3.11/site-packages/tbb-2021.10.0-py3.11-macosx-11.0-x86_64.egg/lib/libtbb.12.10.dylib
/opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:86:in `codesign!': /opt/github.com/oneapi-src/oneTBB/v2021.10.0/lib/python3.11/site-packages/tbb-2021.10.0-py3.11-macosx-11.0-x86_64.egg/lib/libtbb.12.10.dylib: signing failed! (MachO::CodeSigningError)
	from /opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:66:in `fix'
	from /opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:247:in `block (2 levels) in <main>'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/find.rb:49:in `block (2 levels) in find'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/find.rb:48:in `catch'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/find.rb:48:in `block in find'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/find.rb:43:in `each'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/find.rb:43:in `find'
	from /opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:234:in `block in <main>'
	from /opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:233:in `each'
	from /opt/tea.xyz/brewkit/v0.45.5/share/brewkit/fix-machos.rb:233:in `<main>'
error: Uncaught Error: failed to fix machos
  if (!success) throw new Error("failed to fix machos")
                      ^
    at file:///opt/tea.xyz/brewkit/v0.45.5/libexec/fixup.ts:33:23
    at eventLoopTick (ext:core/01_core.js:183:11)
Error: Process completed with exit code 1.

@jhheider
Copy link
Contributor

Try calling codesign on that file from the command line. It might be that the header segment isn't long enough. There's a linker flag (-max_header_install_names, maybe? Other recipes have it) to make more space.

@and-ri
Copy link
Contributor Author

and-ri commented Oct 5, 2023

Something is wrong with gcc

@jhheider
Copy link
Contributor

jhheider commented Oct 5, 2023

believable. see:
#3530
pkgxdev/brewkit#181

btw, our make shouldn't be gmake. we've avoided the g prefixes. but once that brewkit pr is done, cc/make should be always available to brewkit.

@and-ri
Copy link
Contributor Author

and-ri commented Oct 6, 2023

I don't know how to make it work on Linux/aarch64. 🤷🏻‍♂️
But, in fact, many packages will depend on tbb, and then it becomes a problem.

@and-ri and-ri marked this pull request as ready for review October 6, 2023 11:23
@jhheider
Copy link
Contributor

jhheider commented Oct 6, 2023

For mariadb, this worked:

    linux/aarch64:
      ARGS:
      - -DCMAKE_CXX_FLAGS="-mno-outline-atomics"
      - -DCMAKE_C_FLAGS="-mno-outline-atomics"
      #^^ https://www.mail-archive.com/ports@freebsd.org/msg00418.html

@jhheider jhheider force-pushed the new/github.com/oneapi-src/oneTBB branch from d9bdbb0 to 7ce6607 Compare October 6, 2023 18:13
@jhheider
Copy link
Contributor

jhheider commented Oct 6, 2023

None of the usual tricks worked. I added a FIXME, so diligent contributors can find it.

@jhheider jhheider merged commit 929758b into pkgxdev:main Oct 6, 2023
@and-ri and-ri deleted the new/github.com/oneapi-src/oneTBB branch October 23, 2023 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

+tbb (275/548)

2 participants