Bug#816363: RFS: fgetty/0.7-0.1 [NMU] -- very small, efficient, console-only getty
Uploaded new version on mentors.
> >I knew what I did, but feel free to disagree with my reasoning.
> >
> >fgetty-login.diff essentially just renames /bin/login1 to /bin/fgetty-login.
> <I find it confusing, when binaries are renamed. Either way, /bin/login{1,2}
> >are never invoked directly, so I considered installing them into /lib/fgetty.
> >It would eliminate need in manpage, which is barely useful. WDYT?
Installed `login1' and `login2' into /lib/fgetty/. Looks nice to
me. Dropped manpage.
> > Yes, just tried, `getty' from util-linux
> >issues warning, not error in case of ro /dev, but can anyone explain me
> >why it is good thing?
> it might not be the same on Debian derivatives, on chroots, on
> minimal installations, on raspbian and so on. in my opinion this
> one is still needed (and should be upstreamed).
>
> I never had to mount /dev in ro, but I tried in a chroot and it seems possible.
>
> Some particular installations might have this, for security or whatever else.
>
> How do you feel about reverting?
I am not fond of maintaining code for some strage setup, not
supported by upstream. I tried to search web, and I found no information
about read-only /dev. Read-only /, /etc/, /usr -- sure, but not /dev.
Adding Pape in CC, he should be able clarify issue. I will revert, if
someone bug an report about his read-only /dev.
> >3 of 4 binaries are statically linked with no debug information. So
> >the only debug symbols for checkpassword(1). Since `$(wc -l
> >checkpassword.c) = 93` I see little use of debug package.
> ok, but shouldn't the new system provide "empty" debug packages then, or
> maybe even better, understand and don't provide them?
Probably. But it is question to debhelper team. I can't remove "empty"
debug ELF's manually, since their name are some hashes.
> >Sure. Will do better next time.
I documented my position on ro-dev.diff and debug packages in changelog.
--
Accept: text/plain, text/x-diff
Accept-Language: eo,en,ru
X-Keep-In-CC: yes
X-Web-Site: sinsekvu.github.io
Reply to: