<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-guide.

</pre>
<p><a href="https://github.com/macports/macports-guide/commit/741def19b486ba40be997a91fa1b2a9dc052443a">https://github.com/macports/macports-guide/commit/741def19b486ba40be997a91fa1b2a9dc052443a</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 741def1  guide: update revision advice
</span>741def1 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 741def19b486ba40be997a91fa1b2a9dc052443a
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Thu Feb 8 16:01:55 2024 +1100

<span style='display:block; white-space:pre;color:#404040;'>    guide: update revision advice
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    As of MacPorts 2.9, upgrading a port (or its dependents, since upgrade
</span><span style='display:block; white-space:pre;color:#404040;'>    is recursive) will update some metadata including dependencies, even
</span><span style='display:block; white-space:pre;color:#404040;'>    when the port is not outdated. So revision increases are not needed to
</span><span style='display:block; white-space:pre;color:#404040;'>    make this happen.
</span>---
 guide/xml/portfile-keywords.xml | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/guide/xml/portfile-keywords.xml b/guide/xml/portfile-keywords.xml
</span><span style='display:block; white-space:pre;color:#808080;'>index c1b0bf4..596957f 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/guide/xml/portfile-keywords.xml
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/guide/xml/portfile-keywords.xml
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -108,8 +108,8 @@
</span>         result in a change for users who already have the ports
         installed.</para>
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        <para>Some examples of situations in which a port's revision should be
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        increased:</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        <para>Some examples of situations in which a port's revision
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        should usually be increased:</para>
</span> 
         <itemizedlist>
           <listitem>
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -124,7 +124,7 @@
</span>             <para>changing the names or locations of any installed files</para>
           </listitem>
           <listitem>
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            <para>adding a dependency to depends_lib or depends_run</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            <para>adding a dependency that causes the installed files to be different</para>
</span>           </listitem>
           <listitem>
             <para>a library dependency's install_name has changed</para>
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -138,7 +138,7 @@
</span>         </itemizedlist>
 
         <para>Some examples of situations in which a port's revision should not
<span style='display:block; white-space:pre;background:#ffe0e0;'>-        be increased:</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        usually be increased:</para>
</span> 
         <itemizedlist>
           <listitem>
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -146,7 +146,18 @@
</span>           </listitem>
           <listitem>
             <para>adding a dependency to depends_fetch, depends_extract,
<span style='display:block; white-space:pre;background:#ffe0e0;'>-            depends_build, or depends_test</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            depends_patch, depends_build, or depends_test</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          </listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          <listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            <para>adding a direct dependency on a port that was
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            already an indirect dependency</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          </listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          <listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            <para>changing a dependency's type, e.g. from depends_lib
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            to depends_build</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          </listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          <listitem>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            <para>removing a dependency that is not used</para>
</span>           </listitem>
           <listitem>
             <para>setting or changing the port's license</para>
</pre><pre style='margin:0'>

</pre>