New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Boost fiber fix #47537 #47540
Boost fiber fix #47537 #47540
Conversation
include/boost/fiber/detail/context_spmc_queue.hpp:1:1: error: stray ‘\357’ in program
Thank you for spotting this bug. However, a first analysis (see #47537 (comment)) indicates that the header file is broken during the build process, which may be a result of a more general bug. So instead of fixing the symptom in I advise against merging this unless we really understand the cause of the problem. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find and try to fix underlying issue first.
(not really a requested change but a marker not to merge this for now)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, after looking at the issue I can confirm this is the appropriate fix, and there seems to be no more general underlying issue.
@@ -171,7 +171,7 @@ stdenv.mkDerivation { | |||
postFixup = '' | |||
# Make boost header paths relative so that they are not runtime dependencies | |||
cd "$dev" && find include \( -name '*.hpp' -or -name '*.h' -or -name '*.ipp' \) \ | |||
-exec sed '1i#line 1 "{}"' -i '{}' \; | |||
-exec sed 's/^\xef\xbb\xbf//;1i#line 1 "{}"' -i '{}' \; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick just to be safe: can you please change this into 1s/^\xef\xbb\xbf//;
to make sure it only applies to line 1, as the BOM can only be in line 1.
@GrahamcOfBorg build boost |
Success on aarch64-linux (full log) Attempted: boost Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: boost Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: boost Partial log (click to expand)
|
include/boost/fiber/detail/context_spmc_queue.hpp:1:1: error: stray ‘\357’ in program caused by a leftover UTF-8 Byte Order Mark
Thank you! |
include/boost/fiber/detail/context_spmc_queue.hpp:1:1: error: stray ‘\357’ in program caused by a leftover UTF-8 Byte Order Mark
include/boost/fiber/detail/context_spmc_queue.hpp:1:1: error: stray ‘\357’ in program
Motivation for this change
Fix #47537
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)