<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2730/">https://reviewboard.asterisk.org/r/2730/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On August 1st, 2013, 7:08 a.m. UTC, <b>Tzafrir Cohen</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">(Not a revieww. Somme extra details that should have gone into the description)
This is aimed at replacing safe_asterisk.
There are various knobs that safe_asterisk touches. I did not set most of them. See the man pages systemd-exec(5) and systemd-service(5)
http://www.freedesktop.org/software/systemd/man/systemd.exec.html
http://www.freedesktop.org/software/systemd/man/systemd.service.html
Another thing that is not handled here is core files. It could be handled by an ExecStopPost script, however that scriptlet has no information on the exit status of the process. The Fedora Systemd folks have a way more convoluted scheme for handling core files (a daemon listening on a pipe) which required extra setup in the Debian and Fedora system I tested it on.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'm viewing this one as a "good to get in, but probably needs some more TLC before it becomes a true replacement for safe_asterisk"
The danger of that, of course, is that inevitably bugs get filed, and people have to fix them. :-)
On the other hand, if there's very little risk to having this go into 12, than at least we're trying to keep up with the times, aka, Fedora. (I blame Russell for no good reason)</pre>
<br />
<p>- Matt</p>
<br />
<p>On August 1st, 2013, 1:25 a.m. UTC, Matt Jordan wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Matt Jordan.</div>
<p style="color: grey;"><i>Updated Aug. 1, 2013, 1:25 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-21991">ASTERISK-21991</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Patch by Tzafrir.
{quote}
systemd is a new init system used by various Linux distributions. It aims at providing an environment that is compatible among different distributions, and hence its service configuration files ("units") are eto be provided from Upstream.
The attached patch includes a simple version of a systemd unit for Asterisk, as well as installing it. It is not enabled, though.
{quote}</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/Makefile <span style="color: grey">(395920)</span></li>
<li>/trunk/contrib/asterisk.service <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2730/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>