Hello community,

here is the log from the commit of package 
perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash for openSUSE:Leap:15.2 
checked in at 2020-02-21 10:49:43
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing 
/work/SRC/openSUSE:Leap:15.2/perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash
 (Old)
 and      
/work/SRC/openSUSE:Leap:15.2/.perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.new.26092
 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash"

Fri Feb 21 10:49:43 2020 rev:1 rq:775855 version:0.008

Changes:
--------
New Changes file:

--- /dev/null   2019-12-19 10:12:34.003146842 +0100
+++ 
/work/SRC/openSUSE:Leap:15.2/.perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.new.26092/perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.changes
   2020-02-21 10:49:44.166968804 +0100
@@ -0,0 +1,15 @@
+-------------------------------------------------------------------
+Tue Apr 23 05:21:43 UTC 2019 - Stephan Kulow <[email protected]>
+
+- updated to 0.008
+   see 
/usr/share/doc/packages/perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash/Changes
+
+  0.008     2019-04-22 16:44:54+03:00 Asia/Jerusalem
+  
+      * GH #3: Some typo fixed. (Jakub Wilk)
+
+-------------------------------------------------------------------
+Wed Apr  3 07:51:39 UTC 2019 - Stephan Kulow <[email protected]>
+
+- initial package 0.007
+ * created by cpanspec 1.78.09

New:
----
  Perl-Critic-Policy-Variables-ProhibitLoopOnHash-0.008.tar.gz
  cpanspec.yml
  perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.changes
  perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.spec

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash.spec ++++++
#
# spec file for package perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash
#
# Copyright (c) 2019 SUSE LINUX GmbH, Nuernberg, Germany.
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
# upon. The license for this file, and modifications and additions to the
# file, is the same license as for the pristine package itself (unless the
# license for the pristine package is not an Open Source License, in which
# case the license is the MIT License). An "Open Source License" is a
# license that conforms to the Open Source Definition (Version 1.9)
# published by the Open Source Initiative.

# Please submit bugfixes or comments via https://bugs.opensuse.org/
#


Name:           perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash
Version:        0.008
Release:        0
%define cpan_name Perl-Critic-Policy-Variables-ProhibitLoopOnHash
Summary:        Don't write loops on hashes, only on keys and values of hashes
License:        Artistic-1.0 OR GPL-1.0-or-later
Group:          Development/Libraries/Perl
Url:            https://metacpan.org/release/%{cpan_name}
Source0:        
https://cpan.metacpan.org/authors/id/X/XS/XSAWYERX/%{cpan_name}-%{version}.tar.gz
Source1:        cpanspec.yml
BuildArch:      noarch
BuildRoot:      %{_tmppath}/%{name}-%{version}-build
BuildRequires:  perl
BuildRequires:  perl-macros
BuildRequires:  perl(List::Util) >= 1.33
BuildRequires:  perl(Perl::Critic) >= 1.126
BuildRequires:  perl(parent)
Requires:       perl(List::Util) >= 1.33
Requires:       perl(Perl::Critic) >= 1.126
Requires:       perl(parent)
%{perl_requires}

%description
When "looping over hashes," we mean looping over hash keys or hash values.
If you forgot to call 'keys' or 'values' you will accidentally loop over
both.

    foreach my $foo (%hash) {...}        # not ok
    action() for %hash;                  # not ok
    foreach my $foo ( keys %hash ) {...} # ok
    action() for values %hash;           # ok

An effort is made to detect expressions:

    action() for %hash ? keys %hash : ();                             # ok
    action() for %{ $hash{'stuff'} } ? keys %{ $hash{'stuff'} } : (); # ok

(Granted, the second example there doesn't make much sense, but I have
found a variation of it in real code.)

%prep
%setup -q -n %{cpan_name}-%{version}

%build
perl Makefile.PL INSTALLDIRS=vendor
make %{?_smp_mflags}

%check
make test

%install
%perl_make_install
%perl_process_packlist
%perl_gen_filelist

%files -f %{name}.files
%defattr(-,root,root,755)
%doc Changes README
%license LICENSE

%changelog
++++++ cpanspec.yml ++++++
---
#description_paragraphs: 3
#description: |-
#  override description from CPAN
#summary: override summary from CPAN
#no_testing: broken upstream
#sources:
#  - source1
#  - source2
#patches:
#  foo.patch: -p1
#  bar.patch:
#preamble: |-
# BuildRequires:  gcc-c++
#post_prep: |-
# hunspell=`pkg-config --libs hunspell | sed -e 's,-l,,; s,  *,,g'`
# sed -i -e "s,hunspell-X,$hunspell," t/00-prereq.t Makefile.PL 
#post_build: |-
# rm unused.files
#post_install: |-
# sed on %{name}.files
#license: SUSE-NonFree
#skip_noarch: 1
#custom_build: |-
#./Build build flags=%{?_smp_mflags} --myflag
#custom_test: |-
#startserver && make test
#ignore_requires: Bizarre::Module

Reply via email to