From 50f9844b8a97aa06edf09b6d8a538721a0cd24ea Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Thu, 26 Oct 2023 11:15:18 +0200
Subject: Use match options for runtime/buildtime distinction when installing
 libraries

Specifically, now, if a library is installed solely as a prerequisite of
an executable (potentially recursively), then only its runtime files are
installed omitting everything buildtime-related (static/import libraries,
non-versioned symlinks for shared libraries, pkg-config files, headers,
etc). If you are familiar with the runtime and -dev/-devel package splits
for libraries in Debian/Fedora, this is an analogous semantics.
---
 libbuild2/cc/install-rule.hxx | 39 ++++++++++++++++++++++++++++-----------
 1 file changed, 28 insertions(+), 11 deletions(-)

(limited to 'libbuild2/cc/install-rule.hxx')

diff --git a/libbuild2/cc/install-rule.hxx b/libbuild2/cc/install-rule.hxx
index 6998d63..9d9211b 100644
--- a/libbuild2/cc/install-rule.hxx
+++ b/libbuild2/cc/install-rule.hxx
@@ -20,7 +20,7 @@ namespace build2
   {
     class link_rule;
 
-    // Installation rule for exe{} and lib*{}. Here we do:
+    // Installation rule for exe{} and lib[as]{}. Here we do:
     //
     // 1. Signal to the link rule that this is update for install.
     //
@@ -28,17 +28,23 @@ namespace build2
     //
     // 3. Extra un/installation (e.g., libs{} symlinks).
     //
+    // 4. Handling runtime/buildtime match options for lib[as]{}.
+    //
     class LIBBUILD2_CC_SYMEXPORT install_rule: public install::file_rule,
                                                virtual common
     {
     public:
       install_rule (data&&, const link_rule&);
 
-      virtual const target*
+      virtual bool
+      filter (action, const target&, const target&) const override;
+
+      virtual pair<const target*, uint64_t>
       filter (const scope*,
-              action, const target&, prerequisite_iterator&) const override;
+              action, const target&, prerequisite_iterator&,
+              match_extra&) const override;
 
-      // Note: rule::match() override.
+      // Note: rule::match() override (with hint and match_extra).
       //
       virtual bool
       match (action, target&, const string&, match_extra&) const override;
@@ -46,7 +52,10 @@ namespace build2
       using file_rule::match; // Make Clang happy.
 
       virtual recipe
-      apply (action, target&) const override;
+      apply (action, target&, match_extra&) const override;
+
+      virtual void
+      reapply (action, target&, match_extra&) const override;
 
       virtual bool
       install_extra (const file&, const install_dir&) const override;
@@ -58,22 +67,24 @@ namespace build2
       const link_rule& link_;
     };
 
-    // Installation rule for libu*{}.
+    // Installation rule for libu[eas]{}.
     //
     // While libu*{} members themselves are not installable, we need to see
     // through them in case they depend on stuff that we need to install
     // (e.g., headers). Note that we use the alias_rule as a base.
     //
-    class LIBBUILD2_CC_SYMEXPORT libux_install_rule:
-      public install::alias_rule,
-      virtual common
+    class LIBBUILD2_CC_SYMEXPORT libux_install_rule: public install::alias_rule,
+                                                     virtual common
     {
     public:
       libux_install_rule (data&&, const link_rule&);
 
-      virtual const target*
+      // Note: utility libraries currently have no ad hoc members.
+
+      virtual pair<const target*, uint64_t>
       filter (const scope*,
-              action, const target&, prerequisite_iterator&) const override;
+              action, const target&, prerequisite_iterator&,
+              match_extra&) const override;
 
       // Note: rule::match() override.
       //
@@ -82,6 +93,12 @@ namespace build2
 
       using alias_rule::match; // Make Clang happy.
 
+      virtual recipe
+      apply (action, target&, match_extra&) const override;
+
+      virtual void
+      reapply (action, target&, match_extra&) const override;
+
     private:
       const link_rule& link_;
     };
-- 
cgit v1.1