From: Robert David <robert.david.public@gmail.com>
To: "Michał Górny" <mgorny@gentoo.org>
Cc: gentoo-dev@lists.gentoo.org, rich0@gentoo.org
Subject: Re: [gentoo-dev] Reusing systemd unit file format / forking systemd (was: Going against co-maintainer's wishes (ref. bug 412697))
Date: Sun, 26 May 2013 13:31:11 +0200 [thread overview]
Message-ID: <20130526133111.5b8ae745@gmail.com> (raw)
In-Reply-To: <20130526123125.2d0f7836@gentoo.org>
On Sun, 26 May 2013 12:31:25 +0200
Michał Górny <mgorny@gentoo.org> wrote:
> On Sun, 26 May 2013 12:12:49 +0200
> Robert David <robert.david.public@gmail.com> wrote:
>
> > On Sun, 26 May 2013 05:49:48 -0400
> > Rich Freeman <rich0@gentoo.org> wrote:
> >
> > > On Sun, May 26, 2013 at 4:32 AM, Ben de Groot <yngwin@gentoo.org>
> > > wrote:
> > > > On 26 May 2013 15:37, Michał Górny <mgorny@gentoo.org> wrote:
> > > >>
> > > >> Considering the design of OpenRC itself, it wouldn't be *that
> > > >> hard*. Actually, a method similar to one used in oldnet would
> > > >> simply work. That is, symlinking init.d files to a common
> > > >> 'systemd-wrapper' executable which would parse the unit files.
> > > >
> > > > I think this idea actually makes sense. Re-using upstream work
> > > > seems a logical idea, and could ease maintenance. Of course the
> > > > issue is whether the OpenRC devs see any benefit in this.
> > >
> > > Init.d scripts are just shell scripts. All somebody needs to do
> > > is write a shell script that parses a unit file and does what it
> > > says, and exports an openrc-oriented init.d environment. That
> > > can be packaged separately, or whatever, and maybe an eclass
> > > could make it easy to install (point it at the upstream/filesdir
> > > unit and tell it what to call the init.d script, and you get the
> > > appropriate symlink/script).
> > >
> > > The OpenRC devs don't have to endorse anything - sure it would
> > > make sense to bundle it, but it could just as easily be pulled in
> > > as a dep or used manually by a user.
> > >
> > > The script could ignore any unit features that aren't implemented.
> > > You can ignore settings like auto-restart/inetd and just use the
> > > settings that get the daemon started.
> >
> > +1
> >
> > I would rather add shell script to parse unit and generate
> > appropriate init script while building than have initscript wrapper
> > that will call and parse on execution. As you said, some eclass.
>
> This effectively duplicates data for no real benefit.
>
> 1) we waste disk space.
Come on, it is 2013, wasting few inodes. I did not got these problems
in the old good times with my 386 with 4mb ram and few MB hdd.
Those with embedded system will mask many other files, not only
systemd units (so they save one inode more with my approach, when need
no initscript-wrapper).
Users of regular server/desktops/laptops, 10-20 inodes more? They would
rather won't use Gentoo with its portage tree or do not compile
kernel sources, etc.
>
> 2) if user modifies init.d script, systemd unit is out-of-sync.
> And the init.d is rewritten (potentially with CONFIG_PROTECT) on next
> upgrade.
If someone update iniscript, must be prepared to be outofsync with
package version. Thus CONFIG_PROTECT.
>
> 3) if user modifies systemd unit, init.d script is out-of-sync.
>
Why someone will modify systemd unit when will be using init.d
scripts. And for those few people doing this, the same script as portage
use for converting can be used.
Robert.
next prev parent reply other threads:[~2013-05-26 11:31 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-25 16:14 [gentoo-dev] Going against co-maintainer's wishes (ref. bug 412697) Ben de Groot
2013-05-25 16:48 ` Michał Górny
2013-05-25 17:38 ` Rich Freeman
2013-05-25 20:02 ` Chí-Thanh Christopher Nguyễn
2013-05-25 20:40 ` Rich Freeman
2013-05-25 20:45 ` Michał Górny
2013-05-25 21:38 ` Luca Barbato
2013-05-26 7:23 ` Ben de Groot
2013-05-26 7:43 ` Michał Górny
2013-05-26 10:04 ` Rich Freeman
2013-05-26 15:21 ` Ben de Groot
2013-05-26 16:15 ` Rick "Zero_Chaos" Farina
2013-05-26 17:14 ` Matt Turner
2013-05-26 17:19 ` Andreas K. Huettel
2013-05-26 7:54 ` Pacho Ramos
2013-05-25 17:00 ` Pacho Ramos
2013-05-25 17:14 ` Carlos Silva
2013-05-26 7:15 ` Ben de Groot
2013-05-26 7:44 ` Pacho Ramos
2013-05-26 7:45 ` Michał Górny
2013-05-26 9:59 ` Luca Barbato
2013-05-25 18:13 ` Markos Chandras
2013-05-25 19:53 ` Anthony G. Basile
2013-05-25 19:58 ` Mike Gilbert
2013-05-25 21:55 ` Anthony G. Basile
2013-05-25 19:59 ` Rich Freeman
2013-05-26 7:00 ` Michał Górny
2013-05-26 7:22 ` Tiziano Müller
2013-05-26 7:46 ` Pacho Ramos
2013-05-26 7:49 ` Michał Górny
2013-05-26 7:00 ` Ben de Groot
2013-05-26 7:37 ` [gentoo-dev] Reusing systemd unit file format / forking systemd (was: Going against co-maintainer's wishes (ref. bug 412697)) Michał Górny
2013-05-26 8:32 ` Ben de Groot
2013-05-26 9:49 ` Rich Freeman
2013-05-26 10:12 ` Robert David
2013-05-26 10:31 ` Michał Górny
2013-05-26 11:12 ` Rich Freeman
2013-05-26 11:31 ` Robert David [this message]
2013-05-26 11:47 ` [gentoo-dev] Reusing systemd unit file format / forking systemd Luca Barbato
2013-05-26 10:23 ` Luca Barbato
2013-05-26 11:15 ` Michał Górny
2013-05-26 11:59 ` Luca Barbato
2013-05-26 13:35 ` Sergei Trofimovich
2013-05-26 14:22 ` Luca Barbato
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130526133111.5b8ae745@gmail.com \
--to=robert.david.public@gmail.com \
--cc=gentoo-dev@lists.gentoo.org \
--cc=mgorny@gentoo.org \
--cc=rich0@gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox