diff options
| author | Florent Villard <warly@mandriva.com> | 2006-10-17 16:04:49 +0000 |
|---|---|---|
| committer | Florent Villard <warly@mandriva.com> | 2006-10-17 16:04:49 +0000 |
| commit | 1dc5e5210b35dcce9f36c315fa7170ef874c27e3 (patch) | |
| tree | 185b0b7ce17ef9ae97f6c2a0b77cc6406c0edb32 /lib/Youri/Submit/Check/SVN.pm | |
| parent | 36dc49a0fee40e68284ac1422255787fb740fee3 (diff) | |
| download | mga-youri-submit-1dc5e5210b35dcce9f36c315fa7170ef874c27e3.tar mga-youri-submit-1dc5e5210b35dcce9f36c315fa7170ef874c27e3.tar.gz mga-youri-submit-1dc5e5210b35dcce9f36c315fa7170ef874c27e3.tar.bz2 mga-youri-submit-1dc5e5210b35dcce9f36c315fa7170ef874c27e3.tar.xz mga-youri-submit-1dc5e5210b35dcce9f36c315fa7170ef874c27e3.zip | |
now checks must return the error message
Diffstat (limited to 'lib/Youri/Submit/Check/SVN.pm')
| -rw-r--r-- | lib/Youri/Submit/Check/SVN.pm | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/Youri/Submit/Check/SVN.pm b/lib/Youri/Submit/Check/SVN.pm index 0c92ce4..8295656 100644 --- a/lib/Youri/Submit/Check/SVN.pm +++ b/lib/Youri/Submit/Check/SVN.pm @@ -34,7 +34,7 @@ sub run { if ($section =~ /\/(testing|backport)$/) { # FIXME, right now ignore packages in SVN for testing and backports # we need to find a clean way to handle them - return 1 + return } $package->is_source or return 1; @@ -42,14 +42,11 @@ sub run { my $srpm_name = $package->get_canonical_name; if ($repository->package_in_svn($srpm_name)) { if ($file !~ /(^|\/|$define->{prefix}_)@\d+:\Q$srpm_name/) { - $self->{_error} = "package $srpm_name is in the SVN, the uploaded SRPM must look like @<svn rev>:$srpm_name-<version>-<release>.src.rpm (created with getsrpm-mdk $srpm_name)"; - return 0 + return "package $srpm_name is in the SVN, the uploaded SRPM must look like @<svn rev>:$srpm_name-<version>-<release>.src.rpm (created with getsrpm-mdk $srpm_name)"; } else { print "Package $file is correct\n"; - return 1 } } - 1 } sub simple_prompt { |