-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Arcan updates #310831
Arcan updates #310831
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3934 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3977 |
Result of 10 packages built:
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1704 |
I am curious why this is still using the upstream Tracy and not the forked one currently referenced in the arcan clone.sh. Is the clone.sh out of date? |
TBH i did not notice it is now using an in-house copy. |
476059b
to
d3754eb
Compare
tracy worked for me by disabling libunwind using TRACY_LIBUNWIND_BACKTRACE. |
It looks like a workaround rather than a real fixup. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1719 |
Yes, but it allows keeping Tracy support mostly functional, with only the libunwind backtraces not working. It should also be solvable by correctly adding libunwind to the build/runtime dependencies, but I don't know enough about Nix to debug why the obvious approach was not working. |
Is there any serious lack of functionality by not building it with Tracy support? |
You cannot use Tracy to debug problems and have to recompile it with print-logging to get equivalent information. |
532673f
to
ab88fbd
Compare
For now I will revert (?) to the older GitHub. |
Result of 5 packages failed to build:
5 packages built:
|
cat9 and durden: I forgot them! |
It also seems to include some WIP commit, please ping me when you consider the PR ready :) |
a50ea34
to
11b0efe
Compare
@ofborg build arcan-all-wrapped |
Now it's fine! @pbsds |
where is |
Result of 10 packages built:
|
It can be optionally used as an alternative to |
Perhaps make the |
It is easier to control them this way, in order to not pollute the master package file. Also, let's put them on passthru.
|
- hammer-reorder - replace-fail - strictDeps - split outputs
Also update the external/git sources too.
The fork is somewhat broken.
Result of 13 packages built:
|
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.