-
Notifications
You must be signed in to change notification settings - Fork 270
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
Debian Support #27
Comments
I don't currently have any plans to make a Debian package for Endlessh,
whether official or unofficial. There's an "install" target that follows
the standard conventions (CFLAGS, DESTDIR, PREFIX, etc.), so it is
already in good shape for packaging in general. For example, with fpm:
$ make clean
$ rm -rf destdir
$ eval $(dpkg-buildflags --export=sh)
$ export PREFIX=/usr
$ export DESTDIR=destdir
$ make -e install
$ fpm -s dir -t deb -n endlessh -v $(git describe) -C destdir .
Note that this wouldn't install the systemd unit file (which can be
found under util/) nor would it create a default, editable configuration
at /etc/endlessh/config. I'd expect both from a proper Debian package.
|
@ApoRes1337 FYI, I'm currently packaging |
Update: I uploaded |
Awesome! Thanks, @nbraud! I'm happy about this.
|
Any updates on this? I'd love to see and use it from "testing". :) |
For the record:
|
@nbraud Apologies if this isn't the best way to mention this, but I could not open an issue on the linked GitLab. I'm not very familiar with systemd's inner workings, but my gut tells me the following line
should instead be Again, this may be due to my lack of understanding of how packages work, but as it stands I don't see how to configure the package as published short of editing the unit file after installation. Please do correct me if I'm wrong. If I'm not, however, I think it would make for a better user experience if this change was made and a default config was provided, as I know myself and quite a few others who have come to rely on this behavior from other Debian packages. |
FWIW this has been in unstable for a couple of months now. It appears that it's not migrating further due to:
|
@SeinopSys The unit file in the package uses the correct binary path (I added a distro-level patch).
FYI, you can create a guest account on Salsa. |
@BenWiederhake @d7415 Yeap, the release team's policy of requiring binaries built on Debian infrastructure conflicts with the ftp team's policy of requiring binary uploads for NEW (so they can review the built packages). |
Apologies for the reply latency: the package didn't clear NEW until November, at which point I was fairly unavailable (my partner and I had medical complications...) |
Update: since @skeeto released v1.1 tonigh, I uploaded it to sid (with |
|
Hello, I wanted to ask if Debian support is planned?
Greetings Christopher
The text was updated successfully, but these errors were encountered: