]> git.netwichtig.de Git - user/henk/code/inspircd.git/commitdiff
Remove some todo comments which are outdated or misleading.
authorSadie Powell <sadie@witchery.services>
Fri, 5 Mar 2021 04:30:08 +0000 (04:30 +0000)
committerSadie Powell <sadie@witchery.services>
Fri, 5 Mar 2021 04:31:20 +0000 (04:31 +0000)
configure
make/configure.pm
src/users.cpp

index 63256f87ad54d95935bdf4410b5a5170ba78b2ce..ccf6cbd6ab0da32678d70de300d23b4496bdd9a6 100755 (executable)
--- a/configure
+++ b/configure
@@ -101,7 +101,6 @@ exit 1 unless GetOptions(
        'system'               => \$opt_system,
        'uid=s'                => \$opt_uid,
 
-       # TODO: when the modulemanager rewrite is done these should be removed.
        'disable-extras=s@' => \@opt_disableextras,
        'enable-extras=s@'  => \@opt_enableextras,
        'list-extras'       => sub { list_extras; exit 0; },
@@ -385,10 +384,13 @@ if (prompt_bool $interactive, $question, 0) {
                }
        }
 } elsif (!defined $opt_disable_auto_extras) {
-       # TODO: finish modulemanager rewrite and replace this code with:
-       # system './modulemanager', 'enable', '--auto';
        my %modules = (
-               # Missing: m_ldap, m_regex_stdlib, m_ssl_mbedtls
+               # We can't automatically enable the following modules because there is
+               # no way to autodetect their dependencies:
+               #
+               #  * ldap (no pkg-config support)
+               #  * regex_stdlib (no C++11 compiler detection)
+               #  * ssl_mbedtls (no pkg-config support)
                'm_argon2.cpp'          => 'pkg-config --exists libargon2',
                'm_geo_maxmind.cpp'     => 'pkg-config --exists libmaxminddb',
                'm_mysql.cpp'           => 'mysql_config --version',
index 656c9809711be700eb356ed18af77e66ec7495c6..a49ad13186f864995e2c5c36e649530b929df3cc 100644 (file)
@@ -65,8 +65,6 @@ sub __get_socketengines {
        return @socketengines;
 }
 
-# TODO: when buildtool is done this can be mostly removed with
-#       the remainder being merged into parse_templates.
 sub __get_template_settings($$$) {
 
        # These are actually hash references
index de9271c879b112ccef1eb76a5803e40a60ef09be..7029accc052a6c38ae6ee23cafb5f943c59d2671 100644 (file)
@@ -469,7 +469,6 @@ void OperInfo::init()
        }
 
        // Compatibility for older configs that don't have the snomasks field.
-       // TODO: remove this before v4 is released.
        if (defaultsnomasks)
                AllowedSnomasks.set();
 }