8000 cosmopolitan: 2.2 -> 4.0.2 by emaryn · Pull Request #400333 · NixOS/nixpkgs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

cosmopolitan: 2.2 -> 4.0.2 #400333

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

cosmopolitan: 2.2 -> 4.0.2 #400333

wants to merge 2 commits into from

Conversation

emaryn
Copy link
Contributor
@emaryn emaryn commented Apr 20, 2025

Diff: jart/cosmopolitan@2.2...4.0.2

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 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
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

8000
@emaryn emaryn marked this pull request as ready for review April 20, 2025 13:52
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Apr 20, 2025
@nix-owners nix-owners bot requested review from tomberek and bbjubjub2494 April 20, 2025 14:00
@@ -27,6 +28,18 @@ stdenv.mkDerivation (finalAttrs: {
})
];

cosmopolitan-cosmocc = fetchzip {
url = "https://github.com/jart/cosmopolitan/releases/download/${finalAttrs.version}/cosmocc-${finalAttrs.version}.zip";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that this binary also runs on arm64... (if testing with qemu-user)? Then we can have aarch64 in our platforms

@emaryn emaryn force-pushed the cosmopolitan branch 3 times, most recently from 1b77ea4 to de6b9ab Compare April 20, 2025 16:48
postPatch = ''
sed -i '/^DOWNLOAD := $(shell build\/download-cosmocc.sh /d' Makefile
mkdir .cosmocc
ln -s ${cosmocc} .cosmocc/3.9.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this is using the downloaded binary cosmocc to bootstrap, right? Nothing against that just clarifying

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@emaryn
Copy link
Contributor Author
emaryn commented Apr 28, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 400333


x86_64-linux

✅ 4 packages built:
  • cosmocc
  • cosmopolitan
  • cosmopolitan.dist
  • python-cosmopolitan

@tomberek
Copy link
Contributor
tomberek commented May 2, 2025

My build of this ran into:

dbg
cosmopolitan> test/tool/net/lunix_test.lua:123: assertion failed!
cosmopolitan> redbean: test/tool/net/lunix_test.lua:161: UnixTest failed (o/tmp/o/tmp/lunix_test.54380)
cosmopolitan> stack traceback:
cosmopolitan>   [C]: in function 'error'
cosmopolitan>   test/tool/net/lunix_test.lua:161: in function 'main'
cosmopolitan>   test/tool/net/lunix_test.lua:165: in main chunk
cosmopolitan>
cosmopolitan> `make MODE= -j12 o//test/tool/net/lunix_test.lua.runs` exited with 2:
cosmopolitan> o//tool/net/redbean -i test/tool/net/lunix_test.lua
cosmopolitan> consumed 256,502µs wall time
cosmopolitan> ballooned to 1,796kb in size
cosmopolitan> needed 4,888us cpu (42% kernel)
cosmopolitan> caused 449 page faults (96% memcpy)
cosmopolitan> 11 context switches (63% consensual)
cosmopolitan> performed 27 read and 0 write i/o operations
cosmopolitan>
cosmopolitan> make: *** [build/rules.mk:92: o//test/tool/net/lunix_test.lua.runs] Error 2

@emaryn
Copy link
Contributor Author
emaryn commented May 3, 2025

My build of this ran into:

dbg
cosmopolitan> test/tool/net/lunix_test.lua:123: assertion failed!
cosmopolitan> redbean: test/tool/net/lunix_test.lua:161: UnixTest failed (o/tmp/o/tmp/lunix_test.54380)
cosmopolitan> stack traceback:
cosmopolitan>   [C]: in function 'error'
cosmopolitan>   test/tool/net/lunix_test.lua:161: in function 'main'
cosmopolitan>   test/tool/net/lunix_test.lua:165: in main chunk
cosmopolitan>
cosmopolitan> `make MODE= -j12 o//test/tool/net/lunix_test.lua.runs` exited with 2:
cosmopolitan> o//tool/net/redbean -i test/tool/net/lunix_test.lua
cosmopolitan> consumed 256,502µs wall time
cosmopolitan> ballooned to 1,796kb in size
cosmopolitan> needed 4,888us cpu (42% kernel)
cosmopolitan> caused 449 page faults (96% memcpy)
cosmopolitan> 11 context switches (63% consensual)
cosmopolitan> performed 27 read and 0 write i/o operations
cosmopolitan>
cosmopolitan> make: *** [build/rules.mk:92: o//test/tool/net/lunix_test.lua.runs] Error 2

I don't know why, but I can't reproduce it.

@bbjubjub2494
Copy link
Member

My build of this ran into:

dbg
cosmopolitan> test/tool/net/lunix_test.lua:123: assertion failed!
cosmopolitan> redbean: test/tool/net/lunix_test.lua:161: UnixTest failed (o/tmp/o/tmp/lunix_test.54380)
cosmopolitan> stack traceback:
cosmopolitan>   [C]: in function 'error'
cosmopolitan>   test/tool/net/lunix_test.lua:161: in function 'main'
cosmopolitan>   test/tool/net/lunix_test.lua:165: in main chunk
cosmopolitan>
cosmopolitan> `make MODE= -j12 o//test/tool/net/lunix_test.lua.runs` exited with 2:
cosmopolitan> o//tool/net/redbean -i test/tool/net/lunix_test.lua
cosmopolitan> consumed 256,502µs wall time
cosmopolitan> ballooned to 1,796kb in size
cosmopolitan> needed 4,888us cpu (42% kernel)
cosmopolitan> caused 449 page faults (96% memcpy)
cosmopolitan> 11 context switches (63% consensual)
cosmopolitan> performed 27 read and 0 write i/o operations
cosmopolitan>
cosmopolitan> make: *** [build/rules.mk:92: o//test/tool/net/lunix_test.lua.runs] Error 2

I don't know why, but I can't reproduce it.

Also works on my end.

https://github.com/jart/cosmopolitan/blob/4ca513cba2cc8f00b0ab96805496a187a9f68c5c/test/tool/net/lunix_test.lua#L123

Based on the code, I would hypothesize that this has something to do with filesystem block size. For me, /tmp is a tmpfs. @emaryn @tomberek can you check what filesystem would have been on your /tmp?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0