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

Configuration files to generate a Debian package #6

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

Conversation

frandieguez
Copy link

I've just created the required configuration files to generate a Debian package.

There is tons of possible improvements in the package. Those that I can highlight are:

  • There are no manpages for 'latch' and 'latch-shell' binaries, required by UNIX systems
  • Previously the compilation was creating a 'test' binary and that binary has a collission with the 'test' binary from coreutils in UNIX systems. I haver temporarly rename it to 'latch-test' but, looking at the sources I think that it could be removed.
  • I think that I could improve the configuration process by using pam-auth-update and generating the latch.conf file from a debconf wizard

This avoids a file collision with the 'test' binary from coreutils Debian package

Signed-off-by: Fran Dieguez <[email protected]>
Signed-off-by: Fran Dieguez <[email protected]>
@frandieguez
Copy link
Author

Any news about merging this?

@naviprojects
Copy link
Contributor

Hi Fran,

• About 'test' binary you are right. I've already solved it in the latest version.
• Manpages are in development. Come in soon.
• I don't know how to debconf works at all. I have to study it. Any help is appreciated, and feel free to add some changes.

Some changes are required before we can merge your pull request:

  • Permissions should be added in post-installation:
    binaries = Setuid (4755),
    conf files = read-write only root-owner (600).
  • It is necessary to make multiple packages. There are some systems not PAM compatible (like OpenBSD) and all binaries don't need to be necessary installed.
    It should be 3 packages: latch, pam-latch and latch-ssh-command.
  • Debconf seems not to work properly or I don't understand at all its usefulness.
  • Wrong copyrights and authors. It should be LGPL 2.1. and ElevenPaths, respectively.

Thanks,
kind regards.

Iván.

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.

2 participants