[Bug 1168511] Review Request: radamsa - test case generator for robustness testing, aka a fuzzer
https://bugzilla.redhat.com/show_bug.cgi?id=1168511 --- Comment #29 from Fedora Update System --- radamsa-0.4-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/radamsa-0.4-2.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168511] Review Request: radamsa - test case generator for robustness testing, aka a fuzzer
https://bugzilla.redhat.com/show_bug.cgi?id=1168511 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1174288] Review Request: owl-lisp - functional dialect of Scheme
https://bugzilla.redhat.com/show_bug.cgi?id=1174288 Niranjan MR changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2015-01-22 03:55:57 --- Comment #19 from Niranjan MR --- owl-lisp is now in Rawhide, fedora20 and fedora21 https://admin.fedoraproject.org/updates/FEDORA-2015-0662/owl-lisp-0.1.7-2.fc20 https://admin.fedoraproject.org/updates/FEDORA-2015-0649/owl-lisp-0.1.7-2.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168511] Review Request: radamsa - test case generator for robustness testing, aka a fuzzer
https://bugzilla.redhat.com/show_bug.cgi?id=1168511 Bug 1168511 depends on bug 1174288, which changed state. Bug 1174288 Summary: Review Request: owl-lisp - functional dialect of Scheme https://bugzilla.redhat.com/show_bug.cgi?id=1174288 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1171129] Review Request: freeradius-client - Client library and utilities for radius
https://bugzilla.redhat.com/show_bug.cgi?id=1171129 --- Comment #8 from Nikos Mavrogiannopoulos --- Updated version on 1.1.7: Spec URL: http://people.redhat.com/nmavrogi/fedora/freeradius-client.spec SRPM URL: http://people.redhat.com/nmavrogi/fedora/freeradius-client-1.1.7-1.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1020456] Review Request: vagrant - an automation tool used to manage development environments
https://bugzilla.redhat.com/show_bug.cgi?id=1020456 --- Comment #40 from Vít Ondruch --- (In reply to Michael Adam from comment #38) > OK, bug has been assigned back to me. > Do I have to anything else to it? Actually the bug is typically assigned just to reviewer and we don't handover it unless the reviewer is changed in the process. So that should be it. (In reply to Michael Adam from comment #39) > Hmm, according to > http://fedoraproject.org/wiki/New_package_process_for_existing_contributors > the resolution should be set to NEXTRELEASE instead of RAWHIDE. Well, I don't think the policy covers everything. I'd use NEXTRELEASE if we were in branched state, e.g. I'd push this to F22 + Rawhide, but at this moment, we are not branched yet, so the RAWHIDE is more appropriate. If I was pushing this into F21, the the bug would be closed by Bodhi with ERRATA resolution if I am not mistaken. But that is just my personal view on the matter and it does not influence anything, as far as I know. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180460] Review Request: ghc-JuicyPixels - Picture loading and serialization
https://bugzilla.redhat.com/show_bug.cgi?id=1180460 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180460] Review Request: ghc-JuicyPixels - Picture loading and serialization
https://bugzilla.redhat.com/show_bug.cgi?id=1180460 --- Comment #8 from Fedora Update System --- ghc-JuicyPixels-3.2.2-1.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/ghc-JuicyPixels-3.2.2-1.el7 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180460] Review Request: ghc-JuicyPixels - Picture loading and serialization
https://bugzilla.redhat.com/show_bug.cgi?id=1180460 --- Comment #6 from Fedora Update System --- ghc-JuicyPixels-3.2.2-1.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/ghc-JuicyPixels-3.2.2-1.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180460] Review Request: ghc-JuicyPixels - Picture loading and serialization
https://bugzilla.redhat.com/show_bug.cgi?id=1180460 --- Comment #7 from Fedora Update System --- ghc-JuicyPixels-3.2.2-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/ghc-JuicyPixels-3.2.2-1.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180460] Review Request: ghc-JuicyPixels - Picture loading and serialization
https://bugzilla.redhat.com/show_bug.cgi?id=1180460 --- Comment #9 from Jens Petersen --- Currently leaving the rawhide build for f22-ghc. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079090] Review Request: layla-fonts - A collection of traditional Arabic fonts
https://bugzilla.redhat.com/show_bug.cgi?id=1079090 Jonathan Dieter changed: What|Removed |Added CC||jdie...@lesbg.com --- Comment #8 from Jonathan Dieter --- Are you still interested in having this reviewed? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1020456] Review Request: vagrant - an automation tool used to manage development environments
https://bugzilla.redhat.com/show_bug.cgi?id=1020456 --- Comment #41 from Aleksandar Kostadinov --- offtopic but would it be possible to put vagrant officially in fedora21 now? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1020456] Review Request: vagrant - an automation tool used to manage development environments
https://bugzilla.redhat.com/show_bug.cgi?id=1020456 --- Comment #42 from Vít Ondruch --- (In reply to Aleksandar Kostadinov from comment #41) > offtopic but would it be possible to put vagrant officially in fedora21 now? I am sorry, but since the build in Rawhide is already slightly different then what would be needed for F21, I am not very keen to build Vagrant for F21. But I won't oppose to any volunteer ;) and there is still Josef's Copr repo: https://copr.fedoraproject.org/coprs/jstribny/vagrant-f21/ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1020456] Review Request: vagrant - an automation tool used to manage development environments
https://bugzilla.redhat.com/show_bug.cgi?id=1020456 --- Comment #43 from Michael Adam --- (In reply to Vít Ondruch from comment #42) > (In reply to Aleksandar Kostadinov from comment #41) > > offtopic but would it be possible to put vagrant officially in fedora21 now? > > I am sorry, but since the build in Rawhide is already slightly different > then what would be needed for F21, I am not very keen to build Vagrant for > F21. But I won't oppose to any volunteer ;) and there is still Josef's Copr > repo: https://copr.fedoraproject.org/coprs/jstribny/vagrant-f21/ I'd like to bring it to f21 too, but we need also at least two packages in f21 as dependency: - rubygem-hashicorp-checkpoint - rubygem-log4r These are currently only in rawhide. Michael -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #3 from Michael Adam --- I hit a couple of issues: In order to mock build the package at all, for the time being I first enable the [local] in my fedora-rawhide-x86_64 mock config, because official fedora repos don't have the vagrant package yet. (Thanks for the hint, Vít.) I am running "fedora-review -b 1168333 -m fedora-rawhide-x86_64.local". Now the build fails running %check: ~ + pushd ./usr/share/vagrant/gems/gems/vagrant-libvirt-0.0.23 + sed -i '/:git/ s|:git.*$|:path => "/usr/share/vagrant"|' Gemfile + sed -i '/rspec/ s|\[\".*\"]|["~> 2.0"]|' vagrant-libvirt.gemspec + bundle exec rspec2 spec Your Gemfile lists the gem vagrant-libvirt (>= 0) more than once. You should probably keep only one of them. While it's not a problem now, it could cause errors if you change the version of just one of them later. /usr/share/gems/gems/bundler-1.7.8/lib/bundler/resolver.rb:357:in `resolve': Could not find gem 'rake (= 10.1.0) ruby' in the gems available on this machine. (Bundler::GemNotFound) from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/resolver.rb:164:in `start' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/resolver.rb:129:in `resolve' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/definition.rb:193:in `resolve' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/definition.rb:132:in `specs' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/definition.rb:177:in `specs_for' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/definition.rb:166:in `requested_specs' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/environment.rb:18:in `requested_specs' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/runtime.rb:13:in `setup' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler.rb:122:in `setup' from /usr/share/gems/gems/bundler-1.7.8/lib/bundler/setup.rb:17:in `' from /usr/share/rubygems/rubygems/core_ext/kernel_require.rb:54:in `require' from /usr/share/rubygems/rubygems/core_ext/kernel_require.rb:54:in `require' RPM build errors: error: Bad exit status from /var/tmp/rpm-tmp.1BBUt3 (%check) Bad exit status from /var/tmp/rpm-tmp.1BBUt3 (%check) Child return code was: 1 EXCEPTION: Command failed. See logs for output. # bash --login -c /usr/bin/rpmbuild -bb --target x86_64 --nodeps /builddir/build/SPECS/vagrant-libvirt.spec Traceback (most recent call last): File "/usr/lib/python2.7/site-packages/mockbuild/trace_decorator.py", line 84, in trace result = func(*args, **kw) File "/usr/lib/python2.7/site-packages/mockbuild/util.py", line 481, in do raise mockbuild.exception.Error("Command failed. See logs for output.\n # %s" % (command,), child.returncode) Error: Command failed. See logs for output. # bash --login -c /usr/bin/rpmbuild -bb --target x86_64 --nodeps /builddir/build/SPECS/vagrant-libvirt.spec LEAVE do --> EXCEPTION RAISED ~ So there is 1 warning: - Gemfile lists vagrant-libvirt gem more than once. And one error: - The dependency for rake is wrong. Apparently 10.1.0 is hard-wired somewhere, but rawhide now sports 10.4.2: rubygem-rake-0:10.4.2-3.fc22.noarch It exceeds my ruby expertise to fix this quickly. Michael -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 Ralf Corsepius changed: What|Removed |Added Depends On||1184792 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184792 [Bug 1184792] RPE: Add fcontext to support rt -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #20 from Ralf Corsepius --- FYI rt-4.0.22 does not build for EPEL7: ... Error: No Package found for /usr/share/fonts/google-droid/DroidSans.ttf Error: No Package found for /usr/share/fonts/google-droid/DroidSansFallback.ttf Error: No Package found for perl(CGI::PSGI) Error: No Package found for perl(Cache::Simple::TimedExpiry) Error: No Package found for perl(Class::ReturnValue) >= 0.40 Error: No Package found for perl(Convert::Color) Error: No Package found for perl(DBIx::SearchBuilder) >= 1.59 Error: No Package found for perl(Data::ICal) Error: No Package found for perl(HTML::Mason) >= 1.43 Error: No Package found for perl(HTML::Mason::PSGIHandler) Error: No Package found for perl(HTML::Quoted) Error: No Package found for perl(HTML::RewriteAttributes) >= 0.02 Error: No Package found for perl(HTTP::Server::Simple::Mason) >= 0.09 Error: No Package found for perl(Imager) Error: No Package found for perl(Imager::File::GIF) Error: No Package found for perl(Imager::File::JPEG) Error: No Package found for perl(Imager::File::PNG) Error: No Package found for perl(Locale::Maketext::Fuzzy) Error: No Package found for perl(Locale::Maketext::Lexicon) >= 0.32 Error: No Package found for perl(Log::Dispatch::Perl) Error: No Package found for perl(Plack::Handler::Starlet) Error: No Package found for perl(Plack::Middleware::Test::StashWarnings) >= 0.06 Error: No Package found for perl(Regexp::Common::net::CIDR) Error: No Package found for perl(Term::EditorEdit) Error: No Package found for perl(Test::Email) Error: No Package found for perl(Test::Expect) >= 0.31 Error: No Package found for perl(Test::HTTP::Server::Simple) >= 0.09 Error: No Package found for perl(Test::WWW::Mechanize) Error: No Package found for perl(Test::WWW::Mechanize::PSGI) Error: No Package found for perl(Text::Password::Pronounceable) Error: No Package found for perl(Text::Quoted) >= 2.02 Error: No Package found for perl(Text::WikiFormat) >= 0.76 Error: No Package found for perl(Text::Wrapper) Error: No Package found for perl(Tree::Simple) >= 1.04 Error: No Package found for perl(Web::Scraper) ... Lots of perl-modules are missing. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1075605] Review Request: ghc-distributive - Haskell 98 Distributive functors -- Dual to Traversable
https://bugzilla.redhat.com/show_bug.cgi?id=1075605 Bug 1075605 depends on bug 1077543, which changed state. Bug 1077543 Summary: ghc-tagged-0.7.2 is available https://bugzilla.redhat.com/show_bug.cgi?id=1077543 What|Removed |Added Status|POST|CLOSED Resolution|--- |DUPLICATE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1075605] Review Request: ghc-distributive - Haskell 98 Distributive functors -- Dual to Traversable
https://bugzilla.redhat.com/show_bug.cgi?id=1075605 Jens Petersen changed: What|Removed |Added Depends On||1163135 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1163135 [Bug 1163135] ghc-tagged-0.7.3 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 David Dick changed: What|Removed |Added Depends On||1184809 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184809 [Bug 1184809] Request EPEL6 and EPEL7 branchs -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 --- Comment #4 from Vojtech Trefny --- (In reply to Jaroslav Škarvada from comment #3) > I will sponsor you after we finish the review. New spec/srpm: https://vtrefny.fedorapeople.org/spec/blivet-gui.spec https://vtrefny.fedorapeople.org/srpms/blivet-gui-0.2.0-5.fc21.src.rpm > > Issues found: > > - SPEC file provided in comment 0 is different than SPEC file in SRPM, diff: > @@ -33,5 +33,4 @@ > > %install > -rm -rf %{buildroot} > make DESTDIR=%{buildroot} install > > @@ -41,4 +40,7 @@ > %find_lang %{name} > > +%clean > +rm -rf %{buildroot} > + > %files -f %{name}.lang > %{_mandir}/man1/blivet-gui.1* > > - also please link the SPEC file directly (raw), not HTML. > > - checksums of archive included in SRPM and upstream archive are different: > 6e5dd8581fb8b73c17bd0981cd367a18 blivet-gui-0.2.0.tar.gz > 9ded8f2490c7561adbfe170bc4736262 blivet-gui-blivet-gui-0.2.0.tar.gz The problem here is I don't have localisation files in git repository -- MO files are in SPRM because I download them from Zanata during "make archive" (and upstream archives are made automatically by GitHub for every tag). > > Also the provided source0 URL serves blivet-gui-blivet-gui-0.2.0.tar.gz, not > blivet-gui-0.2.0.tar.gz. Fix the source0 URL or add workaround to SPEC, e.g.: > https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/ > SourceURL#Troublesome_URLs Fixed. > > - find_lang macro should be in %install section, not %check > > - desktop_file_validate should be also in %install section, please use the > %check section only if there is working %check target in your makefile with > useful built-time tests > > - in files section use macros, e.g. %{_datadir} for /usr/share, %{_bindir} > for /usr/bin > > - you don't need to run 'rm -rf %{buildroot}' in %install section > > - your package must not own /usr/share/applications and > /usr/share/polkit-1/actions directories, only own individual files/subdirs > Fixed. > > Minor, mostly cosmetic issues: > > - BuildRequires:intltool > Please add space after colon to be consistent > > - Url: http://github.com/rhinstaller/blivet-gui > Please rather use URL: (i.e. in capitals). Fixed. > > > Other: > > - Is the source code python 3 ready? In such case you should package it > conditionally for python 3 (http://fedoraproject.org/wiki/Packaging:Python). > If not, it's OK as is. Not it isn't because python-blivet is not python3 read yet. > > - Source package should contains license text, as upstream please add it. > > - Directories /usr/share/help and /usr/share/help/C are not owned, but this > is something you cannot resolve. AFAIK /usr/share/help is not in FHS > standard. This should be provided by some package. Question is which > package, gtk? gtk-doc? Any opinion? Bug should be filled against the > identified package to include these directories. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1135470] Review Request: libkscreen-qt5 - KDE display configuration library
https://bugzilla.redhat.com/show_bug.cgi?id=1135470 Daniel Vrátil changed: What|Removed |Added Summary|Review Request: kf5-kscreen |Review Request: |- A Tier 3 KDE Frameworks 5 |libkscreen-qt5 - KDE |Library with API to control |display configuration |screen settings |library Alias||libkscreen-qt5 --- Comment #7 from Daniel Vrátil --- Spec URL: https://dvratil.fedorapeople.org/plasma5/review/libkscreen-qt5.spec SRPM URL: https://dvratil.fedorapeople.org/plasma5/review/libkscreen-qt5-5.1.95-1.beta.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 Petr Pisar changed: What|Removed |Added Depends On||672246 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=672246 [Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 Bug 1183218 depends on bug 1184809, which changed state. Bug 1184809 Summary: Request EPEL6 and EPEL7 branchs https://bugzilla.redhat.com/show_bug.cgi?id=1184809 What|Removed |Added Status|NEW |CLOSED Resolution|--- |DUPLICATE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
https://bugzilla.redhat.com/show_bug.cgi?id=672246 Petr Pisar changed: What|Removed |Added Blocks||1183218 CC||dd...@cpan.org --- Comment #17 from Petr Pisar --- *** Bug 1184809 has been marked as a duplicate of this bug. *** Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1183218 [Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
https://bugzilla.redhat.com/show_bug.cgi?id=672246 Petr Pisar changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #18 from Petr Pisar --- Package Change Request == Package Name: perl-Statistics-Basic New Branches: el6 epel7 Owners: ddick InitialCC: perl-sig ddick offered to maintain this package in these branches, see bug #1184809. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 495179] Review Request: perl-Number-Format - Perl extension for formatting numbers
https://bugzilla.redhat.com/show_bug.cgi?id=495179 David Dick changed: What|Removed |Added Blocks||1184809 CC||dd...@cpan.org --- Comment #14 from David Dick --- *** Bug 1184790 has been marked as a duplicate of this bug. *** Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184809 [Bug 1184809] Request EPEL6 and EPEL7 branchs -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 495179] Review Request: perl-Number-Format - Perl extension for formatting numbers
https://bugzilla.redhat.com/show_bug.cgi?id=495179 --- Comment #15 from David Dick --- Hi Petr / Iain, Could i get this package built for EPEL7? I'm happy to build this package if required. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1135470] Review Request: libkscreen-qt5 - KDE display configuration library
https://bugzilla.redhat.com/show_bug.cgi?id=1135470 --- Comment #8 from Jan Grulich --- Output from rpmlint: libkscreen-qt5.src: W: invalid-url URL: https://projects.kde.org/projects/extragear/libs/libkscreen HTTP Error 404: Not Found libkscreen-qt5.src:1: W: macro-in-comment %global libkscreen-qt5.src:2: W: macro-in-comment %global libkscreen-qt5.src:17: W: macro-in-comment %{git_version} 1 packages and 0 specfiles checked; 0 errors, 4 warnings. No blocking issue spotted, just fix the url during import. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183001] Review Request: kdecoration - Plugin-based library to create window decorations
https://bugzilla.redhat.com/show_bug.cgi?id=1183001 Jan Grulich changed: What|Removed |Added Status|NEW |ASSIGNED CC||jgrul...@redhat.com Assignee|nob...@fedoraproject.org|jgrul...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755510] Review Request: gnome-shell-extension-system-monitor-applet - Gnome shell system monitor extension
https://bugzilla.redhat.com/show_bug.cgi?id=755510 --- Comment #74 from nicolas.vievi...@univ-valenciennes.fr --- (In reply to Jeff Peeler from comment #73) > Sorry about the silence here. Delay is not a problem for me. > Your package reviews seem good and I think you've learned a little more about > what the expectations are for Fedora in doing them. This bug properly has > FE-NEEDSPONSOR set (I can't sponsor), so if you don't hear anything in a few > days (since this comment will produce an email) I'd reach out to the fedora > development list. I believe that all that's required is for a sponsor to > approve your review and then an scm request can be done. Thanks for your comment and all you've done for this review request. Cordially, -- NVieville -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 --- Comment #5 from Jaroslav Škarvada --- (In reply to Vojtech Trefny from comment #4) > > 6e5dd8581fb8b73c17bd0981cd367a18 blivet-gui-0.2.0.tar.gz > > 9ded8f2490c7561adbfe170bc4736262 blivet-gui-blivet-gui-0.2.0.tar.gz > > The problem here is I don't have localisation files in git repository -- MO > files are in SPRM because I download them from Zanata during "make archive" > (and upstream archives are made automatically by GitHub for every tag). > It needs to match, could you host the resulting release anywhere? E.g. at fedorahosted.org, or use github release feature: https://github.com/blog/1547-release-your-software -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183001] Review Request: kdecoration - Plugin-based library to create window decorations
https://bugzilla.redhat.com/show_bug.cgi?id=1183001 Jan Grulich changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Jan Grulich --- Rpmlint --- Checking: kdecoration-5.1.95-2.beta.fc20.x86_64.rpm kdecoration-devel-5.1.95-2.beta.fc20.x86_64.rpm kdecoration-5.1.95-2.beta.fc20.src.rpm kdecoration.x86_64: W: spelling-error Summary(en_US) libraty -> library, Libra kdecoration.x86_64: W: spelling-error %description -l en_US libraty -> library, Libra kdecoration-devel.x86_64: W: only-non-binary-in-usr-lib kdecoration-devel.x86_64: W: no-documentation kdecoration.src: W: spelling-error Summary(en_US) libraty -> library, Libra kdecoration.src: W: spelling-error %description -l en_US libraty -> library, Libra 3 packages and 0 specfiles checked; 0 errors, 6 warnings. 1) You have a typo in description 2) Wrong url No other issue spotted and none from mentioned issues is blocking, APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1131825] Review Request: qtile - Small, flexible, scriptable tiling window manager
https://bugzilla.redhat.com/show_bug.cgi?id=1131825 Raphael Groner changed: What|Removed |Added CC||jdula...@fedoraproject.org --- Comment #9 from Raphael Groner --- Proposed F22 Self Contained Change: https://fedoraproject.org/wiki/Changes/qtile -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1131825] Review Request: qtile - Small, flexible, scriptable tiling window manager
https://bugzilla.redhat.com/show_bug.cgi?id=1131825 --- Comment #10 from John Dulaney --- 0.9.0 is the latest release of qtile; I have packaged the dependencies in Fedora and was planning on packaging 0.9.0. I would suggest packaging the latest version; I'll gladly help. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184920] New: Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri
https://bugzilla.redhat.com/show_bug.cgi?id=1184920 Bug ID: 1184920 Summary: Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: vondr...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/cgit/vondruch/public_git/rubygem-rails-dom-testing.git/tree/rubygem-rails-dom-testing.spec?id=a5212605c312a77dd6dbbc4f014be5102b81c1c8 SRPM URL: http://people.redhat.com/vondruch/rubygem-rails-dom-testing-1.0.5-1.fc22.src.rpm Description: Dom and Selector assertions for Rails applications. Fedora Account System Username: vondruch -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Vít Ondruch changed: What|Removed |Added Blocks||1184920 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184920 [Bug 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri
https://bugzilla.redhat.com/show_bug.cgi?id=1184920 Vít Ondruch changed: What|Removed |Added Depends On||1184130 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184130 [Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri
https://bugzilla.redhat.com/show_bug.cgi?id=1184920 Vít Ondruch changed: What|Removed |Added Blocks||1184910 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184910 [Bug 1184910] Ruby on Rails 4.2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Vít Ondruch changed: What|Removed |Added Blocks||1184910 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184910 [Bug 1184910] Ruby on Rails 4.2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Josef Stribny changed: What|Removed |Added Status|NEW |ASSIGNED CC||jstri...@redhat.com Assignee|nob...@fedoraproject.org|jstri...@redhat.com Flags||fedora-review+ --- Comment #1 from Josef Stribny --- The package builds and test suite passes although cannot be installed with older ActiveSupport. If the package works with the old ActiveSupport as well, perhaps we can adjust the dependency requirement. Otherwise the spec looks alright => APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
https://bugzilla.redhat.com/show_bug.cgi?id=672246 --- Comment #19 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
https://bugzilla.redhat.com/show_bug.cgi?id=672246 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183001] Review Request: kdecoration - Plugin-based library to create window decorations
https://bugzilla.redhat.com/show_bug.cgi?id=1183001 Daniel Vrátil changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Daniel Vrátil --- New Package SCM Request === Package Name: kdecoration Short Description: Plugin-based library to create window decorations Upstream URL: https://projects.kde.org/projects/kde/workspace/kdecoration Owners: group::kde-sig Branches: InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1135470] Review Request: libkscreen-qt5 - KDE display configuration library
https://bugzilla.redhat.com/show_bug.cgi?id=1135470 Daniel Vrátil changed: What|Removed |Added Flags||fedora-cvs? --- Comment #9 from Daniel Vrátil --- New Package SCM Request === Package Name: libkscreen-qt5 Short Description: KDE display configuration library Upstream URL: https://projects.kde.org/projects/kde/workspace/libkscreen Owners: group::kde-sig Branches: InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Vít Ondruch changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Vít Ondruch --- (In reply to Josef Stribny from comment #1) > The package builds and test suite passes although cannot be installed with > older ActiveSupport. If the package works with the old ActiveSupport as > well, perhaps we can adjust the dependency requirement. Yes, you are right. I already noticed it and fixed locally. > Otherwise the spec looks alright => APPROVED. Thanks for the review. New Package SCM Request === Package Name: rubygem-rails-deprecated_sanitizer Short Description: Deprecated sanitizer API extracted from Action View Upstream URL: https://github.com/rails/rails-deprecated_sanitizer Owners: vondruch jstribny Branches: InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079090] Review Request: layla-fonts - A collection of traditional Arabic fonts
https://bugzilla.redhat.com/show_bug.cgi?id=1079090 --- Comment #9 from Mohammed Isam --- (In reply to Jonathan Dieter from comment #8) > Are you still interested in having this reviewed? Definitely! This will be wonderful. Thanx a lot man. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1141880] Review Request: golang-googlecode-gcfg - Gcfg reads INI-style configuration files into Go structs
https://bugzilla.redhat.com/show_bug.cgi?id=1141880 --- Comment #13 from Fedora Update System --- golang-googlecode-gcfg-0-0.4.gitc2d3050.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/golang-googlecode-gcfg-0-0.4.gitc2d3050.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1141880] Review Request: golang-googlecode-gcfg - Gcfg reads INI-style configuration files into Go structs
https://bugzilla.redhat.com/show_bug.cgi?id=1141880 --- Comment #15 from Fedora Update System --- golang-googlecode-gcfg-0-0.4.gitc2d3050.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/golang-googlecode-gcfg-0-0.4.gitc2d3050.el6 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1141880] Review Request: golang-googlecode-gcfg - Gcfg reads INI-style configuration files into Go structs
https://bugzilla.redhat.com/show_bug.cgi?id=1141880 --- Comment #14 from Fedora Update System --- golang-googlecode-gcfg-0-0.4.gitc2d3050.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/golang-googlecode-gcfg-0-0.4.gitc2d3050.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 --- Comment #6 from Vojtech Trefny --- Fixed source (using GitHub releases): https://vtrefny.fedorapeople.org/srpms/blivet-gui-0.2.0-6.fc21.src.rpm https://vtrefny.fedorapeople.org/spec/blivet-gui.spec -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 --- Comment #7 from Jaroslav Škarvada --- (In reply to Jaroslav Škarvada from comment #3) > - Directories /usr/share/help and /usr/share/help/C are not owned, but this > is something you cannot resolve. AFAIK /usr/share/help is not in FHS > standard. This should be provided by some package. Question is which > package, gtk? gtk-doc? Any opinion? Bug should be filled against the > identified package to include these directories. FYI initially filed bug against yelp (bug 1184954), let's see what happen :) At least it's tracked now. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 Jaroslav Škarvada changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #8 from Jaroslav Škarvada --- (In reply to Vojtech Trefny from comment #6) > Fixed source (using GitHub releases): > > https://vtrefny.fedorapeople.org/srpms/blivet-gui-0.2.0-6.fc21.src.rpm > https://vtrefny.fedorapeople.org/spec/blivet-gui.spec Thanks, one note: - please do not use Release tag in upstream release, just name the upstream release: %{name}-%{version}.tar.gz, i.e. blivet-gui-0.2.0.tar.gz, the Release tag is Fedora specific, it's there to distinguish updates related to packaging, rebuilds, etc. This is usptream thing and not blocking the review, thus approving. You are now sponsored, welcome aboard :) Please continue from step 7 in: http://fedoraproject.org/wiki/New_package_process_for_existing_contributors -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184040] Review Request: blivet-gui - Tool for data storage configuration
https://bugzilla.redhat.com/show_bug.cgi?id=1184040 Jaroslav Škarvada changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180723] Review Request: clufter - Tool for transforming/analyzing cluster configuration formats
https://bugzilla.redhat.com/show_bug.cgi?id=1180723 --- Comment #13 from Mamoru TASAKA --- One additional note - Placing bash completion files under %_datadir (currently) means that we no longer treat this as %config file. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #21 from Ralf Corsepius --- Yet another update: Spec URL: http://corsepiu.fedorapeople.org/packages/rt.spec SRPM URL: http://corsepiu.fedorapeople.org/packages/rt-4.0.22-2.fc22.src.rpm Changes/Remarks: - The rt package now requires: /etc/http.d/conf.d (The same approach as eg. applied to perl-HTML-Mason) - The running testsuite now only exposes 1 failure (Less than any other rt package did before!) - The SELinux issue with rt-4.0.22/mysql now seems to be resolved (Cf. README.fedora[.in] and RHBZ#1184792) - Many small packaging changes. Most visible: "rpmbuild --with pg" doesn't not require "--without mysql" anymore (as it used to do). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #22 from Ralf Corsepius --- Created attachment 982893 --> https://bugzilla.redhat.com/attachment.cgi?id=982893&action=edit Log from running rt-4.0.22-2.f21's testsuites -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904634] Review Request: mingw-sword - MinGW build of a cross-platform scripture text library
https://bugzilla.redhat.com/show_bug.cgi?id=904634 Michael Cronenworth changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|m...@cchtml.com Flags||fedora-review? --- Comment #7 from Michael Cronenworth --- Taking for review. Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "GPL", "LGPL (v2.1 or later) (with incorrect FSF address)", "GPL (v2 or later)", "Unknown or generated", "LGPL (v2 or later) (with incorrect FSF address)", "zlib/libpng", "GPL (unversioned/unknown version) GPL". 78 files have unknown license. Detailed output of licensecheck in /media/michael/Temp/backup/Projects/fedora/review/904634-mingw- sword/licensecheck.txt [-]: License file installed when any subpackage combination is installed. [!]: Package requires other packages for directories it uses. Note: No known owner of /usr/x86_64-w64-mingw32/sys- root/mingw/share/sword, /usr/i686-w64-mingw32/sys-root/mingw/share/sword [!]: Package must own all directories that it creates. Note: Directories without known owners: /usr/i686-w64-mingw32/sys- root/mingw/share/sword, /usr/x86_64-w64-mingw32/sys- root/mingw/share/sword [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [-]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Package is not known to require an ExcludeArch tag. [-]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 163840 bytes in 8 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x]: Package does not own files or directories owned by other packages. [x]: All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package do not use a name that already exist [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local = SHOULD items = Generic: [x]: Uses parallel make %{?_smp_mflags} macro. [-]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: Final provides and requires are sane (see attachments). [-]: Fully versioned dependency in subpackages if applicable. Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in mingw32-sword , mingw64-sword [x]: Package functions as described. [!]: Latest version is packaged. [x]: Package does not include lic
[Bug 1184996] New: Review Request: rubygem-rails-html-sanitizer - This gem is responsible to sanitize HTML fragments in Rails applications
https://bugzilla.redhat.com/show_bug.cgi?id=1184996 Bug ID: 1184996 Summary: Review Request: rubygem-rails-html-sanitizer - This gem is responsible to sanitize HTML fragments in Rails applications Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: vondr...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/cgit/vondruch/public_git/rubygem-rails-html-sanitizer.git/tree/rubygem-rails-html-sanitizer.spec?id=94dd4e7c329114c700bb5236e9b14584835e998c SRPM URL: http://people.redhat.com/vondruch/rubygem-rails-html-sanitizer-1.0.1-1.fc22.src.rpm Description: HTML sanitization to Rails applications. Fedora Account System Username: vondruch -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184996] Review Request: rubygem-rails-html-sanitizer - This gem is responsible to sanitize HTML fragments in Rails applications
https://bugzilla.redhat.com/show_bug.cgi?id=1184996 Vít Ondruch changed: What|Removed |Added Blocks||1184910 Depends On||1184920 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184910 [Bug 1184910] Ruby on Rails 4.2 https://bugzilla.redhat.com/show_bug.cgi?id=1184920 [Bug 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri
https://bugzilla.redhat.com/show_bug.cgi?id=1184920 Vít Ondruch changed: What|Removed |Added Blocks||1184996 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1184996 [Bug 1184996] Review Request: rubygem-rails-html-sanitizer - This gem is responsible to sanitize HTML fragments in Rails applications -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1180723] Review Request: clufter - Tool for transforming/analyzing cluster configuration formats
https://bugzilla.redhat.com/show_bug.cgi?id=1180723 --- Comment #14 from Jan Pokorný --- > Your srpm does not build Oops, sorry, accidentally I've started COPR build with full-fledged spec (which passed), the subsequent build was skipped (same NVR/SRPM URL), but it was at the time I was working on a fixed version already ([comment 11]), and I hit another problem with COPR build later on [3]. I was hoping to move forward faster (and hence provide updated SRPM sooner) but still at massaging that. > URL format with %version I am using a custom file-naming scheme for pre-releases (well, following what 'python setup.py sdist' produces, and this is not pre-release specific; mostly because of clashes with RPM versioning) and the original meta-specfile is accustomed to that. I'll consider making it play better for standard releases, exactly as you suggest. Do I understand it correctly that another suggestion is to use "%version" instead of "%{version}"? > %{?_isa} specific depdendency on noarch packages Ah, %{?_isa} is to be used only within transitive closure of arch-specific packages only (provided that the dependency itself is not noarch)! Just for the clarity: the only arch-specific subpackage is python-clufter as it builds and uses a private binary (ccs_flatten). > %post It was intended to allow for dynamic regeneration of Bash completion file upon installing/removing plugins (e.g., clufter-lib-pcs that offers a new command, which should be reflected in the completion file). Currently this hasn't been tested, I have to look at that. Creating beforehand is sort of an optimization: any computation that can be precomputed (memoized) should be done early, once for all. > completion file Thanks, I have no idea how could I find this out except for studying spec of bash-completion! It's a shame that even Packaging Guidelines page shows off /etc/bash_completion.d [4]. Re: second point, good catch, I'll will make the package own that dir itself (bash-completion is completely optional). Re: %config, that's good, it is not a configuration file, really (should not be backed up, etc.) > %doc Thanks again for teaching me new stuff. > License (GFDL) Well, this is embarrassing :) > %files Thanks. > %files -n usage with having %{name}- prefix Result of meta-specfile arrangement, will think about that. > usage of "|| :" Yes, I learnt this from some authoritative docs, and four characters overhead is negligible enough to put up with that, IMHO. > ${ret} I am aware of "set -e", but this is more robust (no macro will accidentally bring "set +e", etc.) > Remove macro-in-comment rpmlint Ah, this is a result of meta-specfile simplification process (original contains "%%"), will fix it. [3] https://copr-fe.cloud.fedoraproject.org/coprs/jpokorny/clufter/build/68529/ [4] https://fedoraproject.org/w/index.php?title=Packaging:Guidelines&rd=Packaging%2FGuidelines#The_directory_is_wholly_contained_in_your_package.2C_or_involves_core_functionality_of_your_package -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1178173] Review Request: gnome-calendar - Simple and beautiful calendar application designed to fit GNOME 3
https://bugzilla.redhat.com/show_bug.cgi?id=1178173 --- Comment #4 from Florian "der-flo" Lehner --- hi Igor! The package still fails to build because of libical-devel >= 1.0.1. Cheers, flo For the records: http://koji.fedoraproject.org/koji/taskinfo?taskID=8702624 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185002] New: Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 Bug ID: 1185002 Summary: Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5 Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: mario.blaetterm...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://mariobl.fedorapeople.org/Review/SPECS/appmenu-qt5.spec SRPM URL: https://mariobl.fedorapeople.org/Review/SRPMS/appmenu-qt5-0.3.0-r26.1.fc21.src.rpm Description: This is a different, Qt5-compatible approach of the existing appmenu-qt (https://launchpad.net/appmenu-qt). appmenu-qt5 is a Qt5 QPA theme plugin that adds support for application menus to Qt5 applications. This only works for Qt5 versions >= 5.2 currently. To enable the support, set QT_QPA_PLATFORMTHEME=appmenu-qt5 in your environment. Fedora Account System Username: mariobl I get this from rpmlint: appmenu-qt5.spec:40: W: rpm-buildroot-usage %build make %{?_smp_mflags} INSTALL_ROOT=$RPM_BUILD_ROOT $RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it may break short circuit builds. But I don't know another solution to make the install directory known to qmake-qt5. Moreover, I'm not sure about versioning. Ubuntu uses the same generated tarball for the package version 0.3.0 although I can't find that number anywhere in the source files. As far as I can evaluate, they use it only for distinction between the existing Qt4 version and the Qt5 package. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185002] Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 Mario Blättermann changed: What|Removed |Added Blocks||928937 (qt-reviews) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=928937 [Bug 928937] Qt-related package review tracker -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1178911] Review Request: cairo-dock - Light eye-candy fully themable animated dock
https://bugzilla.redhat.com/show_bug.cgi?id=1178911 --- Comment #15 from Jan Pokorný --- (After-the-fact) suggestion: - %defattr(-,root,root,-) should not be needed as it is a default (similar to %doc/[1180723 comment 12]). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1178911] Review Request: cairo-dock - Light eye-candy fully themable animated dock
https://bugzilla.redhat.com/show_bug.cgi?id=1178911 --- Comment #16 from Jan Pokorný --- ([bug 1180723 comment 12]) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185002] Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 --- Comment #1 from Mario Blättermann --- Some hint about the script in %{_sysconfdir}/profile.d/: This script activates the Qt module globally. Actually it does the same as described in README. But this is eventually not desired for systems with multiple users. It could happen that the menu bar of Qt5 applications doesn't appear on desktops without plasma-widget-menubar. Maybe it would be better to drop CONFIG+=enable-by-default from the spec file to disable installing the script...? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1135470] Review Request: libkscreen-qt5 - KDE display configuration library
https://bugzilla.redhat.com/show_bug.cgi?id=1135470 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1135470] Review Request: libkscreen-qt5 - KDE display configuration library
https://bugzilla.redhat.com/show_bug.cgi?id=1135470 --- Comment #10 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 --- Comment #3 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #4 from Michael Adam --- ok, I fixed it. following up soon. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183001] Review Request: kdecoration - Plugin-based library to create window decorations
https://bugzilla.redhat.com/show_bug.cgi?id=1183001 --- Comment #5 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183001] Review Request: kdecoration - Plugin-based library to create window decorations
https://bugzilla.redhat.com/show_bug.cgi?id=1183001 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #5 from Michael Adam --- Spec URL: https://fedorapeople.org/~obnox/rpms/vagrant-libvirt/0.0.23-4/vagrant-libvirt.spec SRPM URL: https://fedorapeople.org/~obnox/rpms/vagrant-libvirt/0.0.23-4/vagrant-libvirt-0.0.23-4.fc22.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #6 from Michael Adam --- The code can be found on git://fedorapeople.org/~obnox/fedora-vagrant-libvirt.git -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185014] New: Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations
https://bugzilla.redhat.com/show_bug.cgi?id=1185014 Bug ID: 1185014 Summary: Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-atlasrep/gap-pkg-atlasrep.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-atlasrep/gap-pkg-atlasrep-1.5.0-1.fc22.src.rpm Fedora Account System Username: jjames Description: The aim of the AtlasRep package is to provide an interface between GAP and the Atlas of Group Representations, a database that comprises representations of many almost simple groups and information about their maximal subgroups. This database is available independent of GAP. The AtlasRep package consists of this database and a GAP interface. The latter allows the user to get an overview of the database, and to access the data in GAP format. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185015] New: Review Request: gap-pkg-browse - GAP browser for 2-dimensional arrays of data
https://bugzilla.redhat.com/show_bug.cgi?id=1185015 Bug ID: 1185015 Summary: Review Request: gap-pkg-browse - GAP browser for 2-dimensional arrays of data Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-browse/gap-pkg-browse.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-browse/gap-pkg-browse-1.8.6-1.fc22.src.rpm Fedora Account System Username: jjames Description: This is a rename review. The original review is bug 869797. The motivation for this package was to develop functions for an interactive display of two-dimensional arrays of data, for example character tables. They should be displayed with labeled rows and columns, the display should allow some markup for fonts or colors, it should be possible to search for entries, to sort rows or columns, to hide and unhide information, to bind commands to keys, and so on. To achieve this our package now provides three levels of functionality, where in particular the first level may also be used for completely other types of applications: - A low level interface to ncurses: This may be interesting for all kinds of applications which want to display text with some markup and colors, maybe in several windows, using the available capabilities of a terminal. - A medium level interface to a generic function NCurses.BrowseGeneric: We introduce a new operation Browse which is meant as an interactive version of Display for GAP objects. Then we provide a generic function for browsing two-dimensional arrays of data, handles labels for rows and columns, searching, sorting, binding keys to actions, etc. This is for users who want to implement new methods for browsing two-dimensional data. - Applications of these interfaces: We provide some applications of the ncurses interface and of the function NCurses.BrowseGeneric. These may be interesting for end users, and also as examples for programmers of further applications. This includes a method for browsing character tables, several games, and an interface for demos. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185016] New: Review Request: gap-pkg-io - Unix I/O functionality for GAP
https://bugzilla.redhat.com/show_bug.cgi?id=1185016 Bug ID: 1185016 Summary: Review Request: gap-pkg-io - Unix I/O functionality for GAP Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-io/gap-pkg-io.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-io/gap-pkg-io-4.4.4-1.fc22.src.rpm Fedora Account System Username: jjames Description: This is a rename review. The original review is bug 858287. This GAP package provides a link to the standard UNIX I/O functionality that is available through the C library. This part basically consists of functions on the GAP level that allow functions in the C library to be called. Built on top of this is a layer for buffered input/output which is implemented completely in the GAP language. It is intended to be used by programs for which it is not necessary to have full direct access to the operating system. On this level, quite a few convenience functions are implemented for interprocess communication like starting up pipelines of processes to filter data through them and to start up processes and then communicate with them. There is also support for creating network connections over TCP/IP and UDP. Building on this, the package contains an implementation of the client side of the HTTP protocol making it possible among other things to access web pages from within GAP. Another part of the package is a framework for object serialization. That is, GAP objects can be converted into a platform-independent byte sequence which can be stored to a file or sent over the network. The code takes complete care of arbitrarily self-referential data structures like lists containing themselves as an entry. The resulting byte strings can be read back into GAP and the original objects are rebuilt with exactly the same self-references. This works for most of the standard builtin types of GAP like numbers, permutations, polynomials, lists, and records and can be extended to nearly arbitrary GAP objects. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185017] New: Review Request: gap-pkg-polymaking - GAP interface to polymake
https://bugzilla.redhat.com/show_bug.cgi?id=1185017 Bug ID: 1185017 Summary: Review Request: gap-pkg-polymaking - GAP interface to polymake Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-polymaking/gap-pkg-polymaking.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-polymaking/gap-pkg-polymaking-0.8.1-1.fc22.src.rpm Fedora Account System Username: jjames Description: This package provides a very basic GAP interface to polymake. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185018] New: Review Request: gap-pkg-sonata - GAP package for systems of nearrings
https://bugzilla.redhat.com/show_bug.cgi?id=1185018 Bug ID: 1185018 Summary: Review Request: gap-pkg-sonata - GAP package for systems of nearrings Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-sonata/gap-pkg-sonata.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-sonata/gap-pkg-sonata-2.6-4.fc22.src.rpm Fedora Account System Username: jjames Description: This is a rename review. The original review is bug 858081. SONATA stands for "systems of nearrings and their applications". It provides methods for the construction and the analysis of finite nearrings. A left nearring is an algebra (N;+,*), where (N,+) is a (not necessarily abelian) group, (N,*) is a semigroup, and x*(y+z) = x*y + x*z holds for all x,y,z in N. As a typical example of a nearring, we may consider the set of all mappings from a group G into G, where the addition is the pointwise addition of mappings in G, and the multiplication is composition of functions. If functions are written on the right of their arguments, then the left distributive law holds, while the right distributive law is not satisfied for non-trivial G. The SONATA package provides methods for the construction and analysis of finite nearrings. 1. Methods for constructing all endomorphisms and all fixed-point-free automorphisms of a given group. 2. Methods for constructing the following nearrings of functions on a group G: - the nearring of polynomial functions of G (in the sense of Lausch-Nöbauer); - the nearring of compatible functions of G; - distributively generated nearrings such as I(G), A(G), E(G); - centralizer nearrings. 3. A library of all small nearrings (up to order 15) and all small nearrings with identity (up to order 31). 4. Functions to obtain solvable fixed-point-free automorphism groups on abelian groups, nearfields, planar nearrings, as well as designs from those. 5. Various functions to study the structure (size, ideals, N-groups, ...) of nearrings, to determine properties of nearring elements, and to decide whether two nearrings are isomorphic. 6. If the package XGAP is installed, the lattices of one- and two-sided ideals of a nearring can be studied interactively using a graphical representation. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185019] New: Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 Bug ID: 1185019 Summary: Review Request: gap-pkg-tomlib - GAP Table of Marks package Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-tomlib/gap-pkg-tomlib.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-tomlib/gap-pkg-tomlib-1.2.5-1.fc22.src.rpm Fedora Account System Username: jjames Description: This is a rename review. The original review is bug 769450. This package provides access to several hundred tables of marks of almost simple groups and their maximal subgroups. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185021] New: Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups
https://bugzilla.redhat.com/show_bug.cgi?id=1185021 Bug ID: 1185021 Summary: Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-spinsym/gap-pkg-spinsym.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-spinsym/gap-pkg-spinsym-1.5-1.fc22.src.rpm Fedora Account System Username: jjames Description: This package contains some p-modular character tables of Schur covers of symmetric and alternating groups. It also provides some more functionalities related to these groups, for example, a method to construct character tables of their maximal Young subgroups. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185020] New: Review Request: gap-pkg-ctbllib - GAP Character Table Library
https://bugzilla.redhat.com/show_bug.cgi?id=1185020 Bug ID: 1185020 Summary: Review Request: gap-pkg-ctbllib - GAP Character Table Library Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: loganje...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jjames.fedorapeople.org/gap-pkg-ctbllib/gap-pkg-ctbllib.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-ctbllib/gap-pkg-ctbllib-1.2.2-1.fc22.src.rpm Fedora Account System Username: jjames Description: This is a rename review. The original review is bug 785903. This package provides the Character Table Library by Thomas Breuer. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184600] Review Request: python-sep -Astronomical source extraction and photometry in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1184600 Florian "der-flo" Lehner changed: What|Removed |Added Status|NEW |ASSIGNED CC||d...@der-flo.net Assignee|nob...@fedoraproject.org|d...@der-flo.net Flags||fedora-review+ --- Comment #1 from Florian "der-flo" Lehner --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [x]: Package does not contain kernel modules. [x]: Package contains no static executables. [x]: Development (unversioned) .so files in -devel subpackage, if present. Note: Unversioned so-files in private %_libdir subdirectory (see attachment). Verify they are not in ld path. [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "LGPL (v3 or later)", "Unknown or generated". 22 files have unknown license. [x]: License file installed when any subpackage combination is installed. [x]: If the package is under multiple licenses, the licensing breakdown must be documented in the spec. [x]: %build honors applicable compiler flags or justifies otherwise. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [x]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Useful -debuginfo package or justification otherwise. [x]: Package is not known to require an ExcludeArch tag. [x]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 61440 bytes in 10 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x]: Package requires other packages for directories it uses. [x]: Package must own all directories that it creates. [x]: Package does not own files or directories owned by other packages. [x]: All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package do not use a name that already exist [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local Python: [x]: Python eggs must not download any dependencies during the build process. [x]: A package which is used by another package via an egg interface should provide egg info. [x]: Package meets the Packaging Guidelines::Python [x]: Package contains BR: python2-devel or python3-devel [x]: Binary eggs must be removed in %prep = SHOULD items = Generic: [-]: If the source package does not include license text(s) as a separate file from u
[Bug 1185002] Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 Rex Dieter changed: What|Removed |Added CC||rdie...@math.unl.edu Alias||appmenu-qt5 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185002] Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 --- Comment #2 from Rex Dieter --- Good point, for now I think it safer to provide a sample script as %doc and include a README.fedora to document how a user could include it in their environment. Once appmenu-qt5 has formal releases and/or it is better tested, then enabling by default would be ok. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #7 from Michael Adam --- And I did an update to 0.0.24: Spec URL: https://fedorapeople.org/~obnox/rpms/vagrant-libvirt/0.0.24-1/vagrant-libvirt.spec SRPM URL: https://fedorapeople.org/~obnox/rpms/vagrant-libvirt/0.0.24-1/vagrant-libvirt-0.0.24-1.fc22.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1177926] Review Request: smuxi - Powerful, flexible, user-friendly chat client
https://bugzilla.redhat.com/show_bug.cgi?id=1177926 --- Comment #16 from Antonio Trande --- (In reply to Mirco Bauer from comment #15) > The current package split is problematic. It doesn't allow a console > frontend only install without installing X11/GNOME packages. Also a > smuxi-server only install without X11/GNOME packages is not possible. > > Please re-evaluate my recommended package split: > > * smuxi-engine > * smuxi-engine* libraries (the engine has plugins thus the wildcard is > important) > * bin/smuxi-server > * smuxi-server.exe* > * man1/smuxi-server.1 > * all other private libraries (SmartIrcr4net, Twitterizer, ServiceStack, > etc) > > * smuxi-frontend-gnome > * bin/smuxi-frontend-gnome > * smuxi-frontend-gnome* (this frontend has plugins thus the wildcard is > important) > * man1/smuxi-frontend-gnome.1 > > * smuxi-frontend-stfl > * bin/smuxi-frontend-stfl > * smuxi-frontend-stfl.exe* > * man1/smuxi-frontend-stfl.1 > > This the smallest possible split that still allows to install a) server-only > without X11/GUI dependencies, b) just the GNOME frontend c) just the console > frontend (without X11/GUI) Is it okay for you? --> https://www.diffchecker.com/5mm6aq3b I wish keep smuxi, smuxi-engine, smuxi-stfl packages. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184381] Review Request: meson - High productivity build system
https://bugzilla.redhat.com/show_bug.cgi?id=1184381 Florian "der-flo" Lehner changed: What|Removed |Added Status|NEW |ASSIGNED CC||d...@der-flo.net Assignee|nob...@fedoraproject.org|d...@der-flo.net Flags||fedora-review? --- Comment #1 from Florian "der-flo" Lehner --- Hi Igor! There are three errors from rpmlint: Checking: meson-0.22.0-1.fc22.x86_64.rpm meson-0.22.0-1.fc22.src.rpm meson.x86_64: E: no-binary meson.x86_64: E: non-executable-script /usr/share/meson/mparser.py 0644L /usr/bin/python3 meson.x86_64: E: non-executable-script /usr/share/meson/dependencies.py 0644L /usr/bin/env 2 packages and 0 specfiles checked; 3 errors, 0 warnings. Please see https://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_Python_libraries for more. Cheers, Flo -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184381] Review Request: meson - High productivity build system
https://bugzilla.redhat.com/show_bug.cgi?id=1184381 --- Comment #2 from Igor Gnatenko --- > meson.x86_64: E: no-binary don't know why it's happening. i have 3 binaries in /usr/bin/ New SPEC: https://ignatenkobrain.fedorapeople.org/for-review/meson.spec New SRPM: https://ignatenkobrain.fedorapeople.org/for-review/meson-0.22.0-2.fc22.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1178173] Review Request: gnome-calendar - Simple and beautiful calendar application designed to fit GNOME 3
https://bugzilla.redhat.com/show_bug.cgi?id=1178173 --- Comment #5 from Igor Gnatenko --- (In reply to Florian "der-flo" Lehner from comment #4) > hi Igor! Hi, > The package still fails to build because of libical-devel >= 1.0.1. Depends On: Bug 1178069 ;) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
https://bugzilla.redhat.com/show_bug.cgi?id=672246 --- Comment #20 from Fedora Update System --- perl-Statistics-Basic-1.6611-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-Statistics-Basic-1.6611-3.el6 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185002] Review Request: appmenu-qt5 - Support for global DBus-exported application menu in Qt5
https://bugzilla.redhat.com/show_bug.cgi?id=1185002 --- Comment #3 from Mario Blättermann --- (In reply to Rex Dieter from comment #2) > Good point, for now I think it safer to provide a sample script as %doc and > include a README.fedora to document how a user could include it in their > environment. > OK, I will do that. The appropriate config option in KDE 4 doesn't affect the behavior of Qt5 apps, I was wondering if it would work in KDE Frameworks 5? > Once appmenu-qt5 has formal releases and/or it is better tested, then > enabling by default would be ok. I would it never enable by default. Imagine, it is the default on a multiuser system. This is only OK if all users work with Plasma (currently v4 only) and have plasma-widget-menubar enabled. In all other cases Qt5 apps remain without menubar, because the module will be loaded at system startup. Correct me if I'm wrong. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185014] Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations
https://bugzilla.redhat.com/show_bug.cgi?id=1185014 Sandro Mani changed: What|Removed |Added Status|NEW |ASSIGNED CC||manisan...@gmail.com Docs Contact||manisan...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185014] Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations
https://bugzilla.redhat.com/show_bug.cgi?id=1185014 Sandro Mani changed: What|Removed |Added Assignee|nob...@fedoraproject.org|manisan...@gmail.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185014] Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations
https://bugzilla.redhat.com/show_bug.cgi?id=1185014 Sandro Mani changed: What|Removed |Added Docs Contact|manisan...@gmail.com| -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 David Dick changed: What|Removed |Added Depends On|1184809 |495179 (perl-Number-Format) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=495179 [Bug 495179] Review Request: perl-Number-Format - Perl extension for formatting numbers https://bugzilla.redhat.com/show_bug.cgi?id=1184809 [Bug 1184809] Request EPEL6 and EPEL7 branchs -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review