[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #17 from Fedora Update System  ---
FEDORA-2019-3b88c02582 has been submitted as an update to Fedora 31.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-3b88c02582

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1744243] Review Request: wimlib - Open source Windows Imaging (WIM) library

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1744243



--- Comment #9 from Fedora Update System  ---
wimlib-1.13.1-1.fc29 has been pushed to the Fedora 29 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1744267] Review Request: python-fontconfig - Python bindings for Fontconfig library

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1744267



--- Comment #9 from Fedora Update System  ---
python-fontconfig-0.5.1-1.fc29 has been pushed to the Fedora 29 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1741833] Review Request: liberation-circuit - Real-time strategy game with programmable units

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1741833



--- Comment #10 from Fedora Update System  ---
liberation-circuit-1.3-2.20190824git29bc0ce0.fc29 has been pushed to the Fedora
29 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1745846] Review Request: f31-backgrounds - Fedora 31 default desktop background

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1745846

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #24 from Fedora Update System  ---
FEDORA-2019-8b2cccfabf has been submitted as an update to Fedora 31.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-8b2cccfabf

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1744243] Review Request: wimlib - Open source Windows Imaging (WIM) library

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1744243

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2019-09-04 03:11:34



--- Comment #8 from Fedora Update System  ---
wimlib-1.13.1-1.fc30 has been pushed to the Fedora 30 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1741833] Review Request: liberation-circuit - Real-time strategy game with programmable units

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1741833

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2019-09-04 03:11:41



--- Comment #9 from Fedora Update System  ---
liberation-circuit-1.3-2.20190824git29bc0ce0.fc30 has been pushed to the Fedora
30 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1744267] Review Request: python-fontconfig - Python bindings for Fontconfig library

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1744267

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2019-09-04 03:11:33



--- Comment #8 from Fedora Update System  ---
python-fontconfig-0.5.1-1.fc30 has been pushed to the Fedora 30 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748678] New: Review Request: usbtop - Utility to show USB bandwidth

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748678

Bug ID: 1748678
   Summary: Review Request: usbtop - Utility to show USB bandwidth
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: carl@george.computer
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01027810-usbtop/usbtop.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/reviews/fedora-rawhide-x86_64/01027810-usbtop/usbtop-1.0-1.fc32.src.rpm

Description:
usbtop is a top-like utility that shows an estimated instantaneous bandwidth on
USB buses and devices.

Fedora Account System Username: carlwgeorge

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747623] Review Request: golang-github-vultr-govultr - Vultr Go API client

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747623

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
FEDORA-2019-5ba2a2690d has been submitted as an update to Fedora 31.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-5ba2a2690d

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747623] Review Request: golang-github-vultr-govultr - Vultr Go API client

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747623



--- Comment #3 from Carl George  ---
rawhide: https://bodhi.fedoraproject.org/updates/FEDORA-2019-72fef5302f

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748637] Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748637

Jared Smith  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Jared Smith  ---
Package is APPROVED.  I think the package now only provides Python 3 bindings,
but the description in the spec file still specifies Python 2 and 3.

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Python packages must not contain %{pythonX_site(lib|arch)}/* in %files
  Note: Package contains %{pythonX_site(lib|arch)}/* in %files
  See: https://pagure.io/packaging-committee/issue/782


= 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: "Unknown or generated", "Expat License". 17 files have unknown
 license. Detailed output of licensecheck in
 /home/jsmith/Documents/Fedora/Reviews/1748637-pulsectl/licensecheck.txt
[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]: 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 20480 bytes in 2 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 %license.
[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]: 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]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package must not depend on deprecated() packages.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[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]: Packages MUST NOT have dependencies (either build-time or runtime) on
 packages named with the unversioned python- prefix unless no properly
 versioned package exists. Dependencies on Python packages instead MUST
 use names beginning with python2- or python3- as appropriate.
[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 upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: 

[Bug 1748637] Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748637

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748637] Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748637

Paul W. Frields  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #1 from Paul W. Frields  ---
Note for reviewer: I can't cleanly run %check on this package in mock/koji
because some tests require a local network connection to the pulseaudio server
to complete. However, in local non-mock builds, all tests complete OK as
expected.

No other automatic failures found in fedora-review, so I hope this should be an
easy one.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748637] New: Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748637

Bug ID: 1748637
   Summary: Review Request: pulsectl - Python high-level interface
and ctypes-based bindings for PulseAudio (libpulse)
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sticks...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://pfrields.fedorapeople.org/packages/SPECS/pulsectl.spec
SRPM URL:
https://pfrields.fedorapeople.org/packages/SRPMS/pulsectl-18.12.5-1.fc30.src.rpm
Description: Python 3.x high-level interface and ctypes-based bindings for
PulseAudio (libpulse), mostly focused on mixer-like controls and
introspection-related operations (as opposed to e.g. submitting sound samples
to play, player-like client).
Fedora Account System Username: pfrields

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748634] Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748634

Paul W. Frields  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2019-09-03 23:26:38



--- Comment #2 from Paul W. Frields  ---
Oops, filed with wrong account. 'Scuse the mess.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748634] Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748634

Paul W. Frields  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #1 from Paul W. Frields  ---
Note for reviewer: I can't cleanly run %check on this package in mock/koji
because some tests require a local network connection to the pulseaudio server
to complete. However, in local non-mock builds, all tests complete OK as
expected.

No other automatic failures found in fedora-review, so I hope this should be an
easy one.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748634] New: Review Request: pulsectl - Python high-level interface and ctypes-based bindings for PulseAudio (libpulse)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748634

Bug ID: 1748634
   Summary: Review Request: pulsectl - Python high-level interface
and ctypes-based bindings for PulseAudio (libpulse)
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pfrie...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://pfrields.fedorapeople.org/packages/SPECS/pulsectl.spec
SRPM URL:
https://pfrields.fedorapeople.org/packages/SRPMS/pulsectl-18.12.5-1.fc30.src.rpm
Description: Python 3.x high-level interface and ctypes-based bindings for
PulseAudio (libpulse), mostly focused on mixer-like controls and
introspection-related operations (as opposed to e.g. submitting sound samples
to play, player-like client).
Fedora Account System Username: pfrields

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530



--- Comment #5 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/erlang-yconf

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530



--- Comment #4 from Randy Barlow  ---
https://pagure.io/releng/fedora-scm-requests/issue/16230

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Randy Barlow  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|jer...@jcline.org



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1745846] Review Request: f31-backgrounds - Fedora 31 default desktop background

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1745846

Geoffrey Marr  changed:

   What|Removed |Added

 CC||gm...@redhat.com
 Whiteboard||AcceptedBlocker



--- Comment #23 from Geoffrey Marr  ---
Discussed during the 2019-09-03 blocker review meeting: [1]

The decision to classify this bug as an "AcceptedBlocker" was made as it
violates the following criteria:

"The default desktop background must be different from that of the last two
stable releases"

[1]
https://meetbot.fedoraproject.org/fedora-blocker-review/2019-09-03/f31-blocker-review.2019-09-03-16.01.txt

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530



--- Comment #3 from Randy Barlow  ---
https://pagure.io/releng/fedora-scm-requests/issue/16229

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Jeremy Cline  changed:

   What|Removed |Added

 CC||jer...@jcline.org
  Flags||fedora-review+



--- Comment #2 from Jeremy Cline  ---
The spec itself looks good and the license is correct so I'm going to go ahead
and approve this.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Randy Barlow  changed:

   What|Removed |Added

 Depends On||1748545




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1748545
[Bug 1748545] undefined function rebar:main/1
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Randy Barlow  changed:

   What|Removed |Added

 Blocks||1742538
 Depends On||1742455



--- Comment #1 from Randy Barlow  ---
There is a bit of a chicken-and-egg situation with this package. It requires a
newer version of fast_yaml than is in Rawhide, but I cannot update fast_yaml
until I also update ejabberd to the version that can use the new fast_yaml. I
cannot upgrade ejabberd to that version unless this package is in Rawhide.
Thus, I will need it to be reviewed even though it cannot be built on Rawhide
yet.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1742455
[Bug 1742455] erlang-fast_yaml-1.0.20 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1742538
[Bug 1742538] ejabberd-19.08 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748530] New: Review Request: erlang-yconf - YAML configuration processor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748530

Bug ID: 1748530
   Summary: Review Request: erlang-yconf - YAML configuration
processor
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rbar...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://bowlofeggs.fedorapeople.org/erlang-yconf.spec
SRPM URL: https://bowlofeggs.fedorapeople.org/erlang-yconf-1.0.0-1.fc32.src.rpm
Description: YAML configuration processor.
Fedora Account System Username: bowlofeggs

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1741805] Review Request: lolcat - a colorful version of 'cat'

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1741805



--- Comment #5 from josef radinger  ---
i think i have all your suggestions in place.

yours
josef

new versions:

Spec URL: http://www.nosuchhost.net/~cheese/temp/lolcat.spec
SRPM URL:
http://www.nosuchhost.net/~cheese/fedora/packages/30/SRPMS/lolcat-1.0-5.fc30.src.rpm

compiled version for f30 x86_64:
http://www.nosuchhost.net/~cheese/fedora/packages/30/x86_64/lolcat-1.0-5.fc30.x86_64.rpm
compiled version for f30 i386:  
http://www.nosuchhost.net/~cheese/fedora/packages/30/i386/lolcat-1.0-5.fc30.i686.rpm
 
compiled version for f29 x86_64:
http://www.nosuchhost.net/~cheese/fedora/packages/29/x86_64/lolcat-1.0-5.fc29.x86_64.rpm
compiled version for f29 i386:  
http://www.nosuchhost.net/~cheese/fedora/packages/29/i386/lolcat-1.0-5.fc29.i686.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747138] Review Request: perl-DBIx-Class-Helpers - A collection of various components for DBIx::Class

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747138

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2019-09-03 17:39:50



--- Comment #3 from Emmanuel Seyman  ---
This package is now in Rawhide and updates have been released for the stable
distributions. Thank you for the review, Jikta.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747142] Review Request: perl-Mojo-JWT - JSON Web Token the Mojo way

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747142

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2019-09-03 17:39:50



--- Comment #3 from Emmanuel Seyman  ---
This package is now in Rawhide and updates have been released for the stable
distributions. Thank you for the review, Jikta.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747012] Review Request: perl-Algorithm-BloomFilter - A simple bloom filter data structure

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747012

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2019-09-03 17:39:49



--- Comment #3 from Emmanuel Seyman  ---
This package is now in Rawhide and updates have been released for the stable
distributions. Thank you for the review, Jikta.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748471] New: xtensor-python - Python bindings for xtensor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748471

Bug ID: 1748471
   Summary: xtensor-python - Python bindings for xtensor
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sguel...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://sergesanspaille.fedorapeople.org/xtensor-python.spec
SRPM URL:
https://sergesanspaille.fedorapeople.org/xtensor-python-0.23.1-0.fc31.src.rpm
Description: xtensor-python enables inplace use of numpy arrays in C++ with all
the benefits from xtensor: C++ universal function and broadcasting,
STL-compliant APIs, A broad coverage of Numpy APIs.
Fedora Account System Username: sergesanspaille

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748470] New: xtensor - C++ tensors with broadcasting and lazy computing

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748470

Bug ID: 1748470
   Summary: xtensor - C++ tensors with broadcasting and lazy
computing
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sguel...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://sergesanspaille.fedorapeople.org/xtensor.spec
SRPM URL:
https://sergesanspaille.fedorapeople.org/xtensor-0.20.8-0.fc31.src.rpm
Description: xtensor is a C++ library meant for numerical analysis with
multi-dimensional array expressions. xtensor provides an extensible expression
system enabling lazy broadcasting, an API following the idioms of the C++
standard library, tools to manipulate array expressions and build upon xtensor.
Fedora Account System Username: sergesanspaille

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748469] New: xtl - Basic tools (containers, algorithms) used by xtensor-* packages

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748469

Bug ID: 1748469
   Summary: xtl - Basic tools (containers, algorithms) used by
xtensor-* packages
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sguel...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://sergesanspaille.fedorapeople.org/xtl.spec
SRPM URL: https://sergesanspaille.fedorapeople.org/xtl-0.6.5-1.fc31.src.rpm
Description: xtl provides implementation of C++ utilities like complex, span,
hash, optional, variant and similar classes used by xtensor-* packages.
Fedora Account System Username: sergesanspaille

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1726400] Review Request: python-text-unidecode - A Python module for handling non-Roman text data

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1726400

Fabian Affolter  changed:

   What|Removed |Added

  Flags|needinfo?(mail@fabian-affol |
   |ter.ch) |



--- Comment #7 from Fabian Affolter  ---
Well, here is the new files:

Spec URL:
https://fab.fedorapeople.org/packages/SRPMS/python-text-unidecode.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-text-unidecode-1.3-1.fc30.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1743767] Review Request: monitor-edid - simple tool for reading the EDID from a monitor

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1743767



--- Comment #4 from Pat Riehecky  ---
I've loaded a new build addressing the comments listed here:
https://copr.fedorainfracloud.org/coprs/jcpunk/ocsinventory-review/package/monitor-edid/

I'm working with upstream to determine what suitable manpages for the binaries
would look like.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1745846] Review Request: f31-backgrounds - Fedora 31 default desktop background

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1745846



--- Comment #22 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/f31-backgrounds

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748233] Review Request: pgloader - Data loading tool for PostgreSQL

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748233



--- Comment #4 from Lukas Javorsky  ---
Updated URLS:
Spec URL:
https://raw.githubusercontent.com/ljavorsk/pgloader/master/pgloader.spec
SRPM URL:
https://github.com/ljavorsk/pgloader/raw/master/pgloader-3.6.1-1.fc30.src.rpm

pgloader.x86_64: E: devel-dependency openssl-devel
- this dependency is needed as we spoke about it becasue of
/usr/lib64/libcrypto.so

pgloader.x86_64: E: wrong-script-interpreter
/usr/share/doc/pgloader/docs/conf.py /usr/bin/env python3
- fixed, don't include the docs/ directory at all

pgloader.x86_64: W: no-manual-page-for-binary pgloader
- python-sphinx generated man pages from docs/ directory


So it means, that it can't use it like this, and require every package there,
or package them if needed?

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747623] Review Request: golang-github-vultr-govultr - Vultr Go API client

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747623



--- Comment #2 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/golang-github-vultr-govultr

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748233] Review Request: pgloader - Data loading tool for PostgreSQL

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748233



--- Comment #3 from Miroslav Suchý  ---
Please always provide links to the raw version of files.

- If your application is a C or C++ application you must list a
  BuildRequires against gcc, gcc-c++ or clang.  
  Note: No gcc, gcc-c++ or clang found in BuildRequires
  See: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/

Please address these - we discussed that in person:
pgloader.x86_64: E: devel-dependency openssl-devel
pgloader.x86_64: E: wrong-script-interpreter
/usr/share/doc/pgloader/docs/conf.py /usr/bin/env python3
pgloader.x86_64: W: no-manual-page-for-binary pgloader


Hmm, I am really woried about those bundles in tarball - all those
pgloader-bundle-3.6.1/software/* Instead of that you have to use normal
libraries and packages in Fedora. And if they do not exists there yet, they
need to be packaged separately.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748233] Review Request: pgloader - Data loading tool for PostgreSQL

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748233

Miroslav Suchý  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||msu...@redhat.com
 Blocks||177841 (FE-NEEDSPONSOR)
   Assignee|nob...@fedoraproject.org|msu...@redhat.com
  Flags||fedora-review?



--- Comment #2 from Miroslav Suchý  ---
I will do this one.


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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1711611] Review Request: pdfarranger - PDF file merging, rearranging, and splitting. Maintained fork of pdfshuffler

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1711611



--- Comment #6 from David Auer  ---
I'd like to take Felix Schwarz's offer to take this package as a maintainer and
add me as co-maintainer afterwards since I haven't been able to take the steps
required for sponsorship yet.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748323] New: Review Request:rubygem-http-accept - Parse Accept and Accept-Language HTTP headers

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748323

Bug ID: 1748323
   Summary: Review Request:rubygem-http-accept - Parse Accept and
Accept-Language HTTP headers
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pval...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/pvalena/rubygems/fedora-rawhide-x86_64/01027219-rubygem-http-accept/rubygem-http-accept.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/pvalena/rubygems/fedora-rawhide-x86_64/01027219-rubygem-http-accept/rubygem-http-accept-2.1.0-1.fc32.src.rpm
Description: Parse Accept and Accept-Language HTTP headers
Fedora Account System Username: pvalena

Copr build:
https://copr.fedorainfracloud.org/coprs/pvalena/rubygems/build/1027219/
Koji scratch-build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=37431450

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762



--- Comment #16 from MUNEENDRA (Broadcom)  ---
This was the mistake which i did while creating the upstream name and the
original package name(fc_txptd).
It should be fctxpd.

Sorry for the mistake.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762



--- Comment #15 from MUNEENDRA (Broadcom)  ---
Is the missing second "t" on purpose?
>>Yes

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762

Fabio Valentini  changed:

   What|Removed |Added

 CC||decatho...@gmail.com



--- Comment #14 from Fabio Valentini  ---
Is the missing second "t" on purpose?

I mean fctxp(t)d. It's there in the upstream name and the original 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747152] Review Request: perl-MojoX-Log-Log4perl-Tiny - Minimalistic Log4perl adapter for Mojolicious

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747152

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #4 from Fedora Update System  ---
perl-MojoX-Log-Log4perl-Tiny-0.01-2.fc31 has been pushed to the Fedora 31
testing repository. If problems still persist, please make note of it in this
bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-08f45e1775

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762



--- Comment #13 from MUNEENDRA (Broadcom)  ---

THis is the change i did.

rename from fc_txptd.spec to fctxpd.spec
and in spec file i made the below change
-Name:   fc_txptd
+Name:   fctxpd
Version:0.1
Release:1.%{snapshotdate}git%{shortcommit}%{?dist}
Summary:Fibrechannel transport daemo

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762



--- Comment #12 from MUNEENDRA (Broadcom)  ---
Hi Robert,
Updated the spec and SRPM as the package name and the spec file does not match.

I have made the changes and  build the package for few distributions.
https://copr.fedorainfracloud.org/coprs/muneendra/fctxpd/build/1027382/

And below is the links for the updated Spec and SRPM URL

Spec URL:https://github.com/brocade/bsn-fc-txptd/blob/master/SPEC/fctxpd.spec
SRPM
URL:https://copr-be.cloud.fedoraproject.org/results/muneendra/fctxpd/epel-7-x86_64/01027382-fctxpd/fctxpd-0.1-1.20190813gitc195e67.el7.src.rpm


Regards,
Muneendra.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1735762] Review Request: fctxpd - This daemon adds FC network intelligence in host and host intelligence in FC network

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1735762

Petr Pisar  changed:

   What|Removed |Added

 CC||ppi...@redhat.com



--- Comment #11 from Petr Pisar  ---
The spec file name does not match the package name. Please pick up one and redo
the 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747152] Review Request: perl-MojoX-Log-Log4perl-Tiny - Minimalistic Log4perl adapter for Mojolicious

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747152

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



--- Comment #3 from Fedora Update System  ---
FEDORA-2019-08f45e1775 has been submitted as an update to Fedora 31.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-08f45e1775

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748259] Review Request: emacs-hcl-mode - Emacs major mode of HCL (Hashicorp Configuration Language)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748259

Mohamed El Morabity  changed:

   What|Removed |Added

 Blocks||1748262




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1748262
[Bug 1748262] Review Request: emacs-terraform-mode - Major mode of Terraform
configuration 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748262] Review Request: emacs-terraform-mode - Major mode of Terraform configuration file

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748262

Mohamed El Morabity  changed:

   What|Removed |Added

 Depends On||1748259




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1748259
[Bug 1748259] Review Request: emacs-hcl-mode - Emacs major mode of HCL
(Hashicorp Configuration Language)
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748262] New: Review Request: emacs-terraform-mode - Major mode of Terraform configuration file

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748262

Bug ID: 1748262
   Summary: Review Request: emacs-terraform-mode - Major mode of
Terraform configuration file
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pikachu.2...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://melmorabity.fedorapeople.org/packages/emacs-terraform-mode/emacs-terraform-mode.spec
SRPM URL:
https://melmorabity.fedorapeople.org/packages/emacs-terraform-mode/emacs-terraform-mode-0.06-1.fc30.src.rpm
Description: Major mode of terraform configuration file. terraform-mode
provides syntax highlighting, indentation function and formatting.
Fedora Account System Username: melmorabity

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748259] New: Review Request: emacs-hcl-mode - Emacs major mode of HCL (Hashicorp Configuration Language)

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748259

Bug ID: 1748259
   Summary: Review Request: emacs-hcl-mode - Emacs major mode of
HCL (Hashicorp Configuration Language)
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pikachu.2...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://melmorabity.fedorapeople.org/packages/emacs-hcl-mode/emacs-hcl-mode.spec
SRPM URL:
https://melmorabity.fedorapeople.org/packages/emacs-hcl-mode/emacs-hcl-mode-0.03-1.fc30.src.rpm
Description: Emacs major mode of HCL
Fedora Account System Username: melmorabity

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748257] New: Review Request: emacs-dockerfile-mode - An emacs mode for handling Dockerfiles

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748257

Bug ID: 1748257
   Summary: Review Request: emacs-dockerfile-mode - An emacs mode
for handling Dockerfiles
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pikachu.2...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://melmorabity.fedorapeople.org/packages/emacs-dockerfile-mode/emacs-dockerfile-mode.spec
SRPM URL:
https://melmorabity.fedorapeople.org/packages/emacs-dockerfile-mode/emacs-dockerfile-mode-1.2-1.fc30.src.rpm
Description: This package provides a major mode `dockerfile-mode' for use with
the standard `Dockerfile' file format.  Additional convenience functions allow
images to be built easily.
Fedora Account System Username: melmorabity

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747552] Review Request: libdfp - Decimal Floating Point library

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747552

Artur Iwicki  changed:

   What|Removed |Added

 CC||fed...@svgames.pl



--- Comment #1 from Artur Iwicki  ---
>Group: System Environment/Libraries
>BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
>[...]
>Group: Development/Libraries
The Group and BuildRoot tags are not used in Fedora.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections

>%install
>rm -rf $RPM_BUILD_ROOT
>[...]
>%clean
>rm -rf $RPM_BUILD_ROOT
Don't remove the buildroot during %install. The %clean section is not needed.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections

>%install
>make install install_root=$RPM_BUILD_ROOT
You can probably use %{make_install} here.

>cd Build
>[...]
>cd ..
Maybe use pushd / popd instead?

You should also include the COPYING.txt file in %files (and mark it as
%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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748233] Review Request: pgloader - Data loading tool for PostgreSQL

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748233

Lukas Javorsky  changed:

   What|Removed |Added

Version|rawhide |30



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748243] New: Review Request: emacs-blacken - Python Black for Emacs

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748243

Bug ID: 1748243
   Summary: Review Request: emacs-blacken - Python Black for Emacs
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pikachu.2...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://melmorabity.fedorapeople.org/packages/emacs-blacken/emacs-blacken.spec
SRPM URL:
https://melmorabity.fedorapeople.org/packages/emacs-blacken/emacs-blacken-0-0.1.20190521git1874018.fc30.src.rpm
Description: Blacken uses black to format a Python buffer. It can be called
explicitly on a certain buffer, but more conveniently, a minor-mode
'blacken-mode' is provided that turns on automatically running black on a
buffer before saving.
Fedora Account System Username: melmorabity

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748228] Review Request: ghc-xdg-userdirs - Basic implementation of XDG user directories specification

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748228



--- Comment #1 from Jens Petersen  ---
https://koji.fedoraproject.org/koji/taskinfo?taskID=37437377

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748233] New: Review Request: pgloader - Data loading tool for PostgreSQL

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748233

Bug ID: 1748233
   Summary: Review Request: pgloader - Data loading tool for
PostgreSQL
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ljavo...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://github.com/ljavorsk/pgloader/blob/master/pgloader.spec
SRPM URL:
https://github.com/ljavorsk/pgloader/blob/master/pgloader-3.6.1-1.fc30.src.rpm
Description: 
Data loading tool that allows to implement continuous migration 
from current database to PostreSQL.
 
It uses the COPY PostreSQL protocol to stream the data into the server.
   
  Pgloader can read data from CSV and DBF files, or SQLite, MySQL,  
MS SQL Server, PostgreSQL, Redshift databases.
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=37437306
Fedora Account System Username: ljavorsk

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1748228] New: Review Request: ghc-xdg-userdirs - Basic implementation of XDG user directories specification

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1748228

Bug ID: 1748228
   Summary: Review Request: ghc-xdg-userdirs - Basic
implementation of XDG user directories specification
   Product: Fedora
   Version: rawhide
Status: NEW
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: peter...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora




Spec URL: https://petersen.fedorapeople.org/reviews//ghc-xdg-userdirs.spec
SRPM URL:
https://petersen.fedorapeople.org/reviews//ghc-xdg-userdirs-0.1.0.2-1.fc31.src.rpm

Description:
On Unix platforms, this should be a very straightforward implementation of the
XDG User Directory spec. On Windows, it will attempt to do the right thing with
regards to choosing appropriate 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1409780] Review Request: golang-github-hanwen-go-fuse - FUSE bindings for Go

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1409780

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||quantum.anal...@gmail.com
 Resolution|--- |DUPLICATE
Last Closed||2019-09-03 06:36:17



--- Comment #8 from Elliott Sales de Andrade  ---


*** This bug has been marked as a duplicate of bug 1712280 ***

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1712280] Review Request: golang-github-hanwen-fuse - FUSE bindings for Go

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1712280

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 CC||christophe.dela...@gmail.co
   ||m



--- Comment #23 from Elliott Sales de Andrade  ---
*** Bug 1409780 has been marked as a duplicate of this bug. ***


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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1377262] Review Request: golang-github-go-macaron-gzip - Provides Gzip compress to responses for Macaron

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1377262

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||quantum.anal...@gmail.com
 Resolution|--- |DUPLICATE
Last Closed||2019-09-03 06:33:25



--- Comment #3 from Elliott Sales de Andrade  ---


*** This bug has been marked as a duplicate of bug 1691948 ***

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1691948] Review Request: golang-github-macaron-gzip - Provides Gzip compress to responses for Macaron

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1691948

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Blocks||1376389
 CC||mru...@redhat.com



--- Comment #10 from Elliott Sales de Andrade  ---
*** Bug 1377262 has been marked as a duplicate of this bug. ***


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1376389
[Bug 1376389] Tracker for grafana dependencies
-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1412152] Review Request: golang-github-blang-semver - Semantic Versioning library written in golang

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1412152

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Blocks||1340560



--- Comment #14 from Elliott Sales de Andrade  ---
*** Bug 1340558 has been marked as a duplicate of this bug. ***


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1340560
[Bug 1340560] Update to v1.0.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1340558] Review Request: golang-github-blang-semver - Semantic Versioning (semver) library written in golang

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1340558

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||quantum.anal...@gmail.com
 Resolution|--- |DUPLICATE
Last Closed||2019-09-03 06:31:37



--- Comment #1 from Elliott Sales de Andrade  ---


*** This bug has been marked as a duplicate of bug 1412152 ***

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747843] Review Request: golang-github-test-deep - Golang deep variable equality test

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747843

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Status|NEW |POST
   Assignee|nob...@fedoraproject.org|quantum.anal...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747622] Review Request: golang-github-cmd - Go library wrapper around os/exec.Cmd

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747622



--- Comment #4 from Elliott Sales de Andrade  ---
When running tests now, one of them fails:

--- FAIL: TestCmdNotFound (0.00s)
cmd_test.go:272: {Cmd:cmd-does-not-exist PID:0 Complete:false Exit:-1
Error:exec: "cmd-does-not-exist": executable file not found in $PATH StartTs:0
StopTs:0 Runtime:0 Stdout:[] Stderr:[]}
cmd_test.go:273: [Error: *exec.Error != *errors.errorString]

I guess this might be a bug in deep against Go 1.13, maybe? It seems to be
incorrectly comparing the expected Error types.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1747843] Review Request: golang-github-test-deep - Golang deep variable equality test

2019-09-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1747843

Elliott Sales de Andrade  changed:

   What|Removed |Added

 CC||quantum.anal...@gmail.com



--- Comment #1 from Elliott Sales de Andrade  ---
Tests don't run for the same reason as bug 1747622, but they do pass when
go-rpm-integration is fixed.

Approved.

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]: 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 %license.
[x]: License field in the package spec file matches the actual license.
 Note: There is no build directory. Running licensecheck on vanilla
 upstream sources. Licenses found: "Unknown or generated", "Expat
 License". 6 files have unknown license. Detailed output of
 licensecheck in /home/elliott/rpmbuild/review/1747843-golang-github-
 test-deep/licensecheck.txt
[x]: Package does not own files or directories owned by other packages.
[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.
[?]: 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]: 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 10240 bytes in 2 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]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[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]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package must not depend on deprecated() packages.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[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:
[-]: 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).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[!]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Fully versioned dependency in subpackages if applicable.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI