diff options
author | Pascal Terjan <pterjan@mageia.org> | 2011-01-22 14:49:03 +0000 |
---|---|---|
committer | Pascal Terjan <pterjan@mageia.org> | 2011-01-22 14:49:03 +0000 |
commit | 9b8e206898849b530b72f9ca803eab9b711f1a4a (patch) | |
tree | 53279a682204d56f22ba86a4e829c428ba25d57b /lib/Youri/Submit/Check/Deps.pm | |
parent | 7654a1f262d0da0cb3ddea8c92baf07cfab3b1be (diff) | |
download | mga-youri-submit-9b8e206898849b530b72f9ca803eab9b711f1a4a.tar mga-youri-submit-9b8e206898849b530b72f9ca803eab9b711f1a4a.tar.gz mga-youri-submit-9b8e206898849b530b72f9ca803eab9b711f1a4a.tar.bz2 mga-youri-submit-9b8e206898849b530b72f9ca803eab9b711f1a4a.tar.xz mga-youri-submit-9b8e206898849b530b72f9ca803eab9b711f1a4a.zip |
Ugly code rejecting submit when buildrequires are missing
Diffstat (limited to 'lib/Youri/Submit/Check/Deps.pm')
-rw-r--r-- | lib/Youri/Submit/Check/Deps.pm | 87 |
1 files changed, 87 insertions, 0 deletions
diff --git a/lib/Youri/Submit/Check/Deps.pm b/lib/Youri/Submit/Check/Deps.pm new file mode 100644 index 0000000..6812d38 --- /dev/null +++ b/lib/Youri/Submit/Check/Deps.pm @@ -0,0 +1,87 @@ +package Youri::Submit::Check::Deps; + +=head1 NAME + +Youri::Submit::Check::Deps - Check dependencies + +=head1 DESCRIPTION + +This check plugin rejects packages with unresolved dependencies. + +=cut + +use warnings; +use strict; +use Carp; +use Youri::Media::URPM; +use base qw/Youri::Submit::Check/; + +sub resolvedep { + my ($media, @requires) = @_; + + my @errors; + my $index = sub { + my ($package) = @_; + + my @provides = $package->get_provides(); + + @requires = grep { + my $require = $_; + my $notfound = 1; + foreach my $provide (@provides) { + next unless $provide->[Youri::Package::DEPENDENCY_NAME] eq $require->[Youri::Package::DEPENDENCY_NAME]; + if ($require->[Youri::Package::DEPENDENCY_RANGE]) { + next unless $package->check_ranges_compatibility($provide->[Youri::Package::DEPENDENCY_RANGE], $require->[Youri::Package::DEPENDENCY_RANGE]); + } + $notfound = 0; + } + + if ($notfound && $require->[Youri::Package::DEPENDENCY_NAME] =~ m|/|) { + foreach my $file ($package->get_files()) { + next unless $file eq $require->[Youri::Package::DEPENDENCY_NAME]; + $notfound = 0; + last; + } + } + $notfound; + } @requires; + }; + $media->traverse_headers($index); + foreach my $require (@requires) { + push (@errors, "Unresolved dep on " . $require->[Youri::Package::DEPENDENCY_NAME]); + } + return @errors; +} + +sub run { + my ($self, $package, $repository, $target, $define) = @_; + croak "Not a class method" unless ref $self; + + # FIXME Define some Youri::Media with allowed_deps in the config and + # match target + section to a media + my $section = $repository->_get_section($package, $target, $define); + return unless $target eq "cauldron" && $section eq 'core/release'; + + my @requires = $package->get_requires(); + + my $path = $repository->get_install_root() . "/" . $target; + # FIXME we need dependencies on all archs except for ExclusiveArch + my $arch = 'i586'; +# foreach my $arch ($repository->get_extra_arches()) { + my $media = new Youri::Media::URPM(name => "core.".$arch, + type => "binary", + hdlist => "$path/$arch/media/$section/media_info/hdlist.cz"); + return resolvedep($media, @requires); +# } + +} + +=head1 COPYRIGHT AND LICENSE + +Copyright (C) 2011, YOURI project + +This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. + +=cut + +1; |