From: Richard Leitner <richard.leitner@skidata.com>
To: "openembedded-devel@lists.openembedded.org"
<openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-networking][PATCH][jethro] samba: fix smbd and nmbd paths in sysv initscript
Date: Thu, 28 Apr 2016 15:59:45 +0200 [thread overview]
Message-ID: <57221751.8030803@skidata.com> (raw)
In-Reply-To: <1461847944.3302.20.camel@linux.intel.com>
On 04/28/2016 02:52 PM, Joshua G Lock wrote:
> On Thu, 2016-04-28 at 14:30 +0200, Richard Leitner wrote:
>> For the jethro branch.
>
> Any idea whether this change is required in the master branch?
Yes. The Samba 4.4.2 recipe is also using the script from
packaging/LSB/samba.sh, which is the same as in 4.1.12.
>
>> The sysv initscript provided by samba assumes smbd and nmbd are
>> installed in /opt/samba/bin/. In our case both binaries are installed
>> to
>> /usr/sbin/ by default. Therefore fix these paths.
>
> Rather than hard-code these in a patch, which is equally broken for
> people who don't use the default sbindir, it would make more sense to
> sed ${sbindir} into the init file in a do_install_append ().
Ok, you're right, that is definitely a better solution.
I'll prepare a patch for master and jethro.
regards,
Richard
prev parent reply other threads:[~2016-04-28 14:09 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-28 12:30 [meta-networking][PATCH][jethro] samba: fix smbd and nmbd paths in sysv initscript Richard Leitner
2016-04-28 12:52 ` Joshua G Lock
2016-04-28 13:59 ` Richard Leitner [this message]
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=57221751.8030803@skidata.com \
--to=richard.leitner@skidata.com \
--cc=openembedded-devel@lists.openembedded.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 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.