Remove extraneous use/include statements.

Use conditional loading to make docs work in the absence of LibC.

Somehow this also required a use statement to be added. Perl, go figure.

build/lib/pgBackRestBuild/Config/BuildDefine.pm

@@ -19,6 +19,7 @@ use pgBackRest::Common::String;
 use pgBackRest::Version;
 
 use BackRestDoc::Common::DocConfig;
+use BackRestDoc::Common::DocRender;
 
 use pgBackRestBuild::Build::Common;
 use pgBackRestBuild::Config::Data;

build/lib/pgBackRestBuild/Config/Data.pm

@@ -66,7 +66,6 @@ use Getopt::Long qw(GetOptions);
 use Storable qw(dclone);
 
 use pgBackRest::Common::Exception;
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Io::Base;
 use pgBackRest::Common::Log;
 use pgBackRest::Common::Wait;

doc/lib/BackRestDoc/Common/DocExecute.pm

@@ -14,7 +14,6 @@ use Exporter qw(import);
 use Storable qw(dclone);
 
 use pgBackRest::Common::Exception;
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Log;
 use pgBackRest::Common::String;
 use pgBackRest::Version;
@@ -525,6 +524,10 @@ sub backrestConfig
 
     if ($self->{bExe} && $self->isRequired($oSection))
     {
+        # Load module dynamically
+        require pgBackRest::Common::Ini;
+        pgBackRest::Common::Ini->import();
+
         my ($bCacheHit, $strCacheType, $hCacheKey, $hCacheValue) = $self->cachePop('cfg-' . PROJECT_EXE, $hCacheKey);
 
         if ($bCacheHit)

doc/lib/BackRestDoc/Latex/DocLatexSection.pm

@@ -11,7 +11,6 @@ use Carp qw(confess);
 use Exporter qw(import);
     our @EXPORT = qw();
 
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Log;
 use pgBackRest::Common::String;
 

doc/xml/release.xml

@@ -59,6 +59,10 @@
                         <p>Add <code>ServiceError</code> for errors from a service that can be retried.</p>
                     </release-item>
 
+                    <release-item>
+                        <p>Remove extraneous <code>use</code>/<code>include</code> statements.</p>
+                    </release-item>
+
                     <release-item>
                         <p>Remove embedded semicolon from <code>String</code> constant macros.</p>
                     </release-item>

src/common/ini.c

@@ -9,7 +9,6 @@ Ini Handler
 #include "common/memContext.h"
 #include "common/ini.h"
 #include "common/type/keyValue.h"
-#include "storage/helper.h"
 
 /***********************************************************************************************************************************
 Contains information about the ini

test/lib/pgBackRestTest/Common/ContainerTest.pm

@@ -18,7 +18,6 @@ use Exporter qw(import);
 use File::Basename qw(dirname);
 use Getopt::Long qw(GetOptions);
 
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Log;
 use pgBackRest::Common::String;
 use pgBackRest::Version;

test/lib/pgBackRestTest/Common/LogTest.pm

@@ -15,7 +15,6 @@ use Exporter qw(import);
     our @EXPORT = qw();
 use File::Basename qw(dirname);
 
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Log;
 use pgBackRest::Version;
 

test/test.pl

@@ -28,11 +28,9 @@ use lib dirname(dirname($0)) . '/build/lib';
 use lib dirname(dirname($0)) . '/doc/lib';
 
 use pgBackRest::Common::Exception;
-use pgBackRest::Common::Ini;
 use pgBackRest::Common::Log;
 use pgBackRest::Common::String;
 use pgBackRest::Common::Wait;
-use pgBackRest::DbVersion;
 use pgBackRest::Storage::Posix::Driver;
 use pgBackRest::Storage::Local;
 use pgBackRest::Version;