From 444fcda3a062456a3e8500ef23e23d3584b964af Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Wed, 8 Mar 2023 10:14:43 +0200 Subject: Add few comments --- bpkg/system-package-manager-debian.cxx | 4 +++- bpkg/system-package-manager.hxx | 7 ++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/bpkg/system-package-manager-debian.cxx b/bpkg/system-package-manager-debian.cxx index 5c8ace0..1abfb84 100644 --- a/bpkg/system-package-manager-debian.cxx +++ b/bpkg/system-package-manager-debian.cxx @@ -3140,7 +3140,9 @@ namespace bpkg // // Note also that dpkg-buildpackage causes recompilation on every run by // changing the SOURCE_DATE_EPOCH environment variable (which we track for - // changes since it affects GCC). + // changes since it affects GCC). Note that since we don't have this + // SOURCE_DATE_EPOCH during dry-run caused by installed_entries(), there + // would be a recompilation even if the value weren't changing. // cstrings args { "dpkg-buildpackage", diff --git a/bpkg/system-package-manager.hxx b/bpkg/system-package-manager.hxx index 4fd8ba1..8a5b891 100644 --- a/bpkg/system-package-manager.hxx +++ b/bpkg/system-package-manager.hxx @@ -177,9 +177,10 @@ namespace bpkg // the package. // // Return the list of paths to binary packages and any other associated - // files (build metadata, etc). If the result is empty, assume the - // prepare-only mode (or similar) with appropriate result diagnostics - // having been already issued. + // files (build metadata, etc) that could be useful for consumption of + // binary packages. If the result is empty, assume the prepare-only mode + // (or similar) with appropriate result diagnostics having been already + // issued. // struct package { -- cgit v1.1