<p dir="ltr"><br>
On Sep 1, 2015 7:17 PM, &quot;Mihai Moldovan&quot; &lt;<a href="mailto:ionic@macports.org">ionic@macports.org</a>&gt; wrote:<br>
&gt;<br>
&gt; On 01.09.2015 01:32 AM, Michael Beasley wrote:<br>
&gt; &gt; Attached are patches for sysutils/mkpwd v1.6. (current version 0.8)<br>
&gt; &gt; [...]<br>
&gt;<br>
&gt; A couple of comments on that one:<br>
&gt;<br>
&gt; Due to the project now using autotools, a lot of stuff can be simplified. For<br>
&gt; instance, explicitly defining the universal variant is not required,<br>
&gt; build.target needs not be set explicitly (will be &quot;all&quot; by default which fits<br>
&gt; nicely), build.args can be dropped (will be handled through configure) and<br>
&gt; destroot.args is obsolete, too.<br>
&gt;<br>
&gt; Due to real_version not being needed anymore, distname, livecheck.version can be<br>
&gt; omitted due to already using the default values.<br>
&gt;<br>
&gt; Due to .tar.gz now being used instead of .tgz, which (.tar.gz) is the default,<br>
&gt; this can also be omitted.<br>
&gt;<br>
&gt; Regarding the patches:<br>
&gt;<br>
&gt; Please avoid whitespace-only changes to the original code base. This is a<br>
&gt; non-functional change. It probably isn&#39;t a bad idea to upstream some stuff<br>
&gt; (albeit probably not changes to Makefile.in, which is generated by autotools),<br>
&gt; but definitely changes to the copyright header.<br>
&gt;<br>
&gt; Also, the version number fix should naturally be upstreamed.<br>
&gt;<br>
&gt; The same goes for the man page file fix.<br>
&gt;<br>
&gt;<br>
&gt; I&#39;ve committed the changes and general simplification as<br>
&gt; <a href="https://trac.macports.org/changeset/139915">https://trac.macports.org/changeset/139915</a><br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; Mihai<br>
&gt;</p>
<p dir="ltr">Noted, thanks for the feedback.</p>
<p dir="ltr">-Mike<br>
</p>