[Rt-commit] rt branch, 4.2/devel-mode-and-testdeps, created. rt-4.1.6-213-gd3b5854

Thomas Sibley trs at bestpractical.com
Fri Feb 8 14:09:23 EST 2013


The branch, 4.2/devel-mode-and-testdeps has been created
        at  d3b585421480601c2988796c598ae656398f1d31 (commit)

- Log -----------------------------------------------------------------
commit 51d5bf2a1422925c5e48a0d8e6025456dabce29c
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:02:40 2012 -0500

    Make Module::Refresh a core dependency
    
    This means that --devel-mode is only necessary for people who intend to
    run tests or do other development of RT, but that users who merely
    intend to extend RT using Mason, for instance, do not need the extensive
    list of dependencies used during testing.

diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index 01ff8c8..a89d9da 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -229,6 +229,7 @@ Regexp::IPv6
 Symbol::Global::Name
 HTML::Entities
 Role::Basic 0.12
+Module::Refresh 0.03
 .
 
 $deps{'MASON'} = [ text_to_hash( << '.') ];
@@ -285,7 +286,6 @@ HTML::Form
 HTML::TokeParser
 WWW::Mechanize 1.52
 Test::WWW::Mechanize 1.30
-Module::Refresh 0.03
 Test::Expect 0.31
 XML::Simple
 File::Find

commit bc5c27fd5a1c7c551315409c969c91227d8e27a2
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:05:57 2012 -0500

    Rename --devel-mode to --developer, to make break in meaning clear
    
    Previously, --devel-mode both added the dependencies needed for testing
    RT, as well as setting the default value of $DevelMode in RT_Config.pm
    These actions are orthogonal, as many users wish to enable $DevelMode
    but do not need the additional dependencies; and developers may not wish
    to run with $DevelMode.
    
    By renaming the option, existing users of --devel-mode will no longer
    have the additional dependencies if all they wish is Module::Refresh.
    This also makes the setting of $DevelMode in RT_Config.pm no longer
    controlled at configure-time, which is an odd action-at-a-distance, as
    it means one RT's defaults are malleable from one install to another.
    Consistency is improved by forcing users to add it to RT_SiteConfig.pm,
    like any other option.

diff --git a/configure.ac b/configure.ac
index ee56d62..d005b26 100755
--- a/configure.ac
+++ b/configure.ac
@@ -251,19 +251,19 @@ AC_ARG_WITH(apachectl,
             APACHECTL=`which apachectl`)
 AC_SUBST(APACHECTL)
 
-dnl RT's "maintainer mode"
-AC_ARG_WITH(devel-mode,[],RT_DEVEL_MODE=$withval,RT_DEVEL_MODE="0")
-AC_ARG_ENABLE(devel-mode,
-            AC_HELP_STRING([--enable-devel-mode],
-                           [Turn on development aids that might hurt you in production]),
-            RT_DEVEL_MODE=$enableval,
-            RT_DEVEL_MODE=$RT_DEVEL_MODE)
-if test "$RT_DEVEL_MODE" = yes; then
-        RT_DEVEL_MODE="1"
+dnl Dependencies for testing and developing RT
+AC_ARG_WITH(developer,[],RT_DEVELOPER=$withval,RT_DEVELOPER="0")
+AC_ARG_ENABLE(developer,
+            AC_HELP_STRING([--enable-developer],
+                           [Add dependencies needed for testing and developing RT]),
+            RT_DEVELOPER=$enableval,
+            RT_DEVELOPER=$RT_DEVELOPER)
+if test "$RT_DEVELOPER" = yes; then
+        RT_DEVELOPER="1"
 else
-        RT_DEVEL_MODE="0"
+        RT_DEVELOPER="0"
 fi
-AC_SUBST(RT_DEVEL_MODE)
+AC_SUBST(RT_DEVELOPER)
 
 dnl RT's GraphViz dependency charts
 AC_CHECK_LIB([graph],[aginitlib],RT_GRAPHVIZ="1")
diff --git a/etc/RT_Config.pm.in b/etc/RT_Config.pm.in
index 1948731..42a812d 100755
--- a/etc/RT_Config.pm.in
+++ b/etc/RT_Config.pm.in
@@ -2687,7 +2687,7 @@ be added while the server is running.
 
 =cut
 
-Set($DevelMode, "@RT_DEVEL_MODE@");
+Set($DevelMode, 0);
 
 
 =item C<$RecordBaseClass>
diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index a89d9da..52a2528 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -66,7 +66,7 @@ GetOptions(
     'with-MODPERL1',                      'with-MODPERL2',
     'with-STANDALONE',
 
-    'with-DEV',
+    'with-DEVELOPER',
 
     'with-GPG',
     'with-ICAL',
@@ -96,7 +96,7 @@ my %default = (
     'with-CORE' => 1,
     'with-CLI' => 1,
     'with-MAILGATE' => 1, 
-    'with-DEV' => @RT_DEVEL_MODE@, 
+    'with-DEVELOPER' => @RT_DEVELOPER@,
     'with-GPG' => @RT_GPG@,
     'with-ICAL' => 1,
     'with-GRAPHVIZ' => @RT_GRAPHVIZ@,
@@ -104,7 +104,7 @@ my %default = (
     'with-DASHBOARDS' => 1,
     'with-USERLOGO' => 1,
     'with-SSL-MAILGATE' => @RT_SSL_MAILGATE@,
-    'with-HTML-DOC' => @RT_DEVEL_MODE@,
+    'with-HTML-DOC' => @RT_DEVELOPER@,
 );
 $args{$_} = $default{$_} foreach grep !exists $args{$_}, keys %default;
 
@@ -279,7 +279,7 @@ Term::ReadLine
 Term::ReadKey
 .
 
-$deps{'DEV'} = [ text_to_hash( << '.') ];
+$deps{'DEVELOPER'} = [ text_to_hash( << '.') ];
 Email::Abstract
 Test::Email
 HTML::Form
@@ -676,7 +676,7 @@ list the status of all dependencies, rather than just the missing ones.
 
     libraries needed to support the modperl 2 handler
 
-=item with-dev
+=item with-developer
 
     tools needed for RT development
 

commit c11486346233f98f94918b77380a3708d847bc77
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:12:20 2012 -0500

    Sort dependencies and remove duplicates

diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index 52a2528..93fe3b2 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -181,79 +181,79 @@ sub text_to_hash {
 }
 
 $deps{'CORE'} = [ text_to_hash( << '.') ];
+Cache::Simple::TimedExpiry
 Class::Accessor 0.34
+Class::ReturnValue 0.40
+CSS::Squish 0.06
 DateTime 0.44
 DateTime::Locale 0.40
+DBI 1.37
+DBIx::SearchBuilder 1.63
+Devel::GlobalDestruction
+Devel::StackTrace 1.19
 Digest::base
 Digest::MD5 2.27
 Digest::SHA
-DBI 1.37
-Class::ReturnValue 0.40
-DBIx::SearchBuilder 1.63
-Text::Template 1.44
+Email::Address
+Encode 2.39
+File::Glob
 File::ShareDir
 File::Spec 0.8
+File::Temp 0.19
+HTML::Entities
+HTML::FormatText
 HTML::Quoted
 HTML::Scrubber 0.08
 HTML::TreeBuilder
-HTML::FormatText
-Log::Dispatch 2.30
-Sys::Syslog 0.16
+List::MoreUtils
 Locale::Maketext 1.06
-Locale::Maketext::Lexicon 0.32
 Locale::Maketext::Fuzzy 0.11
-MIME::Entity 5.425
+Locale::Maketext::Lexicon 0.32
+Log::Dispatch 2.30
 Mail::Mailer 1.57
-Email::Address
-Text::Wrapper 
-Time::ParseDate
-Time::HiRes 
-File::Temp 0.19
-Text::Quoted 2.02
-Tree::Simple 1.04
-UNIVERSAL::require
-Regexp::Common
-Scalar::Util
+MIME::Entity 5.425
+Module::Refresh 0.03
 Module::Versions::Report 1.05
-Cache::Simple::TimedExpiry
-Encode 2.39
-CSS::Squish 0.06
-File::Glob
-Devel::StackTrace 1.19
-Text::Password::Pronounceable
-Devel::GlobalDestruction
-List::MoreUtils
 Net::CIDR
+Regexp::Common
 Regexp::Common::net::CIDR
 Regexp::IPv6
-Symbol::Global::Name
-HTML::Entities
 Role::Basic 0.12
-Module::Refresh 0.03
+Scalar::Util
+Symbol::Global::Name
+Sys::Syslog 0.16
+Text::Password::Pronounceable
+Text::Quoted 2.02
+Text::Template 1.44
+Text::Wrapper
+Time::HiRes
+Time::ParseDate
+Tree::Simple 1.04
+UNIVERSAL::require
 .
 
 $deps{'MASON'} = [ text_to_hash( << '.') ];
-HTML::Mason 1.43
-Errno
-Digest::MD5 2.27
-CGI::Cookie 1.20
-Storable 2.08
 Apache::Session 1.53
-XML::RSS 1.05
-Text::WikiFormat 0.76
+CGI::Cookie 1.20
 CSS::Squish 0.06
 Devel::StackTrace 1.19
-JSON
+Digest::MD5 2.27
+Errno
+HTML::Mason 1.43
 IPC::Run3
+JSON
+Storable 2.08
+Text::WikiFormat 0.76
+XML::RSS 1.05
 .
 
 $deps{'PSGI'} = [ text_to_hash( << '.') ];
 CGI 3.38
+CGI::Emulate::PSGI
 CGI::PSGI 0.12
 HTML::Mason::PSGIHandler 0.52
 Plack 0.9971
 Plack::Handler::Starlet
-CGI::Emulate::PSGI
 .
 
 $deps{'MAILGATE'} = [ text_to_hash( << '.') ];
@@ -264,44 +264,43 @@ Pod::Usage
 
 $deps{'SSL-MAILGATE'} = [ text_to_hash( << '.') ];
 Crypt::SSLeay
-Net::SSL
-LWP::UserAgent 6.0
 LWP::Protocol::https
+LWP::UserAgent 6.0
 Mozilla::CA
+Net::SSL
 .
 
 $deps{'CLI'} = [ text_to_hash( << '.') ];
 Getopt::Long 2.24
-LWP
 HTTP::Request::Common
-Text::ParseWords
-Term::ReadLine
+LWP
 Term::ReadKey
+Term::ReadLine
+Text::ParseWords
 .
 
 $deps{'DEVELOPER'} = [ text_to_hash( << '.') ];
 Email::Abstract
-Test::Email
+File::Find
 HTML::Form
 HTML::TokeParser
-WWW::Mechanize 1.52
-Test::WWW::Mechanize 1.30
-Test::Expect 0.31
-XML::Simple
-File::Find
-Test::Deep 0 # needed for shredder tests
-String::ShellQuote 0 # needed for gnupg-incoming.t
 Log::Dispatch::Perl
-Test::Warn
-Test::Builder 0.90 # needed for is_passing
-Test::MockTime
-Log::Dispatch::Perl
-Test::WWW::Mechanize::PSGI
-Plack::Middleware::Test::StashWarnings 0.06
-Test::LongString
 Mojo::DOM
+Plack::Middleware::Test::StashWarnings 0.06
 Set::Tiny
+String::ShellQuote 0 # needed for gnupg-incoming.t
+Test::Builder 0.90 # needed for is_passing
+Test::Deep 0 # needed for shredder tests
+Test::Email
+Test::Expect 0.31
+Test::LongString
+Test::MockTime
 Test::NoWarnings
+Test::Warn
+Test::WWW::Mechanize 1.30
+Test::WWW::Mechanize::PSGI
+WWW::Mechanize 1.52
+XML::Simple
 .
 
 $deps{'FASTCGI'} = [ text_to_hash( << '.') ];
@@ -310,13 +309,12 @@ FCGI::ProcManager
 .
 
 $deps{'MODPERL1'} = [ text_to_hash( << '.') ];
-Apache::Request
 Apache::DBI 0.92
+Apache::Request
 .
 
 $deps{'MODPERL2'} = [ text_to_hash( << '.') ];
 Apache::DBI
-HTML::Mason 1.36
 .
 
 $deps{'MYSQL'} = [ text_to_hash( << '.') ];
@@ -366,8 +364,8 @@ Convert::Color
 .
 
 $deps{'HTML-DOC'} = [ text_to_hash( <<'.') ];
-Pod::Simple 3.17
 HTML::Entities
+Pod::Simple 3.17
 .
 
 my %AVOID = (

commit 6c0f1b26c0ecfd0c0dfab193a03ee3509d1b9d7c
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:21:17 2012 -0500

    Merge core dependencies into one list, and eliminate duplicates
    
    PSGI and MASON are core dependencies which cannot be disabled using
    command-line flags; fold them into CORE.  Though CLI and MAILGATE are
    similarly required, and cannot be disabled using command-line flags,
    keep them separate for now, as they are standalone binaries which are
    intended to be able to be distributed separately.  Keeping a separate
    list of their dependencies is thus of use.

diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index 93fe3b2..15939cc 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -91,8 +91,6 @@ if ( $args{help} ) {
 
 # Set up defaults
 my %default = (
-    'with-MASON' => 1,
-    'with-PSGI' => 1,
     'with-CORE' => 1,
     'with-CLI' => 1,
     'with-MAILGATE' => 1, 
@@ -181,7 +179,12 @@ sub text_to_hash {
 }
 
 $deps{'CORE'} = [ text_to_hash( << '.') ];
+Apache::Session 1.53
 Cache::Simple::TimedExpiry
+CGI 3.38
+CGI::Cookie 1.20
+CGI::Emulate::PSGI
+CGI::PSGI 0.12
 Class::Accessor 0.34
 Class::ReturnValue 0.40
 CSS::Squish 0.06
@@ -196,15 +199,20 @@ Digest::MD5 2.27
 Digest::SHA
 Email::Address
 Encode 2.39
+Errno
 File::Glob
 File::ShareDir
 File::Spec 0.8
 File::Temp 0.19
 HTML::Entities
 HTML::FormatText
+HTML::Mason 1.43
+HTML::Mason::PSGIHandler 0.52
 HTML::Quoted
 HTML::Scrubber 0.08
 HTML::TreeBuilder
+IPC::Run3
+JSON
 List::MoreUtils
 Locale::Maketext 1.06
 Locale::Maketext::Fuzzy 0.11
@@ -215,47 +223,28 @@ MIME::Entity 5.425
 Module::Refresh 0.03
 Module::Versions::Report 1.05
 Net::CIDR
+Plack 0.9971
+Plack::Handler::Starlet
 Regexp::Common
 Regexp::Common::net::CIDR
 Regexp::IPv6
 Role::Basic 0.12
 Scalar::Util
+Storable 2.08
 Symbol::Global::Name
 Sys::Syslog 0.16
 Text::Password::Pronounceable
 Text::Quoted 2.02
 Text::Template 1.44
+Text::WikiFormat 0.76
 Text::Wrapper
 Time::HiRes
 Time::ParseDate
 Tree::Simple 1.04
 UNIVERSAL::require
-.
-
-$deps{'MASON'} = [ text_to_hash( << '.') ];
-Apache::Session 1.53
-CGI::Cookie 1.20
-CSS::Squish 0.06
-Devel::StackTrace 1.19
-Digest::MD5 2.27
-Errno
-HTML::Mason 1.43
-IPC::Run3
-JSON
-Storable 2.08
-Text::WikiFormat 0.76
 XML::RSS 1.05
 .
 
-$deps{'PSGI'} = [ text_to_hash( << '.') ];
-CGI 3.38
-CGI::Emulate::PSGI
-CGI::PSGI 0.12
-HTML::Mason::PSGIHandler 0.52
-Plack 0.9971
-Plack::Handler::Starlet
-.
-
 $deps{'MAILGATE'} = [ text_to_hash( << '.') ];
 Getopt::Long
 LWP::UserAgent 6.0

commit f78b44b3275e492cb510939d6340a37eb44601cc
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:29:51 2012 -0500

    Rename --with-postgresql to --with-pg, which is what the makefile calls

diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index 15939cc..e375d72 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -61,7 +61,7 @@ my @orig_argv = @ARGV;
 GetOptions(
     \%args,                               'v|verbose',
     'install!',                           'with-MYSQL',
-    'with-POSTGRESQL|with-pg|with-pgsql', 'with-SQLITE',
+    'with-PG',                            'with-SQLITE',
     'with-ORACLE',                        'with-FASTCGI',
     'with-MODPERL1',                      'with-MODPERL2',
     'with-STANDALONE',
@@ -314,7 +314,7 @@ $deps{'ORACLE'} = [ text_to_hash( << '.') ];
 DBD::Oracle
 .
 
-$deps{'POSTGRESQL'} = [ text_to_hash( << '.') ];
+$deps{'PG'} = [ text_to_hash( << '.') ];
 DBD::Pg 1.43
 .
 
@@ -639,7 +639,7 @@ list the status of all dependencies, rather than just the missing ones.
 
     database interface for mysql
 
-=item --with-postgresql
+=item --with-pg
 
     database interface for postgresql 
 

commit d3b585421480601c2988796c598ae656398f1d31
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Mon Nov 5 18:30:20 2012 -0500

    Minor reformatting of options and POD

diff --git a/sbin/rt-test-dependencies.in b/sbin/rt-test-dependencies.in
index e375d72..e4275a9 100644
--- a/sbin/rt-test-dependencies.in
+++ b/sbin/rt-test-dependencies.in
@@ -60,11 +60,9 @@ my %deps;
 my @orig_argv = @ARGV;
 GetOptions(
     \%args,                               'v|verbose',
-    'install!',                           'with-MYSQL',
-    'with-PG',                            'with-SQLITE',
-    'with-ORACLE',                        'with-FASTCGI',
-    'with-MODPERL1',                      'with-MODPERL2',
-    'with-STANDALONE',
+    'install!',
+    'with-MYSQL', 'with-PG', 'with-SQLITE', 'with-ORACLE',
+    'with-FASTCGI', 'with-MODPERL1', 'with-MODPERL2', 'with-STANDALONE',
 
     'with-DEVELOPER',
 
@@ -637,35 +635,37 @@ list the status of all dependencies, rather than just the missing ones.
 
 =item --with-mysql
 
-    database interface for mysql
+database interface for mysql
 
 =item --with-pg
 
-    database interface for postgresql 
+database interface for postgresql
 
-=item with-oracle       
-    
-    database interface for oracle
+=item --with-oracle
+
+database interface for oracle
+
+=item --with-sqlite
 
-=item with-sqlite 
+database interface and driver for sqlite (unsupported)
 
-    database interface and driver for sqlite (unsupported)
+=item --with-fastcgi
 
-=item with-fastcgi 
+libraries needed to support the fastcgi handler
 
-    libraries needed to support the fastcgi handler
+=item --with-modperl1
 
-=item with-modperl1
+libraries needed to support the modperl 1 handler
 
-    libraries needed to support the modperl 1 handler
+=item --with-modperl2
 
-=item with-modperl2
+libraries needed to support the modperl 2 handler
 
-    libraries needed to support the modperl 2 handler
+=item --with-developer
 
-=item with-developer
+tools needed for RT development
 
-    tools needed for RT development
+=item
 
 =back
 

-----------------------------------------------------------------------


More information about the Rt-commit mailing list