Skip to content

Commit

Permalink
make it clear that we are not supposed to have optional deps
Browse files Browse the repository at this point in the history
  • Loading branch information
shadowcat-mst committed Sep 11, 2009
1 parent b7c7995 commit 93c5088
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
4 changes: 4 additions & 0 deletions Changes
@@ -1,5 +1,9 @@
Revision history for DBIx::Class

- remove the recommends from Makefile.PL, DBIx::Class is not
supposed to have optional dependencies. ever.
- mangle the DBIx/Class.pm POD to be more clear about
copyright and license
- Change ResultSetColumn->reset() to no longer return $self, which
fixes an issue with using Cursor::Cached and ResultSetColumn
together.
Expand Down
7 changes: 2 additions & 5 deletions Makefile.PL
Expand Up @@ -5,6 +5,8 @@ use POSIX ();

use 5.006001; # delete this line if you want to send patches for earlier.

# ****** DO NOT ADD OPTIONAL DEPENDENCIES. EVER. --mst ******

name 'DBIx-Class';
perl_version '5.006001';
all_from 'lib/DBIx/Class.pm';
Expand Down Expand Up @@ -52,11 +54,6 @@ my %replication_requires = (
'Hash::Merge', => '0.11',
);

# when changing also adjust $DBIx::Class::minimum_sqlt_version
my $sqlt_recommends = '0.11002';

recommends 'SQL::Translator' => $sqlt_recommends;

my %force_requires_if_author = (
%replication_requires,

Expand Down

0 comments on commit 93c5088

Please sign in to comment.