Skip to content
Commit 09551876 authored by Mathieu Parent's avatar Mathieu Parent
Browse files

Create /dev/ptmx like debootstrap does

See https://salsa.debian.org/installer-team/debootstrap/blob/6f3f6f8b76e2d1a24ddbf05f065439412c3b81a1/functions#L1263-1268 and https://www.kernel.org/doc/Documentation/filesystems/devpts.txt

Otherwise, scripts/pre_remove_50_find_bad_permissions fails with:
  ERROR: BAD PERMISSIONS
  crw-rw-rw-. 1 root root 5, 2 Oct 16 03:49 /dev/ptmx
parent ed36be39
Loading
Loading
Loading
Loading
  • Author Maintainer

    @anbe, @holger I didn't want to commit directly to develop branch. I wanted to create a merge request first. Sorry.

    For context, see https://salsa.debian.org/samba-team/samba/-/jobs/55820 (from samba-team/samba!10 (merged))

  • Author Maintainer

    See also the debootstrap commit message: installer-team/debootstrap@c997b80c.

    Revert if you need more discussion.

  • Maintainer

    Hi Mathieu,

    @anbe, @holger I didn't want to commit directly to develop branch. I wanted to create a merge request first. Sorry.

    thanks for explaining.

    even more importantly: have you tested this change?

    For context, see https://salsa.debian.org/samba-team/samba/-/jobs/55820 (from samba-team/samba!10 (merged))

    this is a bit long for context. please explain.

    I see " ERROR: BAD PERMISSIONS crw-rw-rw-. 1 root root 5, 2 Oct 16 03:49 /dev/ptmx" in there but then I dont see this on https://piuparts.debian.org/sid/source/s/samba.html so for now I'm just assuming your salsa runner environment is borken.

    Your commit message also said "Create /dev/ptmx like debootstrap does" and you pointed me to the blob, but not the commit where this was introduced... :/ an then piuparts is using debootstrap too, so why does piuparts need that code as well?

    Last and least, a debian/changelog entry would have been nice and I think I will make it mandatory via the CONTRIBUTING document. Because now the work would be on me to write a meaningful changelog entry which atm I cannot do, because... see above ;)

    So, yes, I think I will revert these commits (also we use develop to deploy this directly to piuparts.d.o) for now and would appreciate it very much if you could resubmit them via the BTS (or a gitlab MR) while including a debian/changelog entry.

    And finally: thank you for your contribution (even if the above..!) :)

    signature.asc

  • Holger Levsen @holger

    mentioned in commit 491e4534

    ·

    mentioned in commit 491e4534

    Toggle commit list
  • Mathieu Parent @sathieu

    mentioned in merge request !2 (merged)

    ·

    mentioned in merge request !2 (merged)

    Toggle commit list
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment