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
devcontainer: init at 0.58.0 #301313
devcontainer: init at 0.58.0 #301313
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/3742 |
@WolfangAukang I saw that you reviewed PR #276208 , could it be possible if you have time to review this one? |
7c43a8e
to
5a71c98
Compare
I had to move from mkYarnPackage -> stdenv.mkDerivation, it simplifies the derivation and doesn't make ofborg checks fail. |
9fa7732
to
c39c53e
Compare
Result of 1 package failed to build:
|
This is totally unexpected since I can build the derivation myself, will try the nixpkgs review. Thanks a lot for the review! |
Result of 1 package built:
|
@WolfangAukang It seems that it is working fine for me... should I add nodejs to the nativeBuildInputs list or not? |
After seeing some examples on Nixpkgs, it doesn't look like it is necessary |
@WolfangAukang could you approve the PR if it is ok for you? |
Result of 1 package built:
|
@Rucadi are you able to use ❯ git clone https://github.com/microsoft/vscode-remote-try-rust
❯ /home/bbigras/.cache/nixpkgs-review/pr-301313/results/devcontainer/bin/devcontainer up --workspace-folder ./vscode-remote-try-rust
[1 ms] @devcontainers/cli 0.58.0. Node.js v20.11.1. linux 6.8.4 x64.
Error: ENOENT: no such file or directory, open '/nix/store/vbpyr6xdfrzsirsj2745d0z21yyva2kl-devcontainer-0.58.0/libexec/scripts/updateUID.Dockerfile'
{"outcome":"error","message":"ENOENT: no such file or directory, open '/nix/store/vbpyr6xdfrzsirsj2745d0z21yyva2kl-devcontainer-0.58.0/libexec/scripts/updateUID.Dockerfile'","description":"An error occurred setting up the container."} |
Let me take a look, We are using this derivation at work successfully. |
@bbigras can you try this version. If this works, maybe you can help me fix the checks issues.
|
|
That's the issue I don't really know how to solve with this version because this seems to come from yarn2nix :\ Another thing we could try is to put the package as a nodePackage here? pkgs/development/node-packages I have not tested it. Maybe we can fix the version of the first commit to just have the scripts in the correct place? |
@bbigras Copying the scripts folder to libexec worked, got go container running, can you test it?
|
Result of 1 package built:
|
👍 |
Result of 1 package built:
|
Description of changes
Devcontainer (https://containers.dev/) is an opensource MIT implementation of the devcontainer specs.
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.