[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1057098: marked as done (RFS: runit/2.1.2-57 -- system-wide service supervision)

Your message dated Tue, 13 Feb 2024 15:06:46 +0100
with message-id <Zct3dlqfeMT1tmjG@isildor2.loewenhoehle.ip>
and subject line Re: Bug#1057098: RFS: runit/2.1.2-57 -- system-wide service supervision
has caused the Debian Bug report #1057098,
regarding RFS: runit/2.1.2-57 -- system-wide service supervision
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1057098: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1057098
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "runit":

 * Package name     : runit
   Version          : 2.1.2-57
   Upstream contact : Gerrit Pape <pape@smarden.org>
 * URL              : http://smarden.org/runit/
 * License          : BSD-3-clause, GPL-3+, CC0-1.0
 * Vcs              : https://salsa.debian.org/debian/runit
   Section          : admin

The source builds the following binary packages:

  runit - system-wide service supervision
  runit-run - service supervision (systemd and sysv integration)
  getty-run - runscripts to supervise getty processes
  runit-init - system-wide service supervision (as init system)

To access further information about this package, please visit the
following URL:

  https://mentors.debian.net/package/runit/

Alternatively, you can download the package with 'dget' using this
command:

  dget -x
  https://mentors.debian.net/debian/pool/main/r/runit/runit_2.1.2-57.dsc

  git:
  https://salsa.debian.org/debian/runit/-/tree/next?ref_type=heads

Changes since the last upload:

 runit (2.1.2-57) experimental; urgency=medium
 .
   * Stage {1,2,3,} and ctraltdel: change PATH order
   * runit.preinst: print a message on unmerged systems
        warning that such configuration is expected
        to fail to boot very soon.
   * trigger_sv: incorporate sv_wtime code (Closes: #1057095)
   * invoke-run: improve test for bin metafile
   * 40-runit: forward non standard actions to
        sysv scripts (Closes: #1057094)
   * Drop runit-systemd empty package,
      it's an empty transitional package since
      Bullseye (Closes: #1038301)

Regards,
Lorenzo

--- End Message ---
--- Begin Message --- 
Hi Lorenzo,

thanks for following up!

You've misunderstood me with NEWS file (but this is not a stopper for
the upload.)

What I meant is that the NEWS file is that old that it should be
removed. (Debhelper just pointed me to the file, even if it did
not correctly disagnose the issue)
This is even more correct if debhelper decided to strip out that
version from d/changelog anyways, users will not get this NEWS
files displayed, and the information in it shouldn't be relveant
anymore. 
So my suggestion would be to just rm it.

As said, that can be done later; I'll upload after sending this
mail...

--
tobi

On Mon, Feb 12, 2024 at 03:30:43PM +0100, Lorenzo wrote:
> Control: tags -1 -moreinfo
> 
> Hi Tobi,
> thanks for having a look at this package :)
> Uploaded again to mentors with the following changes:
>  - removed message in preinst (no longer useful, see below)
>  - cleaned up obsolete lintian tag to avoid a new lintian error
> 
> new links:
> 
> https://mentors.debian.net/package/runit/
> 
> dget -x
> https://mentors.debian.net/debian/pool/main/r/runit/runit_2.1.2-57.dsc
> 
> Git repo (updated):
>   https://salsa.debian.org/debian/runit/-/tree/next?ref_type=heads
> 
> 
> > There is a lintian warning:
> > W debian-news-entry-has-unknown-version
> > 2.1.2-14 [usr/share/doc/getty-run/NEWS.Debian.gz:1]
> 
> this was fixed in Debhelper [1], now the NEWS file is automatically
> trimmed
> 
> > the warning of unmerged systems will stop booting soon MUST
> > be in Debian.NEWS, preinst is the wrong place and it will be
> > very likely missed.  NEWS is a well-known mechanism to convey
> > important information. 
> > 
> > On the other hand, is this warning acutally required?  Once this will
> > get a real issue, Depend: on usr-is-merged and you should
> > be set. If you cannot tell when that is, enforce it now, as non-merged
> > systems are no longer supported anyway. 
> > 
> 
> The message was targeted downstream (especially Devuan) where the merged
> layout is not enforced but no plan was made to actively support unmerged
> layout. A relevant share of users in testing/unstable where running with
> the system not merged with no clue that the system was about to break.
> 
> The problem with the NEWS file is that people don't have apt-listchanges
> installed and also the NEWS can't be displayed conditionally (only to
> unmerged systems).
> 
> The message is already in runit_2.1.2-54+usrmerge (unstable/testing)
> and subsequently a long discussion started both on Devuan-users mailing
> list and the Devuan forum, so users are no longer unaware of the issue
> and I'm satisfied with that.
> (unmerged systems are already broken now so the message is no longer
> useful)
> 
> 
> > 
> > TL:DR: init-system-helpers is Essential: Yes, so you can already now
> > Depend: on usr-is-merged if you want to make sure.
> > (That will also cover where people held init-systems-helpers to an
> > older
> > 
> 
> I prefer if this is dealt at distro level (downstream), so they either
> provide support for unmerged layout or they enforce the merge.
> 
> Best Regards,
> Lorenzo
> 
> [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=%231021607
>

--- End Message ---
Reply to: