All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/openrc: install sysv-rcs file
Date: Thu, 19 Dec 2019 22:24:49 +0100	[thread overview]
Message-ID: <20191219212449.GP26395@scaer> (raw)
In-Reply-To: <20191219205036.GO26395@scaer>

Adam, All,

On 2019-12-19 21:50 +0100, Yann E. MORIN spake thusly:
> On 2019-12-19 21:44 +0100, Thomas Petazzoni spake thusly:
> > On Thu, 19 Dec 2019 11:01:35 -0800
> > aduskett at gmail.com wrote:
> > 
> > > From: Adam Duskett <aduskett@greenlots.com>
> > > 
> > > package/openrc has the file sysv-rcs which starts sysvinit services
> > > not written for openrc. However, currently it is not installed to
> > > the target.
> > > 
> > > Install this file to $(TARGET_DIR)/etc/init.d during the
> > > target_install step.
> > > 
> > > Signed-off-by: Adam Duskett <aduskett@greenlots.com>
> > Michal had what seems like a more elaborate solution:
> >   http://patchwork.ozlabs.org/patch/1098590/
> We already have 162044407b (package/skeleton-init-openrc: add support
> for starting sysv scripts) which was edited by me, but I missed the fact
> that the script was not installed.
> 
> Adam, care to investigate what is different between our script, and the
> one from the package itself?
> 
> I am all in favour of dropping our script, if upstream has a functional
> one.

In fact, upstream does not have such a script. It is just that we forgot
to install our own script.

Applied to master, changed the destination to a full path, and not just
the directory, and pushed. Thanks.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2019-12-19 21:24 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-19 19:01 [Buildroot] [PATCH 1/1] package/openrc: install sysv-rcs file aduskett at gmail.com
2019-12-19 20:44 ` Thomas Petazzoni
2019-12-19 20:50   ` Yann E. MORIN
2019-12-19 21:24     ` Yann E. MORIN [this message]
2019-12-25 18:05 ` Peter Korsgaard

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=20191219212449.GP26395@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.