[paludis-user] Minor hooks documentation patch(es)

Benjamin R. Haskell paludis at benizi.com
Mon Jan 18 03:59:34 UTC 2010


I didn't manage to trace the logic for how hooks get called in paludis, 
but using 'dev-util/lafilefixer' as a tiny example package that still 
has a defined src_configure(){}, it appears the correct names for the 
hooks are:

ebuild_configure_{pre,fail,post} ('configure' not 'config')

Is that correct?  The first attached patch fixes the docs if so.

It also took me a long time to realize why my '.hook' hook wasn't 
working for ebuild_configure_post.  The second patch is a suggested 
reminder for dummies like me who gloss over the relevant chart at the 
top of the page.

Best,
Ben
-------------- next part --------------
diff --git a/doc/configuration/hooks.html.part b/doc/configuration/hooks.html.part
index 314061a..939fae2 100644
--- a/doc/configuration/hooks.html.part
+++ b/doc/configuration/hooks.html.part
@@ -191,9 +191,9 @@ hooks.</p>
     <li><code>ebuild_setup_pre</code></li>
     <li><code>ebuild_setup_fail</code></li>
     <li><code>ebuild_setup_post</code></li>
-    <li><code>ebuild_config_pre</code></li>
-    <li><code>ebuild_config_fail</code></li>
-    <li><code>ebuild_config_post</code></li>
+    <li><code>ebuild_configure_pre</code></li>
+    <li><code>ebuild_configure_fail</code></li>
+    <li><code>ebuild_configure_post</code></li>
     <li><code>ebuild_nofetch_pre</code></li>
     <li><code>ebuild_nofetch_fail</code></li>
     <li><code>ebuild_nofetch_post</code></li>
-------------- next part --------------
diff --git a/doc/configuration/hooks.html.part b/doc/configuration/hooks.html.part
index 314061a..5ae6252 100644
--- a/doc/configuration/hooks.html.part
+++ b/doc/configuration/hooks.html.part
@@ -158,7 +158,7 @@ hooks.</p>
 
 <h3 id="ebuild-phase-hooks">Ebuild Phase Hooks</h3>
 
-<p>The following ebuild phase hooks are available:</p>
+<p>The following ebuild phase hooks are available (<code>.bash</code> hooks only):</p>
 
 <ul>
     <li><code>ebuild_metadata_pre</code></li>
@@ -220,7 +220,7 @@ define the function in question.</p>
 
 <h3 id="ebuild-message-hooks">Ebuild Message Hooks</h3>
 
-<p>The following ebuild message hooks are available:</p>
+<p>The following ebuild message hooks are available (<code>.bash</code> hooks only):</p>
 
 <ul>
     <li><code>einfo</code></li>


More information about the paludis-user mailing list