From f61ef5ef2a19fbf24eac4874fa62402e538f7bea Mon Sep 17 00:00:00 2001
From: bernhard <Bernhard.Schmalhofer@gmx.de>
Date: Thu, 21 Oct 2021 12:30:46 +0200
Subject: [PATCH] Issue #1320: explicitly requires Mojolicious for S3 support

Before it only was implicitly.
---
 bin/otobo.CheckModules.pl | 14 +++++++++++++-
 cpanfile                  |  8 +++++++-
 cpanfile.docker           |  5 ++++-
 3 files changed, 24 insertions(+), 3 deletions(-)

diff --git a/bin/otobo.CheckModules.pl b/bin/otobo.CheckModules.pl
index 2f7b56530b..99dcaed2a4 100755
--- a/bin/otobo.CheckModules.pl
+++ b/bin/otobo.CheckModules.pl
@@ -204,7 +204,7 @@ =head1 DESCRIPTION
     'gazelle'       => 'Required packages if you want to use Gazelle webserver',
     'mail'          => 'Features enabling communication with a mail-server',
     'performance'   => 'Optional features which can increase performance',
-    'storage:s3'    => 'Amazon Web Services, currently only S3',
+    'storage:s3'    => 'AWS S3 compatible storage',
     'zzznone'       => 'Uncategorized',
 );
 
@@ -578,6 +578,18 @@ =head1 DESCRIPTION
     },
 
     # Feature storage:s3
+    {
+        Module    => 'Mojolicious',
+        Features  => ['storage:s3'],
+        Comment   => 'support for the REST requests to the S3 storage',
+        InstTypes => {
+            aptget => undef,
+            emerge => undef,
+            yum    => undef,
+            zypper => undef,
+            ports  => undef,
+        },
+    },
     {
         Module    => 'Mojolicious::Plugin::AWS',
         Features  => ['storage:s3'],
diff --git a/cpanfile b/cpanfile
index faea3a7526..b3b8f8b2b2 100644
--- a/cpanfile
+++ b/cpanfile
@@ -239,6 +239,9 @@ feature 'mail:ssl', 'Suppport for mail:ssl' => sub {
 };
 
 feature 'optional', 'Suppport for optional' => sub {
+    # support for the REST requests to the S3 storage
+    requires 'Mojolicious';
+
     # support for S3 using Mojo::UserAgent
     requires 'Mojolicious::Plugin::AWS';
 
@@ -352,7 +355,10 @@ feature 'performance:redis', 'Suppport for performance:redis' => sub {
 
 };
 
-feature 'storage:s3', 'Amazon Web Services, currently only S3' => sub {
+feature 'storage:s3', 'AWS S3 compatible storage' => sub {
+    # support for the REST requests to the S3 storage
+    requires 'Mojolicious';
+
     # support for S3 using Mojo::UserAgent
     requires 'Mojolicious::Plugin::AWS';
 
diff --git a/cpanfile.docker b/cpanfile.docker
index f8fd3394c1..1285881f28 100644
--- a/cpanfile.docker
+++ b/cpanfile.docker
@@ -238,7 +238,10 @@ requires 'Const::Fast';
 
 # };
 
-# feature 'storage:s3', 'Amazon Web Services, currently only S3' => sub {
+# feature 'storage:s3', 'AWS S3 compatible storage' => sub {
+    # support for the REST requests to the S3 storage
+    requires 'Mojolicious';
+
     # support for S3 using Mojo::UserAgent
     requires 'Mojolicious::Plugin::AWS';