-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
devpod: init at 0.4.1 #251858
devpod: init at 0.4.1 #251858
Conversation
Does the package work correctly? Last time I tried to package it I stumbled upon this issue loft-sh/devpod#329. |
@dit7ya The CLI is fully functional. The GUI also appears fully functional with minor graphical bugs (e.g. some icons missing on Linux, impossible to drag window on macOS) |
Actually I had forgotten to enable the sandbox on macOS on the first build, I hit this issue now: h4llow3En/mac-notification-sys#28 Not sure if there is an easy workaround, I have marked the GUI as supported on Linux only for now. (Personally, I do not plan to use the GUI, so I'd be happy with packaging the CLI only.) |
f91b0ca
to
221baa7
Compare
0aad320
to
da2e4c4
Compare
@maxbrunet did you use it with docker provider? I tried but I think it has a design flow preventing it to work on nix. I mentioned it in the issue opened by @dit7ya |
Ok. I checked your version and it is working because the binary is linked statically to the libc. So copying the file in the container works. |
Yes, it has been working well (note: I use
Cgo is disabled, so no libc involved |
|
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/2783 |
@SuperSandro2000 Can you have a lootk to this please? |
The cli seems to work 👍 . The rest is a bit complex for me to review. |
What is up with the ofborg-eval test not passing? |
@maxbrunet can you have a look here? 23.11 is coming and it would be nice to have this in the stable release if it is not too late yet. |
da2e4c4
to
4f25030
Compare
4f25030
to
4f2a792
Compare
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 @pbsds for the review!
Result of 2 packages built:
|
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.
- package path fits guidelines
- package name fits guidelines
- package version fits guidelines
- package build on x86_64-linux
- executables tested on x86_64-linux
-
meta.description
is set and fits guidelines -
meta.license
fits upstream license -
meta.platforms
is set -
meta.maintainers
is set - build time only dependencies are declared in
nativeBuildInputs
- source is fetched using the appropriate function
- the list of
phases
is not overridden - when a phase (like
installPhase
) is overridden it starts withrunHook preInstall
and ends withrunHook postInstall
. - patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
- patches that are remotely available are fetched rather than vendored
Anyone has an example of using devpod with a nix flake? I guess a simple |
Description of changes
This adds 2 new packages:
devpod
(CLI written in Go) anddevpod-desktop
(GUI written in Rust with Tauri and a TypeScript front-end). The GUI invokes/wraps the CLI.Closes #239252
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)