[Bug 1186572] Review Request: rubygem-rmagick - Ruby binding to ImageMagick
https://bugzilla.redhat.com/show_bug.cgi?id=1186572 --- Comment #6 from Mamoru TASAKA --- Thank you! New Package SCM Request === Package Name: rubygem-rmagick Short Description: Ruby binding to ImageMagick Upstream URL: https://github.com/gemhome/rmagick Owners: mtasaka Branches: f21 f20 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 1186572] Review Request: rubygem-rmagick - Ruby binding to ImageMagick
https://bugzilla.redhat.com/show_bug.cgi?id=1186572 Mamoru TASAKA changed: What|Removed |Added Flags||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 1171260] Review Request: rubygem-jmespath - JMESPath - Ruby Edition
https://bugzilla.redhat.com/show_bug.cgi?id=1171260 Mamoru TASAKA changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Mamoru TASAKA --- Okay. * spec file / srpm clean * can be built cleanly * generated binary clean * can be installed * at least the following works $ ruby -e "require 'jmespath' ; puts JMESPath::VERSION" 1.0.2 --- This package (rubygem-jmespath) is APPROVED by mtasaka --- -- 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] Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups
https://bugzilla.redhat.com/show_bug.cgi?id=1185021 Jerry James changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #2 from Jerry James --- Thank you so much for the reviews! If you need something reviewed in exchange, please let me know. I will make the changes above when I import this 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 1185020] Review Request: gap-pkg-ctbllib - GAP Character Table Library
https://bugzilla.redhat.com/show_bug.cgi?id=1185020 --- Comment #2 from Jerry James --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #1) > %{_smp_mflags} would be nice here too. Yes, I will make that change. > Hm, package fails to build locally, I get a gap> prompt. Maybe there should > be some redirect Also, is the license specified upstream? It is in doc/manual.pdf, on page 2, under the Copyright declaration. Here is the version with %{_smp_mflags}: 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-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 1177486] Review Request: golang-github-glacjay-goini - INI file parser in go
https://bugzilla.redhat.com/show_bug.cgi?id=1177486 Fedora Update System changed: What|Removed |Added Fixed In Version|golang-github-glacjay-goini |golang-github-glacjay-goini |-0-0.1.git5261baf.fc20 |-0-0.1.git5261baf.fc21 --- Comment #13 from Fedora Update System --- golang-github-glacjay-goini-0-0.1.git5261baf.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1182960] Review Request: perl-Apache-Session-Browseable - Add index and search methods to Apache::Session
https://bugzilla.redhat.com/show_bug.cgi?id=1182960 --- Comment #10 from Fedora Update System --- perl-Apache-Session-Browseable-1.0.2-2.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1186755] Review Request: dap2rpm - Tool for generating RPM specfiles for DevAssistant DAP packages
https://bugzilla.redhat.com/show_bug.cgi?id=1186755 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- dap2rpm-0.1.7-2.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1151369] Review Request: perl-Crypt-URandom - Non-blocking randomness for Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1151369 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-Crypt-URandom-0.34-1.f |perl-Crypt-URandom-0.34-1.f |c21 |c20 --- Comment #14 from Fedora Update System --- perl-Crypt-URandom-0.34-1.fc20 has been pushed to the Fedora 20 stable repository. -- 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 1186889] Review Request: geoipupdate - decouple update utility from GeoIP as per upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1186889 --- Comment #9 from Philip Prindeville --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #8) > It'd help if you uploaded the new version :) Sorry, thought I had. Just double-checked and it is 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 855192] Review Request: felix-fileinstall - A utility to automatically install bundles from a directory
https://bugzilla.redhat.com/show_bug.cgi?id=855192 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System --- felix-fileinstall-3.4.0-3.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1178861] Review Request: mapdb - Java database engine
https://bugzilla.redhat.com/show_bug.cgi?id=1178861 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||mapdb-1.0.6-1.fc21 Resolution|--- |ERRATA Last Closed||2015-01-29 23:42:44 --- Comment #9 from Fedora Update System --- mapdb-1.0.6-1.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1186346] Review Request: golang-github-influxdb-hyperleveldb-go - Go wrapper for HyperLevelDB
https://bugzilla.redhat.com/show_bug.cgi?id=1186346 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- golang-github-influxdb-hyperleveldb-go-0-0.1.gite24de94.fc20 has been pushed to the Fedora 20 testing repository. -- 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 974756] Review Request: flatpack - Flat File Parser via XML Mappings (fixed length,CSV)
https://bugzilla.redhat.com/show_bug.cgi?id=974756 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System --- flatpack-4.0.0-2.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1178934] Review Request: perl-Exporter-Tidy - Another way of exporting symbols
https://bugzilla.redhat.com/show_bug.cgi?id=1178934 --- Comment #9 from Fedora Update System --- perl-Exporter-Tidy-0.08-1.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1149212] Review Request: springframework-plugin - Simple plugin infrastructure
https://bugzilla.redhat.com/show_bug.cgi?id=1149212 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System --- springframework-plugin-1.1.0-2.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1179355] Review Request: kite - Kite SDK to simplify the development of data-related systems
https://bugzilla.redhat.com/show_bug.cgi?id=1179355 Bug 1179355 depends on bug 1179342, which changed state. Bug 1179342 Summary: Review Request: ua-parser-java - Java implementation of user agent parser https://bugzilla.redhat.com/show_bug.cgi?id=1179342 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 1175293] Review Request: publicsuffix-list - Cross-vendor public domain suffix database
https://bugzilla.redhat.com/show_bug.cgi?id=1175293 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||publicsuffix-list-20141230- ||1.fc21 Resolution|--- |ERRATA Last Closed||2015-01-29 23:38:18 --- Comment #14 from Fedora Update System --- publicsuffix-list-20141230-1.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1179342] Review Request: ua-parser-java - Java implementation of user agent parser
https://bugzilla.redhat.com/show_bug.cgi?id=1179342 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ua-parser-java-1.3.0-1.fc21 Resolution|--- |ERRATA Last Closed||2015-01-29 23:38:00 --- Comment #6 from Fedora Update System --- ua-parser-java-1.3.0-1.fc21 has been pushed to the Fedora 21 stable repository. -- 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 866273] Review Request: jaxb2-fluent-api - JAXB2 Commons - Fluent API Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=866273 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- jaxb2-fluent-api-3.0-1.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1181197] Review Request: golang-github-skynetservices-skydns - DNS service discovery for etcd
https://bugzilla.redhat.com/show_bug.cgi?id=1181197 Fedora Update System changed: What|Removed |Added Fixed In Version|golang-github-skynetservice |golang-github-skynetservice |s-skydns-0-0.3.git245a121.f |s-skydns-0-0.3.git245a121.f |c20 |c21 --- Comment #12 from Fedora Update System --- golang-github-skynetservices-skydns-0-0.3.git245a121.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report. -- 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 1173683] Review Request: pg-semver - A semantic version data type for PostgreSQL
https://bugzilla.redhat.com/show_bug.cgi?id=1173683 Fedora Update System changed: What|Removed |Added Fixed In Version|pg-semver-0.5.0-2.fc21 |pg-semver-0.5.0-2.fc20 --- Comment #13 from Fedora Update System --- pg-semver-0.5.0-2.fc20 has been pushed to the Fedora 20 stable repository. -- 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 1182175] Review Request: python-testing.postgresql - Automatically setup a PostgreSQL testing instance
https://bugzilla.redhat.com/show_bug.cgi?id=1182175 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- python-testing.postgresql-1.1.0-2.fc20 has been pushed to the Fedora 20 testing repository. -- 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 1182960] Review Request: perl-Apache-Session-Browseable - Add index and search methods to Apache::Session
https://bugzilla.redhat.com/show_bug.cgi?id=1182960 --- Comment #9 from Fedora Update System --- perl-Apache-Session-Browseable-1.0.2-2.fc20 has been pushed to the Fedora 20 stable repository. -- 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 1177487] Review Request: golang-github-rakyll-globalconf - Effortlessly persist and read flag values from an ini config file
https://bugzilla.redhat.com/show_bug.cgi?id=1177487 Bug 1177487 depends on bug 1177486, which changed state. Bug 1177486 Summary: Review Request: golang-github-glacjay-goini - INI file parser in go https://bugzilla.redhat.com/show_bug.cgi?id=1177486 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 1177486] Review Request: golang-github-glacjay-goini - INI file parser in go
https://bugzilla.redhat.com/show_bug.cgi?id=1177486 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-glacjay-goini ||-0-0.1.git5261baf.fc20 Resolution|--- |ERRATA Last Closed||2015-01-29 23:35:43 --- Comment #12 from Fedora Update System --- golang-github-glacjay-goini-0-0.1.git5261baf.fc20 has been pushed to the Fedora 20 stable repository. -- 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 1175471] Review Request: fleet - A distributed init system
https://bugzilla.redhat.com/show_bug.cgi?id=1175471 Bug 1175471 depends on bug 1177486, which changed state. Bug 1177486 Summary: Review Request: golang-github-glacjay-goini - INI file parser in go https://bugzilla.redhat.com/show_bug.cgi?id=1177486 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 1177487] Review Request: golang-github-rakyll-globalconf - Effortlessly persist and read flag values from an ini config file
https://bugzilla.redhat.com/show_bug.cgi?id=1177487 Fedora Update System changed: What|Removed |Added Fixed In Version|golang-github-rakyll-global |golang-github-rakyll-global |conf-0-0.1.git415abc3.fc20 |conf-0-0.1.git415abc3.fc21 --- Comment #9 from Fedora Update System --- golang-github-rakyll-globalconf-0-0.1.git415abc3.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1175471] Review Request: fleet - A distributed init system
https://bugzilla.redhat.com/show_bug.cgi?id=1175471 Bug 1175471 depends on bug 1177487, which changed state. Bug 1177487 Summary: Review Request: golang-github-rakyll-globalconf - Effortlessly persist and read flag values from an ini config file https://bugzilla.redhat.com/show_bug.cgi?id=1177487 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 1177487] Review Request: golang-github-rakyll-globalconf - Effortlessly persist and read flag values from an ini config file
https://bugzilla.redhat.com/show_bug.cgi?id=1177487 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-rakyll-global ||conf-0-0.1.git415abc3.fc20 Resolution|--- |ERRATA Last Closed||2015-01-29 23:34:30 --- Comment #8 from Fedora Update System --- golang-github-rakyll-globalconf-0-0.1.git415abc3.fc20 has been pushed to the Fedora 20 stable repository. -- 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 1181197] Review Request: golang-github-skynetservices-skydns - DNS service discovery for etcd
https://bugzilla.redhat.com/show_bug.cgi?id=1181197 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-skynetservice ||s-skydns-0-0.3.git245a121.f ||c20 Resolution|--- |ERRATA Last Closed||2015-01-29 23:34:43 --- Comment #11 from Fedora Update System --- golang-github-skynetservices-skydns-0-0.3.git245a121.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report. -- 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 977017] Review Request: dozer - Java Bean to Java Bean mapper
https://bugzilla.redhat.com/show_bug.cgi?id=977017 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System --- dozer-5.5.1-1.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1186345] Review Request: golang-github-influxdb-rocksdb - Go wrapper for RocksDB
https://bugzilla.redhat.com/show_bug.cgi?id=1186345 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- golang-github-influxdb-rocksdb-0-0.1.git7adff3e.fc20 has been pushed to the Fedora 20 testing repository. -- 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 1185582] Review Request: openambit - Open software for the Suunto Ambit(2)
https://bugzilla.redhat.com/show_bug.cgi?id=1185582 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System --- openambit-0.3-2.git5f2b784.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1185509] Review Request: python-photutils - Astropy affiliated package for image photometry tasks
https://bugzilla.redhat.com/show_bug.cgi?id=1185509 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System --- python-photutils-0.1-3.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1181194] Review Request: scamp - compute astrometric and photometric solutions
https://bugzilla.redhat.com/show_bug.cgi?id=1181194 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System --- scamp-2.0.4-1.fc20 has been pushed to the Fedora 20 testing repository. -- 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 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #18 from Fedora Update System --- mingw-sword-1.7.4-1.fc21 has been pushed to the Fedora 21 testing repository. -- 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 1173683] Review Request: pg-semver - A semantic version data type for PostgreSQL
https://bugzilla.redhat.com/show_bug.cgi?id=1173683 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pg-semver-0.5.0-2.fc21 Resolution|--- |ERRATA Last Closed||2015-01-29 23:29:47 --- Comment #12 from Fedora Update System --- pg-semver-0.5.0-2.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1151369] Review Request: perl-Crypt-URandom - Non-blocking randomness for Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1151369 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-Crypt-URandom-0.34-1.e |perl-Crypt-URandom-0.34-1.f |l7 |c21 --- Comment #13 from Fedora Update System --- perl-Crypt-URandom-0.34-1.fc21 has been pushed to the Fedora 21 stable repository. -- 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 1142407] Review Request: drpm - deltarpm manipulation library
https://bugzilla.redhat.com/show_bug.cgi?id=1142407 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #24 from Fedora Update System --- drpm-0.1.2-1.fc21 has been pushed to the Fedora 21 testing repository. -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #10 from Zbigniew Jędrzejewski-Szmek --- License is OK, package builds in mock, sources are OK, name seems OK, right flags are used. The use of /usr/lib/gap is explained above, debuginfo package is OK, Provides and Obsoletes are correct. rpmlint: gap-pkg-tomlib.noarch: W: only-non-binary-in-usr-lib 2 packages and 0 specfiles checked; 0 errors, 1 warnings. OK. Package is 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 1185019] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 Jerry James changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #9 from Jerry James --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #7) > It's recommended to use %{_smp_mflags}. This allows the number of threads to > be overridden if necessary: > > parallel %{?_smp_mflags} --no-notice gzip -1 ::: data/*.tom Ah, good solution. I will make that change. > Is the license specified anywhere upstream? A few versions ago, there was nothing about the license anywhere in the distribution. I wrote to Liam Naughton, one of the upstream developers, as follows: "Hello Liam, I am working on adding GAP and some of its packages to the Fedora Linux distribution. However, I don't see any indication of the license under which TomLib is distributed. Did you intend it to have the same license as GAP itself, namely GPL version 2 or any later version? If so, would you mind adding a statement about the license to the TomLib web site or, even better, to the tar file itself?" He responded as follows: "Hi Jerry, I've added a copyright notice to the tomlib homepage and to the manual. We adopt the same copyright notice as GAP. All the best Liam" And indeed, you can now see the sentence "We adopt the copyright regulations of GAP as detailed in the copyright notice in the GAP manual." on http://schmidt.nuigalway.ie/tomlib/ and in several files in doc/. The manual is included in the gap-online-help package, or you can view the copyright section online here: http://www.gap-system.org/Manuals/doc/ref/chap1.html#X7950EFA183E3F666 where you can see that it indicates GPLv2+ as the license. That is much more roundabout than I would prefer, but that's the story. (In reply to Zbigniew Jędrzejewski-Szmek from comment #8) > One more question: why /usr/lib/gap/pkg and not /usr/share/something? Is the > place upstream expects? Yes, it is. Even though most GAP add-on packages are noarch, some (such as gap-pkg-io and gap-pkg-browse) have binary components. Therefore, /usr/share seems inappropriate for those packages. However, they can't be multiarch, since they depend on the gap binary, so there is no need to put them in %{_libdir} either. Hence, they go into /usr/lib, thereby allowing the noarch packages to remain noarch. Here is the version using %{?_smp_mflags}: 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-3.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 1182261] Review Request: libabigail - Tool for constructing, manipulating, serializing and de-serializing ABI-relevant artifacts
https://bugzilla.redhat.com/show_bug.cgi?id=1182261 --- Comment #23 from Parag AN(पराग) --- Sinny, Also, Can you submit few more packages also? That way we can know more about your packaging work and how well you are understanding packaging guidelines. -- 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] Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups
https://bugzilla.redhat.com/show_bug.cgi?id=1185021 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Flags|fedora-review? |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 1185021] Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups
https://bugzilla.redhat.com/show_bug.cgi?id=1185021 --- Comment #1 from Zbigniew Jędrzejewski-Szmek --- Please change find . -name ._\* | xargs rm -f to find . -name ._\* -delete It's short and not vulnerable to spaces and bash special characters in the name. /usr/lib/gap/pkg/spinsym/COPYING should be %excluded. rpmlint: gap-pkg-spinsym.noarch: E: explicit-lib-dependency gap-pkg-ctbllib It's fine. The dependency is not detected automatically. gap-pkg-spinsym.noarch: W: spelling-error %description -l en_US functionalities -> functionalists, functionality, functionalist gap-pkg-spinsym.noarch: W: only-non-binary-in-usr-lib gap-pkg-spinsym.src: W: spelling-error %description -l en_US functionalities -> functionalists, functionality, functionalist 2 packages and 0 specfiles checked; 1 errors, 3 warnings. All OK. Package is trivial, license is OK and specified upstream, I see no issues. Package is 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 1185021] Review Request: gap-pkg-spinsym - GAP package for Brauer tables of spin-symmetric groups
https://bugzilla.redhat.com/show_bug.cgi?id=1185021 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added CC||zbys...@in.waw.pl Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl 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 1185020] Review Request: gap-pkg-ctbllib - GAP Character Table Library
https://bugzilla.redhat.com/show_bug.cgi?id=1185020 --- Comment #1 from Zbigniew Jędrzejewski-Szmek --- %{_smp_mflags} would be nice here too. Hm, package fails to build locally, I get a gap> prompt. Maybe there should be some redirect https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1185019] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #8 from Zbigniew Jędrzejewski-Szmek --- One more question: why /usr/lib/gap/pkg and not /usr/share/something? Is the place upstream expects? -- 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] Review Request: gap-pkg-ctbllib - GAP Character Table Library
https://bugzilla.redhat.com/show_bug.cgi?id=1185020 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added CC||zbys...@in.waw.pl Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl -- 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 1186889] Review Request: geoipupdate - decouple update utility from GeoIP as per upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1186889 --- Comment #8 from Zbigniew Jędrzejewski-Szmek --- (In reply to Philip Prindeville from comment #7) > (In reply to Paul Howarth from comment #6) > > (In reply to Philip Prindeville from comment #5) > > > If you've any intention of updating EPEL to GeoIP 1.6.x then this is a must > > for those > EPEL branches, as GeoIP would need to depend on it (this was > > split off from GeoIP at > 1.6). > > That was my thinking. OK. > > > Actually, that was cut & pasted verbatim from website's description of the > > > package. I figured it was simplest to keep the description in the > > > authors' > > > words. > > > > It's probably not worth keeping anything not relevant to Fedora/EPEL users. > > Fixed. It'd help if you uploaded the new version :) -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #7 from Zbigniew Jędrzejewski-Szmek --- It's recommended to use %{_smp_mflags}. This allows the number of threads to be overridden if necessary: parallel %{?_smp_mflags} --no-notice gzip -1 ::: data/*.tom Is the license specified anywhere upstream? -- 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 672956] Review Request: python26-lxml - ElementTree-like Python bindings for libxml2 and libxslt
https://bugzilla.redhat.com/show_bug.cgi?id=672956 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added CC||zbys...@in.waw.pl --- Comment #5 from Zbigniew Jędrzejewski-Szmek --- _smp_mflags macro should be used for the number of threads. This makes it possible to override the number if necessary: parallel %{?_smp_mflags} --no-notice gzip -1 ::: data/*.tom The spec file says license is GPLv3+, is it specified anywhere? -- 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 1122777] Review Request: tcpcrypt - Opportunistically encrypt TCP connections
https://bugzilla.redhat.com/show_bug.cgi?id=1122777 Paul Wouters changed: What|Removed |Added Flags||fedora-cvs? --- Comment #8 from Paul Wouters --- New Package SCM Request === Package Name: tcpcrypt Short Description: Opportunistically encrypt TCP connections Upstream URL: http://tcpcrypt.org/ Owners: pwouters Branches: f21 el6 epel7 InitialCC: pwouters -- 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 1122777] Review Request: tcpcrypt - Opportunistically encrypt TCP connections
https://bugzilla.redhat.com/show_bug.cgi?id=1122777 Patrick Uiterwijk changed: What|Removed |Added Status|ASSIGNED|POST Flags||fedora-review+ --- Comment #7 from Patrick Uiterwijk --- Okay, after your last changes and the upstream report of macro usage, this package is 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 1171129] Review Request: freeradius-client - Client library and utilities for radius
https://bugzilla.redhat.com/show_bug.cgi?id=1171129 Nikos Mavrogiannopoulos changed: What|Removed |Added Flags||fedora-cvs? --- Comment #21 from Nikos Mavrogiannopoulos --- Wrong copy paste. Here is the correct. New Package SCM Request === Package Name: freeradius-client Short Description: Client library for radius Owners: nmav Branches: f21 epel7 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 1186572] Review Request: rubygem-rmagick - Ruby binding to ImageMagick
https://bugzilla.redhat.com/show_bug.cgi?id=1186572 František Dvořák changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from František Dvořák --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated = MUST items = C/C++: [x]: Package does not contain kernel modules. [x]: Package contains no static executables. [-]: Development (unversioned) .so files in -devel subpackage, if present. Note: Unversioned so-files in private %_libdir subdirectory (see attachment). 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: "MIT/X11 (BSD like)", "Unknown or generated". 252 files have unknown license. /home/valtri/fedora2-scm/REVIEWS/rubygem-rmagick/1186572-rubygem- rmagick/licensecheck.txt [x]: License file installed when any subpackage combination is installed. [x]: Package must own all directories that it creates. [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. [-]: 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. [x]: 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]: 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]: Package requires other packages for directories it uses. [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]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 0 bytes in 0 files. [x]: Packages must not store files under /srv, /opt or /usr/local Ruby: [x]: Platform dependent files must all go under %{gem_extdir_mri}, platform independent under %{gem_dir}. [x]: Gem package must not define a non-gem subpackage [x]: Macro %{gem_extdir} is deprecated. [x]: Gem package is named rubygem-%{gem_name} [x]: Package contains BuildRequires: rubygems-devel. [x]: Gem package must define %{gem_name} macro. [x]: Package does not contain Requires: ruby(abi). = SHOULD items = Generic: [x]: Avoid bundling fonts in non-fonts packages. Note: Package contains font files. It's in generated docs. [x]: 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). [x]: Fully versioned dependency in subpackages if applicable. [x]: Package functions as described. [x]: Latest version is packaged. [x]: Packag
[Bug 1185685] Review Request: vagrant-lxc - LXC-provider for vagrant
https://bugzilla.redhat.com/show_bug.cgi?id=1185685 --- Comment #3 from Michael Adam --- Cleanups and fixes found by Günther in review: Spec Url: https://obnox.fedorapeople.org/rpms/vagrant-lxc/1.1.0/6.fc22/vagrant-lxc.spec SRPM Url: https://obnox.fedorapeople.org/rpms/vagrant-lxc/1.1.0/6.fc22/vagrant-lxc-1.1.0-6.fc22.src.rpm Copr is updated. -- 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 1122777] Review Request: tcpcrypt - Opportunistically encrypt TCP connections
https://bugzilla.redhat.com/show_bug.cgi?id=1122777 --- Comment #6 from Patrick Uiterwijk --- - Source files are very weird: the size and checksum between the srpm tarball and upstream tarball differ, but diff -r shows no differences. - Your useradd uses -g tcpcrypt, which misses the trailing "d". - Package still runs rm -rf %{buildroot} at the beginning of %install. - Package still uses obsolete m4 macros: AC_PROG_LIBTOOL found in: tcpcrypt-bb990b1bfb0e411f0613abdaf3b71fdce50a82cf/configure.ac:55 - Rpmlint on Installed: tcpcrypt-libs.x86_64: W: undefined-non-weak-symbol /usr/lib64/libtcpcrypt.so.0.0.0 socket_address_is_null tcpcrypt-libs.x86_64: W: undefined-non-weak-symbol /usr/lib64/libtcpcrypt.so.0.0.0 socket_address_clear tcpcrypt-libs.x86_64: W: undefined-non-weak-symbol /usr/lib64/libtcpcrypt.so.0.0.0 resolve_socket_address_local - Rpmlint on built: tcpcrypt-devel.x86_64: W: only-non-binary-in-usr-lib tcpcrypt-devel.x86_64: W: no-documentation tcpcrypt.src: W: strange-permission tcpcryptd-firewall 0755L -- 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 1171260] Review Request: rubygem-jmespath - JMESPath - Ruby Edition
https://bugzilla.redhat.com/show_bug.cgi?id=1171260 --- Comment #3 from František Dvořák --- (In reply to Mamoru TASAKA from comment #2) > For 1.0.2-1: > > Well, very simple package. Now a few questions. > > * BR: rubygem(simplecov) > - Is this BR needed? (any coverage tests?) > Yes, it looks like simplecov is not really needed, although it needs to remove some 4 lines: sed -i spec/compliance_spec.rb spec/spec_helper.rb -e '/simplecov\|SimpleCov/d' > * Tweaking multi_json dependency > - does not seem to be working. Please check this. > (and is this needed?) > Nice catch! Removed. I misread the '~> 1.0' string. > * %setup - unpacking %{SOURCE1} > > %setup -q -D -T -n %{gem_name}-%{version} > tar -xzf %{SOURCE1} > > - "%setup -q -D -T -n %{gem_name}-%{version} -a 1" is more smart. OK, '-a 1' will be better. New version: Spec URL: http://scientific.zcu.cz/fedora/rubygem-jmespath-1.0.2-2/rubygem-jmespath.spec SRPM URL: http://scientific.zcu.cz/fedora/rubygem-jmespath-1.0.2-2/rubygem-jmespath-1.0.2-2.fc22.src.rpm Thu Jan 29 2015 František Dvořák - 1.0.2-2 - Removed rubygem(simplecov) BR - Cleanups -- 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 969763] Review Request: taudem - Terrain Analysis Using Digital Elevation Models
https://bugzilla.redhat.com/show_bug.cgi?id=969763 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org --- Comment #3 from Thomas Spura --- (In reply to Volker Fröhlich from comment #0) > I'm not totally sure about the sub-package layout and the MPI naming suffix. > There's no serial version and only one supported MPI implementation. - Hmm, why only one supported MPI implementation? You should be able to build against openmpi also. - mpich2 is named mpich now IIRC (You could take mpi4py or cp2k or so as example for MPI building packages.) - The sources moved and there is a version 5.1.2 around. Could you update it to it? - A quick smoke test doesn't build and fails with: CMake Error at CMakeLists.txt:65 (add_executable): Cannot find source file: shape/cell.cpp -- 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 923671] Review Request: tex-prerex-gui - Graphical front-end and viewer for the prerex editor
https://bugzilla.redhat.com/show_bug.cgi?id=923671 Thomas Spura changed: What|Removed |Added Depends On||923669 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=923669 [Bug 923669] Review Request: tex-prerex-bin - Interactive editor for prerequisite charts -- 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 923669] Review Request: tex-prerex-bin - Interactive editor for prerequisite charts
https://bugzilla.redhat.com/show_bug.cgi?id=923669 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org Blocks||923671 --- Comment #2 from Thomas Spura --- The link to the source seems to have changed. This seems to work with the new version: http://mirrors.ctan.org/graphics/prerex/doc/prerex-%version.tar.gz The buildroot seems odd. Do you still need it? IIRC, we had a different set of allowed buildroot tags such as %{_tmppath}/%{name}-%{version}-%{release} and not _topdir. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=923671 [Bug 923671] Review Request: tex-prerex-gui - Graphical front-end and viewer for the prerex editor -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #6 from Jerry James --- Okay, I like the use of GNU parallel. However, I don't like the magic number 12. What about dropping the -j option entirely and letting it use whatever number of CPUs are available? Like this: 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-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 657071] Review Request: fuse-ext2 - Ext2/3/4 file-system support for FUSE
https://bugzilla.redhat.com/show_bug.cgi?id=657071 Thomas Spura changed: What|Removed |Added CC||dwro...@ertelnet.rybnik.pl, ||toms...@fedoraproject.org Flags||needinfo?(dwrobel@ertelnet. ||rybnik.pl) --- Comment #10 from Thomas Spura --- Any news to the bundling issue in the last comment? -- 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 1122777] Review Request: tcpcrypt - Opportunistically encrypt TCP connections
https://bugzilla.redhat.com/show_bug.cgi?id=1122777 --- Comment #5 from Paul Wouters --- Spec URL: ftp://ftp.nohats.ca/tcpcrypt/tcpcrypt.spec SRPM URL: ftp://ftp.nohats.ca/tcpcrypt/tcpcrypt-0.4-0.2.bb990b1b.fc19.src.rpm Description: Opportunistically encrypt TCP connections Fedora Account System Username: pwouters -- 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 1160496] Review Request: flask-dropbox - package for add flask at dropbox
https://bugzilla.redhat.com/show_bug.cgi?id=1160496 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org Blocks||177841 (FE-NEEDSPONSOR) --- Comment #9 from Thomas Spura --- If I see it correctly, this is your first package and you are looking for a sponsor. Is that right? If so, you can find more information on that over here: http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group Did you try to install and use your package? Because of the wrong Requires above, you shouldn't be able to install it... Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 Jon Ciesla changed: What|Removed |Added Flags|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 1171129] Review Request: freeradius-client - Client library and utilities for radius
https://bugzilla.redhat.com/show_bug.cgi?id=1171129 --- Comment #20 from Jon Ciesla --- WARNING: Requested package name ocserv doesn't match bug summary freeradius-client Please correct. -- 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 Brett Lentz changed: What|Removed |Added Flags||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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 Jon Ciesla changed: What|Removed |Added Flags|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 1172732] Review Request: fedora-productimg-atomic - Installer branding and configuration for Fedora Atomic
https://bugzilla.redhat.com/show_bug.cgi?id=1172732 Colin Walters changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Colin Walters --- Thanks for the review! I'll include the license. -- 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 672956] Review Request: python26-lxml - ElementTree-like Python bindings for libxml2 and libxslt
https://bugzilla.redhat.com/show_bug.cgi?id=672956 Jason Tibbitts changed: What|Removed |Added Status|NEW |CLOSED Blocks||201449 (FE-DEADREVIEW) Resolution|--- |NOTABUG Last Closed||2015-01-29 15:47:21 --- Comment #4 from Jason Tibbitts --- This has been stalled for so long I can't see why it would still be open, and the comment above doesn't appear to have any relevance to this package. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug. -- 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 --- Comment #19 from Brett Lentz --- Gah. Sorry. Copy/Paste error. Package Change Request == Package Name: rubygem-aws-sdk New Branches: epel7 Owners: blentz goldmann -- 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 --- Comment #18 from Jon Ciesla --- Invalid package name. -- 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 1186725] Review Request: perl-Apache-Session-NoSQL - NoSQL implementation of Apache::Session
https://bugzilla.redhat.com/show_bug.cgi?id=1186725 --- Comment #2 from Paul Howarth --- (In reply to David Dick from comment #1) > License is correct > > Package builds in rawhide > http://koji.fedoraproject.org/koji/taskinfo?taskID=8773114 > > rpmlint is clean > > perl(base) should be put in as a BuildRequire for all three packages > > you should put in the Change, README and examples doc files. Fixed in -2: Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Apache-Session-NoSQL/branches/fedora/perl-Apache-Session-NoSQL.spec SRPM URL: http://www.city-fan.org/~paul/extras/perl-Apache-Session-NoSQL/perl-Apache-Session-NoSQL-0.1-2.fc22.src.rpm > isn't it worth building Net-Cassandra in rawhide at least? I'd be delighted if someone did (and once it was in Rawhide, it could probably go to F-20, F-21 and EPEL too). I'm just not sure that I'm the right person to maintain it as I've no interest in it at all personally, and it's a big package that might need a fair bit of maintenance down the line. -- 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 1148224] Review Request: mmdb2 - protein coordinate library
https://bugzilla.redhat.com/show_bug.cgi?id=1148224 --- Comment #25 from Mukundan Ragavan --- Tim, looks like my understanding on the Obsoletes/Provides was at least incomplete ... Comments about the 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 1166947] Review Request: perl-Dancer2 - Lightweight yet powerful web application framework
https://bugzilla.redhat.com/show_bug.cgi?id=1166947 --- Comment #7 from Fedora Update System --- perl-Dancer2-0.158000-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/perl-Dancer2-0.158000-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 1166947] Review Request: perl-Dancer2 - Lightweight yet powerful web application framework
https://bugzilla.redhat.com/show_bug.cgi?id=1166947 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 1186889] Review Request: geoipupdate - decouple update utility from GeoIP as per upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1186889 --- Comment #7 from Philip Prindeville --- (In reply to Paul Howarth from comment #6) > (In reply to Philip Prindeville from comment #5) > If you've any intention of updating EPEL to GeoIP 1.6.x then this is a must > for those > EPEL branches, as GeoIP would need to depend on it (this was > split off from GeoIP at > 1.6). That was my thinking. > > Actually, that was cut & pasted verbatim from website's description of the > > package. I figured it was simplest to keep the description in the authors' > > words. > > It's probably not worth keeping anything not relevant to Fedora/EPEL users. Fixed. -- 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 1166947] Review Request: perl-Dancer2 - Lightweight yet powerful web application framework
https://bugzilla.redhat.com/show_bug.cgi?id=1166947 --- Comment #6 from Fedora Update System --- perl-Dancer2-0.158000-1.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/perl-Dancer2-0.158000-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 1119063] Review Request: python-service-identity - Service identity verification for pyOpenSSL
https://bugzilla.redhat.com/show_bug.cgi?id=1119063 Matěj Cepl changed: What|Removed |Added Status|NEW |ASSIGNED CC||mc...@redhat.com --- Comment #10 from Matěj Cepl --- Should be ASSIGNED to nobody jumps on it (like me). -- 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 1186889] Review Request: geoipupdate - decouple update utility from GeoIP as per upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1186889 --- Comment #6 from Paul Howarth --- (In reply to Philip Prindeville from comment #5) > (In reply to Zbigniew Jędrzejewski-Szmek from comment #4) > > This in not EPEL material, you can drop the compat cruft: > > > > # Noarch subpackages available from Fedora 10, RHEL 6 > > %global noarch_subpkgs 0%{?fedora} > 9 || 0%{?rhel} > 5 > > > > rm -rf %{buildroot} > > I need to remove the 'rm' as well? > > And why not get this into EPEL? A lot of CentOS users run mail servers with > SpamAssassin, MIMEDefang, and perl-Geo-IP... If you've any intention of updating EPEL to GeoIP 1.6.x then this is a must for those EPEL branches, as GeoIP would need to depend on it (this was split off from GeoIP at 1.6). > > There's no need to talk about non-Linux (or even non-Fedora) in the package > > description. Remove "Currently the program only supports Linux and > > other Unix- like systems." > > Actually, that was cut & pasted verbatim from website's description of the > package. I figured it was simplest to keep the description in the authors' > words. It's probably not worth keeping anything not relevant to Fedora/EPEL users. Hopefully I'll have time to take a look at this tomorrow. -- 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 1186889] Review Request: geoipupdate - decouple update utility from GeoIP as per upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1186889 --- Comment #5 from Philip Prindeville --- (In reply to Zbigniew Jędrzejewski-Szmek from comment #4) > This in not EPEL material, you can drop the compat cruft: > > # Noarch subpackages available from Fedora 10, RHEL 6 > %global noarch_subpkgs 0%{?fedora} > 9 || 0%{?rhel} > 5 > > rm -rf %{buildroot} I need to remove the 'rm' as well? And why not get this into EPEL? A lot of CentOS users run mail servers with SpamAssassin, MIMEDefang, and perl-Geo-IP... > Something strange happens with the provides: > Provides: %{name}-cron = %{version}-%{release} > Obsoletes: %{name}-cron < %{version}-%{release} > This is implied, no need to add it. Fixed. > There's no need to talk about non-Linux (or even non-Fedora) in the package > description. Remove "Currently the program only supports Linux and > other Unix- like systems." Actually, that was cut & pasted verbatim from website's description of the package. I figured it was simplest to keep the description in the authors' words. > Also, you don't need to copy ${SOURCE1} twice. Just install it directly in > %install. Fixed. > Otherwise looks OK. Thanks. -- 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 1186725] Review Request: perl-Apache-Session-NoSQL - NoSQL implementation of Apache::Session
https://bugzilla.redhat.com/show_bug.cgi?id=1186725 --- Comment #1 from David Dick --- License is correct Package builds in rawhide http://koji.fedoraproject.org/koji/taskinfo?taskID=8773114 rpmlint is clean perl(base) should be put in as a BuildRequire for all three packages you should put in the Change, README and examples doc files. isn't it worth building Net-Cassandra in rawhide at least? if you wanted to build Net-Cassandra in the EPEL branches, for EPEL6 and 7 at least it looks like the only thing stopping you is perl(MooseX::StrictConstructor) which builds cleanly in EPEL6 and 7. http://koji.fedoraproject.org/koji/taskinfo?taskID=8773086 http://koji.fedoraproject.org/koji/taskinfo?taskID=8773093 -- 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 823227] Review Request: skype4py - Skype API wrapper for Python
https://bugzilla.redhat.com/show_bug.cgi?id=823227 Matěj Cepl changed: What|Removed |Added Status|NEW |CLOSED CC||mc...@redhat.com Blocks||182235 (FE-Legal) Resolution|--- |WONTFIX Last Closed||2015-01-29 15:00:33 --- Comment #4 from Matěj Cepl --- https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Packages_which_are_not_useful_without_external_bits Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal 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 1181321] Review Request: perl-URL-Encode-XS - XS implementation of URL::Encode
https://bugzilla.redhat.com/show_bug.cgi?id=1181321 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-URL-Encode-XS-0.03-1.e |perl-URL-Encode-XS-0.03-1.e |l7 |l6 --- Comment #15 from Fedora Update System --- perl-URL-Encode-XS-0.03-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- 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 1181321] Review Request: perl-URL-Encode-XS - XS implementation of URL::Encode
https://bugzilla.redhat.com/show_bug.cgi?id=1181321 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-URL-Encode-XS-0.03-1.f |perl-URL-Encode-XS-0.03-1.e |c21 |l7 --- Comment #14 from Fedora Update System --- perl-URL-Encode-XS-0.03-1.el7 has been pushed to the Fedora EPEL 7 stable repository. -- 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 1186389] Review Request: python-mako1.0 - Mako template library for Python
https://bugzilla.redhat.com/show_bug.cgi?id=1186389 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System --- python-mako1.0-1.0.1-2.el6 has been pushed to the Fedora EPEL 6 testing repository. -- 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 1183924] Review Request: perl-Time-TAI64 - Perl extension for converting TAI64 strings into standard unix timestamps
https://bugzilla.redhat.com/show_bug.cgi?id=1183924 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- perl-Time-TAI64-2.11-1.el6 has been pushed to the Fedora EPEL 6 testing repository. -- 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 Brett Lentz changed: What|Removed |Added CC||brett.le...@gmail.com Flags||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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=723472 Brett Lentz changed: What|Removed |Added CC||ble...@redhat.com --- Comment #17 from Brett Lentz --- Package Change Request == Package Name: pkgname New Branches: epel7 Owners: blentz goldmann -- 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 1186725] Review Request: perl-Apache-Session-NoSQL - NoSQL implementation of Apache::Session
https://bugzilla.redhat.com/show_bug.cgi?id=1186725 David Dick changed: What|Removed |Added Status|NEW |ASSIGNED CC||dd...@cpan.org Assignee|nob...@fedoraproject.org|dd...@cpan.org 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 1187337] New: Review Request: sil-coval-fonts - font derived from sans-serif
https://bugzilla.redhat.com/show_bug.cgi?id=1187337 Bug ID: 1187337 Summary: Review Request: sil-coval-fonts - font derived from sans-serif Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: empateinfin...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://empateinfinito.fedorapeople.org/font/sil-coval-fonts/sil-coval-fonts.spec SRPM URL: https://empateinfinito.fedorapeople.org/font/sil-coval-fonts/sil-coval-fonts-0.1-1.fc21.src.rpm Fedora-review: https://empateinfinito.fedorapeople.org/font/sil-coval-fonts/review.txt Description: Coval font is a derivated of sans serif with OFL license Fedora Account System Username: empateinfinito FE-NEEDSPONSOR -- 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 1177009] Review Request: python-tempest-lib - OpenStack Functional Testing Library
https://bugzilla.redhat.com/show_bug.cgi?id=1177009 --- Comment #14 from Steve Linabery --- Spec URL: https://slinabery.fedorapeople.org/python-tempest-lib.spec SRPM URL: https://slinabery.fedorapeople.org/python-tempest-lib-0.0.4-1.fc22.src.rpm Let's give this revision a try. -Moved README.rst back to noarch pkg. -rm the jquery.js file in %install -removed the Requires on noarch from doc -- 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 #18 from Jan Pokorný --- Mamoru, sorry for the lag. I was partly busy with an extension to the arrangement mentioned in [comment 10] (as a byproduct, there is now an isolated/clufter-agnostic project distill-spec [5]). I've tried to addresss your remarks and also made a few additional changes, e.g., there is now a filesystem in Requires for /usr/share/man/man1: https://people.redhat.com/jpokorny/pkgs/clufter/review/clufter-0.3.6-0.3.a_git.e88876e.fc21.src.rpm https://copr-fe.cloud.fedoraproject.org/coprs/jpokorny/clufter/build/69760/ Please note that the spec doesn't look entirely sleek now, especially with the repeated occurrences of "0.3.6a-git.e88876e". This come from pre-release versioning (its arrangement from the meta-specfile). Please ignore this for now and once you are otherwise happy, I will release a proper version and all such occurrences will turn into "%{version}" along with other simplifications that will the be possible at that point. You can then re-review such a final spec and give me a final ACK. [5] https://github.com/jnpkrn/distill-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 1185019] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #5 from Zbigniew Jędrzejewski-Szmek --- Actually: ls -1 data/*.tom | parallel --no-notice -j12 gzip --best -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #4 from Zbigniew Jędrzejewski-Szmek --- Actually parallel might be a better option: ls data/*.tom | parallel gzip --best -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #3 from Zbigniew Jędrzejewski-Szmek --- (In reply to Jerry James from comment #2) > Thank you for taking this review. > > (In reply to Zbigniew Jędrzejewski-Szmek from comment #1) > > 1. gzip --best is slow, and not very efficient. Would it be possible to > > switch to xz? > > Unfortunately, no. GAP itself can read gzipped files, but does not have the > capability of reading files compressed with anything else. I will have to > look at coding up xz support and sending it upstream. For now, though, gzip > is the only possibility. > > Since gzip is required, I prefer to use --best for the space savings. On my > 6 year old x86_64 machine, compressing the files with gzip takes slightly > over 1 minute. Spending 1 minute at build time to save space on every > user's machine seems like a good tradeoff to me. (Of course, the package > might be built on an ARM machine, which would require a greater amount of > time, but it still seems like a good tradeoff, in my opinion.) OK, that's a valid trade-off. > > 2. Compression should be parallized. Maybe add a Makefile.gzip as Source1 > > > > %.gz: % > > gzip $< > > > > and call make -f ${SOURCE1} $(for i in data/*.tom; echo $i.gz) _%{smp_flags} > > I don't see the point. The compression is done once at build time. The > package is noarch, so it only needs to be built on one architecture. It > only takes a minute (or so) to do. There isn't much to save by compressing > in parallel. And, on the other hand, compressing in parallel will use more > CPU and memory at one time, which may or may not be a problem, depending on > what other builds the boxing is doing at the same time. Guidelines are pretty clear: parallelized builds are required, unless the package cannot be built that way. But it's not a problem to do it here. -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #2 from Jerry James --- Thank you for taking this review. (In reply to Zbigniew Jędrzejewski-Szmek from comment #1) > 1. gzip --best is slow, and not very efficient. Would it be possible to > switch to xz? Unfortunately, no. GAP itself can read gzipped files, but does not have the capability of reading files compressed with anything else. I will have to look at coding up xz support and sending it upstream. For now, though, gzip is the only possibility. Since gzip is required, I prefer to use --best for the space savings. On my 6 year old x86_64 machine, compressing the files with gzip takes slightly over 1 minute. Spending 1 minute at build time to save space on every user's machine seems like a good tradeoff to me. (Of course, the package might be built on an ARM machine, which would require a greater amount of time, but it still seems like a good tradeoff, in my opinion.) > 2. Compression should be parallized. Maybe add a Makefile.gzip as Source1 > > %.gz: % > gzip $< > > and call make -f ${SOURCE1} $(for i in data/*.tom; echo $i.gz) _%{smp_flags} I don't see the point. The compression is done once at build time. The package is noarch, so it only needs to be built on one architecture. It only takes a minute (or so) to do. There isn't much to save by compressing in parallel. And, on the other hand, compressing in parallel will use more CPU and memory at one time, which may or may not be a problem, depending on what other builds the boxing is doing at the same time. -- 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] Review Request: gap-pkg-tomlib - GAP Table of Marks package
https://bugzilla.redhat.com/show_bug.cgi?id=1185019 --- Comment #1 from Zbigniew Jędrzejewski-Szmek --- Package is nice and clean. But compression could be improved. 1. gzip --best is slow, and not very efficient. Would it be possible to switch to xz? $ ls -1s tms13.tom* 35516 tms13.tom 8528 tms13.tom.gz(this is with --best, 12s) 8696 tms13.tom.gz(this is with default settings, 4s) 3976 tms13.tom.xz(this is with default settings, 31s) 5932 tms13.tom.xz(this is with -3, 7s) Switching to xz would give nice space savings. Using defaults for gzip or slightly reduced compression for xz would give nice time savings. 2. Compression should be parallized. Maybe add a Makefile.gzip as Source1 %.gz: % gzip $< and call make -f ${SOURCE1} $(for i in data/*.tom; echo $i.gz) _%{smp_flags} -- 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