Skip to content

llvmPackages: use libcxxrt on FreeBSD#211230

Merged
alyssais merged 3 commits intoNixOS:masterfrom
alyssais:libcxxrt
Jan 24, 2023
Merged

llvmPackages: use libcxxrt on FreeBSD#211230
alyssais merged 3 commits intoNixOS:masterfrom
alyssais:libcxxrt

Conversation

@alyssais
Copy link
Member

@alyssais alyssais commented Jan 17, 2023

Description of changes

FreeBSD doesn't use LLVM's cxxabi implementation, for backwards compatibility reasons. Software expects the libcxxrt API when building on FreeBSD. This fixes the build of pkgsCross.x86_64-freebsd.boost.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@alyssais alyssais added the 6.topic: bsd Running or building packages on BSD label Jan 17, 2023
@alyssais alyssais marked this pull request as draft January 17, 2023 12:23
@alyssais
Copy link
Member Author

alyssais commented Jan 19, 2023

Eval is failing because of LLVMs 5 and 6 again. >:(

@alyssais alyssais force-pushed the libcxxrt branch 3 times, most recently from 33b3365 to 2c85dbe Compare January 19, 2023 13:46
@github-actions github-actions bot added the 6.topic: stdenv Standard environment label Jan 19, 2023
Ericson2314
Ericson2314 reviewed Jan 19, 2023
View reviewed changes
@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Jan 19, 2023
@ofborg ofborg bot requested review from 7c6f434c, dtzWill, lovek323 and primeos January 19, 2023 16:36
@ofborg ofborg bot added 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Jan 19, 2023
@alyssais
Copy link
Member Author

alyssais commented Jan 19, 2023

Huh, I guess this doesn't really need to target staging.

@alyssais alyssais changed the base branch from staging to master January 19, 2023 18:17
@alyssais alyssais force-pushed the libcxxrt branch 2 times, most recently from 99edc9e to 4b83d46 Compare January 20, 2023 13:08
@ofborg ofborg bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. and removed 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. labels Jan 20, 2023
@alyssais alyssais marked this pull request as ready for review January 20, 2023 16:36
alyssais added a commit to alyssais/nixpkgs that referenced this pull request Jan 22, 2023
@alyssais
gperftools: broaden platforms
aa7627a 
Builds fine for NetBSD, will build for FreeBSD once
NixOS#211230 is merged, and the README
mentions some amount of Windows support.
@alyssais alyssais mentioned this pull request Jan 22, 2023
Merged
13 tasks
@alyssais
llvmPackages: don't exclude FreeBSD from -lunwind
0bab802 
The exception for FreeBSD was added in 0afe9d1 ("freebsd packages:
Init at 13.1"), but it seems to have been erroneous, as e.g. ncurses
fails to build:

	x86_64-unknown-freebsd13-clang++  -o demo ../obj_s/demo.o -L../lib -lncurses++w -L../lib -lformw -lmenuw -lpanelw -lncursesw   -lutil     -DHAVE_CONFIG_H -DBUILDING_NCURSES_CXX -I../c++ -I. -I../include  -DNDEBUG -O2  -fPIC -DPIC
	x86_64-unknown-freebsd13-ld: error: undefined symbol: _Unwind_Resume
	>>> referenced by demo.cc
	>>>               ../obj_s/demo.o:(TestApplication::run())
	>>> referenced by demo.cc
	>>>               ../obj_s/demo.o:(_GLOBAL__sub_I_demo.cc)
	>>> referenced by demo.cc
	>>>               ../obj_s/demo.o:(NCursesUserItem<UserData>::NCursesUserItem(char const*, char const*, UserData const*))
	>>> referenced 46 more times
	clang-12: error: linker command failed with exit code 1 (use -v to see invocation)

This is fixed by undoing the change, adding -lunwind on FreeBSD.
@alyssais
Copy link
Member Author

alyssais commented Jan 22, 2023

I've split the -lunwind change into its own commit, since it turns out it's needed independently of the change to libcxxrt. But I don't think there's any point in splitting it into a separate PR.

Ericson2314
Ericson2314 reviewed Jan 24, 2023
View reviewed changes
@ofborg ofborg bot requested a review from Ericson2314 January 24, 2023 19:03
@alyssais
llvmPackages: use libcxxrt on FreeBSD
1d20fb6 
FreeBSD doesn't use LLVM's cxxabi implementation, for backwards
compatibility reasons.  Software expects the libcxxrt API when
building on FreeBSD.  This fixes the build of
pkgsCross.x86_64-freebsd.boost.
@alyssais
Copy link
Member Author

alyssais commented Jan 24, 2023

Updated to be less stringy based on a suggestion from @Ericson2314 on Matrix. We now pass cxxabi around as a package, and look at its pname when necessary to tell them apart, rather than passing a string around and trying to map it back to a packag.

@alyssais alyssais merged commit 2ae30c9 into NixOS:master Jan 24, 2023
@alyssais alyssais deleted the libcxxrt branch January 24, 2023 21:18
@rrbutani rrbutani mentioned this pull request Jan 25, 2023
Merged
92 tasks
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Jan 27, 2023
@rrbutani @wegank
llvmPackages_15: apply NixOS#211230 to llvmPackages_15
4fabcf4 
See the comments here for context: NixOS#194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
github-actions bot pushed a commit that referenced this pull request Jan 28, 2023
@rrbutani @wegank @github-actions
llvmPackages_15: apply #211230 to llvmPackages_15
97a97f1 
See the comments here for context: #194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
(cherry picked from commit 4fabcf4)
K900 pushed a commit that referenced this pull request Jan 28, 2023
@rrbutani @wegank @K900
llvmPackages_15: apply #211230 to llvmPackages_15
55b8f72 
See the comments here for context: #194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
xanderio pushed a commit to xanderio/nixpkgs that referenced this pull request Feb 13, 2023
@rrbutani @wegank @xanderio
llvmPackages_15: apply NixOS#211230 to llvmPackages_15
a743872 
See the comments here for context: NixOS#194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
RaitoBezarius pushed a commit that referenced this pull request Mar 26, 2023
@rrbutani @wegank @RaitoBezarius
llvmPackages_git: apply #211230 to llvmPackages_git
2f0ae7f 
See the comments here for context: #194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
RaitoBezarius pushed a commit that referenced this pull request Apr 5, 2023
@rrbutani @wegank @RaitoBezarius
llvmPackages_git: apply #211230 to llvmPackages_git
5fbf2cd 
Port of 4fabcf4 ("llvmPackages_15:
apply #211230 to llvmPackages_15").

See the comments here for context: #194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
rrbutani added a commit to rrbutani/nixpkgs that referenced this pull request Apr 5, 2023
@rrbutani @wegank
llvmPackages_git: apply NixOS#211230 to llvmPackages_git
98918de 
Port of 4fabcf4 ("llvmPackages_15:
apply NixOS#211230 to llvmPackages_15").

See the comments here for context: NixOS#194634 (comment)

Co-authored-by: Weijia Wang <[email protected]>
@ghost ghost mentioned this pull request May 15, 2024
Merged
14 tasks
@rrbutani rrbutani added the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Ericson2314 Ericson2314 Ericson2314 approved these changes

@matthewbauer matthewbauer Awaiting requested review from matthewbauer

@lovek323 lovek323 Awaiting requested review from lovek323

@dtzWill dtzWill Awaiting requested review from dtzWill

@7c6f434c 7c6f434c Awaiting requested review from 7c6f434c

@primeos primeos Awaiting requested review from primeos

@sternenseemann sternenseemann Awaiting requested review from sternenseemann

Assignees

No one assigned

Labels

6.topic: bsd Running or building packages on BSD 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related 6.topic: stdenv Standard environment 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alyssais @Ericson2314 @rrbutani