Skip to content
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

feat: add netcat for remote development support #77

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

Conversation

twelho
Copy link

@twelho twelho commented Feb 3, 2025

Zed requires the BSD version of netcat for remote development. It then depends on libbsd, which subsequently depends on libmd.

Starting a remote development session on another system over SSH works now with the password prompt correctly appearing.

Fixes #58.

@flathubbot
Copy link
Contributor

Started test build 178178

@flathubbot
Copy link
Contributor

Build 178178 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/161325/dev.zed.Zed.flatpakref

- install -Dm0755 nc /app/bin/nc
sources:
- type: archive
url: https://salsa.debian.org/debian/netcat-openbsd/-/archive/debian/1.228-1/netcat-openbsd-debian-1.228-1.tar.gz
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the upstream tarball is not used here?

Copy link
Author

Choose a reason for hiding this comment

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

The upstream tarball is for BSD only and doesn't work on Linux without patches, this Debian package bundles those patches, is tracked by Anitya, and is also used by both Arch Linux and Alpine Linux.

url-template: https://libbsd.freedesktop.org/releases/libbsd-$version.tar.xz
cleanup:
- /lib/pkgconfig
- /share/man
Copy link
Contributor

Choose a reason for hiding this comment

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

None of these packages are part of the freedesktop SDK?

Copy link
Author

Choose a reason for hiding this comment

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

AFAIK neither libbsd nor libmd is in the SDK, they are also compiled in other Flatpaks that need them such as com.valvesoftware.Steam.

@flathubbot
Copy link
Contributor

Started test build 178301

@flathubbot
Copy link
Contributor

Build 178301 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/161449/dev.zed.Zed.flatpakref

Zed [requires](https://github.com/zed-industries/zed/blob/eb820ab800ea2a0023667b5c0eb91daa7c9ab084/crates/remote/src/ssh_session.rs#L1423-L1441) the BSD version of `netcat` for remote development. It then depends on `libbsd`, which subsequently depends on `libmd`.

Starting a remote development session on another system over SSH works now with the password prompt correctly appearing.

Fixes flathub#58.
@flathubbot
Copy link
Contributor

Started test build 179089

@flathubbot
Copy link
Contributor

Build 179089 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/162240/dev.zed.Zed.flatpakref

@jerome-benoit
Copy link
Contributor

jerome-benoit commented Feb 7, 2025

@tomaswarynyca: have you noticed that PR? I think it deserves a review and a merge ;)

@tomaswarynyca
Copy link
Collaborator

I have my doubts about whether we should include this :)
there is a forcepush after your review, do you see anything weird added?

@jerome-benoit
Copy link
Contributor

jerome-benoit commented Feb 7, 2025

I have my doubts about whether we should include this :)

Since the flatpak is offering the option to run fully sandboxed, I think the PR makes sense for that use case.

there is a forcepush after your review, do you see anything weird added?

I will recheck tomorrow and re-approve once done. But I do not think the PR author is trying to include anything weird :)

@flathubbot
Copy link
Contributor

Started test build 179319

@flathubbot
Copy link
Contributor

Build 179319 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/162476/dev.zed.Zed.flatpakref

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing nc in Flatpak Sandbox
4 participants