[Bug 1576413] Review Request: boom-boot - boot manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576413



--- Comment #9 from Marian Csontos  ---
I think snpshots of /boot has its own set of problems. One of snapper's flaws
when using BTRFS is exactly that - it keeps information about snapshots inside
the filesystem which is "snapshotted".

I wonder, if you have multiple snapshots of /boot, where do you keep
information about those snapshots and grub.cfg to pick the
snapshot+kernel+initrd+cmdline to boot?

And if you boot into a snapshot, and want to make an update to that config,
where to save that information?

Either you still keep data inside "master" snapshot of /boot, or you need to
keep it up to date somehow, don't you?

Or do you move the problem to EFI boot partition or elsewhere? And than one can
not "snapshot" that, and files are saved/installed there too.

And should the FS get dirty, how does one fsck it? Of course /boot can get
dirty itself too, but it is rather low traffic, compared to /.

-- 
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

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556

Antonio Trande  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Antonio Trande  ---
./autogen.sh
LDFLAGS="-Wl,--as-needed %{build_ldflags}"
%configure --enable-gtk-doc
%make_build V=1

In this way, 'configure' is ran two times; try with

%set_build_flags
./autogen.sh --enable-gtk-doc

and force '-Wl,--as-needed' by Make's command:

%make_build V=1 CFLAGS="%optflags %__global_ldflags -Wl,--as-needed"
LDFLAGS="%__global_ldflags -Wl,--as-needed"

Package 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1577468] Review Request: R-tinytex - Helper Functions to Install and Maintain 'TeX Live', and Compile 'LaTeX' Documents

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577468



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-tinytex

-- 
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

[Bug 1577543] Review Request: R-tufte - Tufte' s Styles for R Markdown Documents

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577543



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-tufte

-- 
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

[Bug 1577690] Review Request: ignition-cmake - CMake modules to be used by the Ignition projects

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577690

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
 - Remove the shebang for:

ignition-cmake-devel.noarch: E: wrong-script-interpreter
/usr/share/ignition/ignition-cmake1/codecheck/cpplint.py /usr/bin/env python
ignition-cmake-devel.noarch: E: non-executable-script
/usr/share/ignition/ignition-cmake1/codecheck/cpplint.py 644 /usr/bin/env
python

Package otherwise 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: Checking patched sources after %prep for licenses. Licenses
 found: "Apache (v2.0)", "Apache (v2.0) BSD (3 clause)", "BSD (2
 clause)", "Unknown or generated", "BSD (3 clause)". 46 files have
 unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/ignition-cmake/review-ignition-
 cmake/licensecheck.txt
[x]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/share/ignition(ignition-
 transport, ignition-msgs)
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: 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 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 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]: 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

= 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 

[Bug 1574121] Review Request: python-graphviz - Simple Python interface for Graphviz

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574121



--- Comment #7 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-graphviz

-- 
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

[Bug 1576413] Review Request: boom-boot - boot manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576413



--- Comment #11 from Bryn M. Reeves  ---
It's also worth noting that the boot-as-directory-of-root scenario is somewhat
incompatible with the aims of BLS: to provide a single, system-wide boot
volume. If it's a directory of one particular OS instance it can never fulfil
this role.

It's possible that we may simply need to rule this configuration out as
supportable and to make that clear in our documentation and guidance.

-- 
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

[Bug 1574128] Review Request: python-Automat - Self-service finite-state machines for the programmer on the go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574128



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-Automat

-- 
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

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556



--- Comment #5 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/keybinder-3.0

-- 
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

[Bug 1577953] New: Review Request: golang-github-kolo-xmlrpc - Implementation of XMLRPC protocol in Go language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577953

Bug ID: 1577953
   Summary: Review Request: golang-github-kolo-xmlrpc -
Implementation of XMLRPC protocol in Go language
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: pg...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://pgier.fedorapeople.org/golang/golang-github-kolo-xmlrpc/golang-github-kolo-xmlrpc.spec
SRPM URL:
https://pgier.fedorapeople.org/golang/golang-github-kolo-xmlrpc/golang-github-kolo-xmlrpc-0-0.1.20180513git0826b98.fc27.src.rpm
Description: Implementation of XMLRPC protocol in Go language
Fedora Account System Username: pgier

-- 
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

[Bug 1577953] Review Request: golang-github-kolo-xmlrpc - Implementation of XMLRPC protocol in Go language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577953

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@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

[Bug 1576413] Review Request: boom-boot - boot manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576413



--- Comment #12 from Alasdair Kergon  ---
1) document more clearly?
2) have the tool recognise this situation and provide helpful messages?

-- 
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

[Bug 1576836] Review Request: zam-plugins - A collection of LV2/LADSPA/ JACK audio plugins

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576836



--- Comment #2 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/zam-plugins

-- 
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

[Bug 1574121] Review Request: python-graphviz - Simple Python interface for Graphviz

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574121



--- Comment #8 from Fedora Update System  ---
python-graphviz-0.8.3-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a3693fb2ca

-- 
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

[Bug 1573695] Review Request: battray - simple tray icon to show a laptop’s battery status.

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573695



--- Comment #14 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/battray

-- 
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

[Bug 1577011] Review Request: wiringpi - PIN based GPIO access library for BCM283x SoC devices

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577011



--- Comment #7 from Vaughan  ---
Ok, I have refactored the spec and fixed a few more rpmlint grumbles, plus
added some pkg-config files.

Updated Spec URL:
https://raw.githubusercontent.com/agrez/wiringpi/master/wiringpi.spec


Updated SRPM URL:
https://github.com/agrez/wiringpi/releases/download/2.46-1/wiringpi-2.46-1.fc28.src.rpm

koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=26960053


Cheers!
Vaughan

-- 
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

[Bug 1576413] Review Request: boom-boot - boot manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576413



--- Comment #10 from Bryn M. Reeves  ---
> My own setup has /boot as part of the OS disk that is snapshotted with the 
> rest 
> of the OS

We're aware of this limitation and are trying to find long-term solutions,
however, even before this there's a bigger problem with this configuration: the
grub2 BLS support is fundamentally broken in this scenario (since it is unable
to recognise that it must search within the boot/ directory contained within
the root device).

-- 
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

[Bug 1574128] Review Request: python-Automat - Self-service finite-state machines for the programmer on the go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574128



--- Comment #4 from Fedora Update System  ---
python-Automat-0.6.0-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-3af65508e7

-- 
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

[Bug 1574128] Review Request: python-Automat - Self-service finite-state machines for the programmer on the go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574128

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1575412] Review Request: libmml - MML Widget

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1575412

Tom "spot" Callaway  changed:

   What|Removed |Added

 CC||tcall...@redhat.com
 Blocks|182235 (FE-Legal)   |



--- Comment #5 from Tom "spot" Callaway  ---
licensing (and exception) is fine. Lifting FE-Legal.


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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1577011] Review Request: wiringpi - PIN based GPIO access library for BCM283x SoC devices

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577011

Peter Oliver  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Peter Oliver  ---
Best exclude aarch64 until it can be tested.  Apart from that, I approve the
review.  Thanks for submitting the package!

You'll now need to wait for a sponsor to sponsor you as a new packager.  I'll
draw to the attention of any potentials sponsors your work over at
https://github.com/fedberry/.

-- 
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

[Bug 1575412] Review Request: libmml - MML Widget

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1575412

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #6 from Robert-André Mauchin  ---
Package 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1577953] Review Request: golang-github-kolo-xmlrpc - Implementation of XMLRPC protocol in Go language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577953

Jared Smith  changed:

   What|Removed |Added

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



--- Comment #2 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  It has a proper license. It builds properly in mock.  It 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556



--- Comment #6 from Fedora Update System  ---
keybinder-3.0-0.3.2-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-d775377512

-- 
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

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1576056] Review Request: python-pyte - Simple VTXXX-compatible linux terminal emulator

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576056



--- Comment #5 from Mateusz Mikuła  ---
Thank you for comments.

Links remain the same.
Adressed:
- Removed backslashes from description
- Changed %doc to pick README instead of symlink

Guess I'll look for a sponsor 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1575601] Review Request: beakerlib-libraries - Beakerlib Libraries

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1575601



--- Comment #5 from Andrei Stepanov  ---
Artur Iwicki Hi!
Thank you for your suggestions. 
I made requested changes:

Spec file is :
https://pagure.io/beakerlib-libraries/blob/master/f/beakerlib-libraries.spec

Changes are:

https://pagure.io/beakerlib-libraries/c/9e83077b04221e2e21438548415dba7fd319f7f1?branch=master

I kept /usr/share/beakerlib-libraries/*/*.sh files without executable bit.
There is reason for this: These files supposed only be included in other
scripts. By no means they should be called directly.

Could you please check again?

Thank you!

-- 
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

[Bug 1576413] Review Request: boom-boot - boot manager

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576413



--- Comment #13 from Bryn M. Reeves  ---
> If Boom cannot handle this, then it needs to do something to make that work.

I don't think that it's reasonable to expect boom to make things work that are
explicitly outside the scope of the standard (BLS) that we are following. We
can try to find ways to accommodate it to the best extent possible, and will
look at doing so, but fundamentally we will always be limited by the
applicability of the standard that we are adopting.

> My understanding is that boom doesn't do any _real_ boot management 
> functions, 
> and instead just generates configuration for loaders like GRUB2, rEFInd, or 
> sd-
> boot to use instead.

Define "real boot management" ;-)

The purpose of this is as documented: to allow *any* booted instance of the OS
(including snapshots, which may have been taken prior to the creation of other
managed instances) to manage *any other*. This can only be done if there is a
single, shared system-wide source for this configuration (which is exactly what
the BLS sets out to create).

-- 
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

[Bug 1577454] Review Request: rust-xdg - Library for storing and retrieving files according to XDG

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577454



--- Comment #2 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/rust-xdg

-- 
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

[Bug 1518262] Review Request: nodejs-yarn - Fast, reliable, and secure dependency management

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1518262



--- Comment #23 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/nodejs-yarn

-- 
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

[Bug 1577953] Review Request: golang-github-kolo-xmlrpc - Implementation of XMLRPC protocol in Go language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577953

Paul Gier  changed:

   What|Removed |Added

 Blocks||1512702



--- Comment #1 from Paul Gier  ---
This is a dependency of prometheus node_exporter.

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=26963742


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1512702
[Bug 1512702] Review Request: golang-github-prometheus-node_exporter -
Prometheus exporter for hardware and OS metrics exposed by *NIX kernels
-- 
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

[Bug 1512702] Review Request: golang-github-prometheus-node_exporter - Prometheus exporter for hardware and OS metrics exposed by *NIX kernels

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1512702

Paul Gier  changed:

   What|Removed |Added

 Depends On||1577953




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1577953
[Bug 1577953] Review Request: golang-github-kolo-xmlrpc - Implementation of
XMLRPC protocol in Go 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

[Bug 1574121] Review Request: python-graphviz - Simple Python interface for Graphviz

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574121

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1578094] New: Review Request: rubygem-mini_mime - A lightweight mime type lookup toy

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578094

Bug ID: 1578094
   Summary: Review Request: rubygem-mini_mime - A lightweight mime
type lookup toy
   Product: Fedora
   Version: rawhide
 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



Spec URL: https://pvalena.fedorapeople.org/gems/rubygem-mini_mime.spec
SRPM URL:
https://pvalena.fedorapeople.org/gems/rubygem-mini_mime-1.0.0-1.fc29.src.rpm
Description: Minimal mime type implementation for use with the mail and
rest-client gem.
Fedora Account System Username: pvalena
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=26967563

-- 
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

[Bug 1578098] New: Review Request: python-hpack - Pure-Python HPACK header compression

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578098

Bug ID: 1578098
   Summary: Review Request: python-hpack - Pure-Python HPACK
header compression
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754002-python-hpack/python-hpack.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754002-python-hpack/python-hpack-3.0.0-1.fc29.src.rpm

Description:
HTTP/2 Header Encoding for Python This module contains a pure-Python
HTTP/2 header encoding (HPACK) logic for use in Python programs that implement
HTTP/2. It also contains a compatibility layer that automatically enables the
use of nghttp2 if it's available.

Fedora Account System Username:

-- 
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

[Bug 1578101] New: Review Request: python-h2 - HTTP/ 2 State-Machine based protocol implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578101

Bug ID: 1578101
   Summary: Review Request: python-h2 - HTTP/2 State-Machine based
protocol implementation
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754003-python-h2/python-h2.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754003-python-h2/python-h2-3.0.1-1.fc29.src.rpm

Description:
HTTP/2 Protocol Stack This repository contains a pure-Python
implementation of a HTTP/2 protocol stack. It's written from the ground up to
be embeddable in whatever program you choose to use, ensuring that you can
speak HTTP/2 regardless of your programming paradigm.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1577468] Review Request: R-tinytex - Helper Functions to Install and Maintain 'TeX Live', and Compile 'LaTeX' Documents

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577468



--- Comment #4 from Fedora Update System  ---
R-tinytex-0.5-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-53b0d0d053

-- 
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

[Bug 1578102] Review Request: rubygem-hiredis - Ruby wrapper for hiredis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578102

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #2 from Robert-André Mauchin  ---
Let me know when the PR is merged and built.

-- 
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

[Bug 1564500] Review Request: pipenv - The higher level Python packaging tool

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1564500

Miro Hrončok  changed:

   What|Removed |Added

 Blocks|182235 (FE-Legal)   |




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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1578098] Review Request: python-hpack - Pure-Python HPACK header compression

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578098



--- Comment #1 from Robert-André Mauchin  ---
Fedora Account System Username: eclipseo

-- 
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

[Bug 1578107] New: Review Request: python-priority - A pure-Python implementation of the HTTP/ 2 priority tree

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578107

Bug ID: 1578107
   Summary: Review Request: python-priority - A pure-Python
implementation of the HTTP/2 priority tree
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753944-python-priority/python-priority.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753944-python-priority/python-priority-1.3.0-1.fc29.src.rpm

Description:
A HTTP/2 Priority Implementation. Priority is a pure-Python
implementation of the priority logic for HTTP/2, set out in RFC 7540 Section
5.3 (Stream Priority)_. This logic allows for clients to express a preference
for how the server allocates its (limited) resources to the many outstanding
HTTP requests that may be running over a single HTTP/2 connection.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1577468] Review Request: R-tinytex - Helper Functions to Install and Maintain 'TeX Live', and Compile 'LaTeX' Documents

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577468

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1577468] Review Request: R-tinytex - Helper Functions to Install and Maintain 'TeX Live', and Compile 'LaTeX' Documents

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577468



--- Comment #5 from Fedora Update System  ---
R-tinytex-0.5-1.fc27 has been submitted as an update to Fedora 27.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-ee2c85734b

-- 
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

[Bug 1574121] Review Request: python-graphviz - Simple Python interface for Graphviz

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574121

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
python-graphviz-0.8.3-1.fc28 has been pushed to the Fedora 28 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-2018-a3693fb2ca

-- 
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

[Bug 1574128] Review Request: python-Automat - Self-service finite-state machines for the programmer on the go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574128

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System  ---
python-Automat-0.6.0-1.fc28 has been pushed to the Fedora 28 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-2018-3af65508e7

-- 
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

[Bug 1564500] Review Request: pipenv - The higher level Python packaging tool

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1564500



--- Comment #7 from Miro Hrončok  ---
The License tag needs to cover everything bundled unfortunately.

Also, I think we need to mark all LICENSE files with %license, hopefully it
supports absolute paths as does %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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1578099] New: Review Request: python-hyperframe - HTTP/ 2 framing layer for Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578099

Bug ID: 1578099
   Summary: Review Request: python-hyperframe - HTTP/2 framing
layer for Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754004-python-hyperframe/python-hyperframe.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00754004-python-hyperframe/python-hyperframe-5.1.0-1.fc29.src.rpm

Description:
Pure-Python HTTP/2 framing This library contains the HTTP/2
framing code used in the hyper_ project. It provides a pure-Python codebase
that is capable of decoding a binary stream into HTTP/2 frames. This library is
used directly by hyper_ and a number of other projects to provide HTTP/2 frame
decoding logic.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1578102] New: Review Request: rubygem-hiredis - Ruby wrapper for hiredis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578102

Bug ID: 1578102
   Summary: Review Request: rubygem-hiredis - Ruby wrapper for
hiredis
   Product: Fedora
   Version: rawhide
 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



Spec URL: https://pvalena.fedorapeople.org/gems/rubygem-hiredis.spec
SRPM URL:
https://pvalena.fedorapeople.org/gems/rubygem-hiredis-0.6.1-1.fc29.src.rpm
Description: Ruby wrapper for hiredis (protocol serialization/deserialization
and blocking I/O).
Fedora Account System Username: pvalena
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=26968035

-- 
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

[Bug 1578101] Review Request: python-h2 - HTTP/ 2 State-Machine based protocol implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578101

Robert-André Mauchin  changed:

   What|Removed |Added

 Depends On||1578098, 1578099




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1578098
[Bug 1578098] Review Request: python-hpack - Pure-Python HPACK header
compression
https://bugzilla.redhat.com/show_bug.cgi?id=1578099
[Bug 1578099] Review Request: python-hyperframe - HTTP/2 framing layer for
Python
-- 
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

[Bug 1578099] Review Request: python-hyperframe - HTTP/ 2 framing layer for Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578099

Robert-André Mauchin  changed:

   What|Removed |Added

 Blocks||1578101




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1578101
[Bug 1578101] Review Request: python-h2 - HTTP/2 State-Machine based
protocol implementation
-- 
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

[Bug 1578098] Review Request: python-hpack - Pure-Python HPACK header compression

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578098

Robert-André Mauchin  changed:

   What|Removed |Added

 Blocks||1578101




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1578101
[Bug 1578101] Review Request: python-h2 - HTTP/2 State-Machine based
protocol implementation
-- 
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

[Bug 1578108] New: Review Request: python-towncrier - Building newsfiles for your project

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578108

Bug ID: 1578108
   Summary: Review Request: python-towncrier - Building newsfiles
for your project
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753989-python-towncrier/python-towncrier.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753989-python-towncrier/python-towncrier-17.8.0-1.fc29.src.rpm

Description:
Towncrier is a utility to produce useful, summarised news files for your 
project. Rather than reading the Git history as some newer tools to produce it,
or having one single file which developers all write to, towncrier reads "news 
fragments" which contain information useful to end users.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1578091] New: Missing package Remmina from EPEL/EPEL-Testing

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578091

Bug ID: 1578091
   Summary: Missing package Remmina from EPEL/EPEL-Testing
   Product: Fedora EPEL
   Version: epel7
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: jas_be...@hotmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Description of problem:
I'm not sure if this is the right place to submit this but package remmina is
missing from EPEL/EPEL-Testing Repos. According to the developers it should be
available.  It's currently here:

https://bodhi.fedoraproject.org/updates/?packages=remmina

Version-Release number of selected component (if applicable):

How reproducible:

Always

Steps to Reproduce:
1.yum-config-manager --enable epel-testing
2.yum --disablerepo=* --enablerepo=epel install remmina


Actual results:
Loaded plugins: fastestmirror, langpacks
Loading mirror speeds from cached hostfile

epel: mirror.compevo.com
epel-debuginfo: mirror.oss.ou.edu
epel-source: mirror.compevo.com
epel-testing: mirror.compevo.com
epel-testing-debuginfo: mirror.oss.ou.edu
epel-testing-source: mirror.compevo.com
No package remmina available.
Error: Nothing to do

Expected results:
installs 'remmina*' 'freerdp*'

Additional info:

-- 
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

[Bug 1578093] New: Review Request: python-constantly - Symbolic constants in Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578093

Bug ID: 1578093
   Summary: Review Request: python-constantly - Symbolic constants
in Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753886-python-constantly/python-constantly.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753886-python-constantly/python-constantly-15.1.0-1.fc29.src.rpm

Description:
A library that provides symbolic constant support. It includes
collections and constants with text, numeric, and bit flag values. Originally
twisted.python.constants from the Twisted project.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1388294] Review Request: pyflame - Ptracing Profiler For Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1388294

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags|fedora-review+  |fedora-review?



--- Comment #20 from Zbigniew Jędrzejewski-Szmek  ---
A lot of time has passed, let's 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

[Bug 1578094] Review Request: rubygem-mini_mime - A lightweight mime type lookup toy

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578094

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
Package approved.


Package Review
==

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


Issues:
===
- Package contains Requires: ruby(release).


= 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: "MIT/X11 (BSD like)", "Unknown or generated". 14 files have
 unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/rubygem-mini_mime/review-rubygem-
 mini_mime/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/gems,
 /usr/share/gems/doc
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[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]: 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 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 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]: 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]: Pure Ruby package must be built as noarch
[x]: Package does not contain Requires: ruby(abi).

= SHOULD items =

Generic:
[-]: Avoid bundling fonts in non-fonts packages.
 Note: Package contains font files
[-]: 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 

[Bug 1578101] Review Request: python-h2 - HTTP/ 2 State-Machine based protocol implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578101

Robert-André Mauchin  changed:

   What|Removed |Added

  Flags|needinfo?(zebo...@gmail.com |
   |)   |



--- Comment #2 from Robert-André Mauchin  ---
You don't seem to be member of the Packager group?

-- 
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

[Bug 1578102] Review Request: rubygem-hiredis - Ruby wrapper for hiredis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578102

Pavel Valena  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #1 from Pavel Valena  ---
Depends on new macros.rubygems:

https://src.fedoraproject.org/rpms/ruby/pull-request/19

-- 
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

[Bug 1578103] New: Review Request: python-hyperlink - A featureful, immutable, and correct URL for Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578103

Bug ID: 1578103
   Summary: Review Request: python-hyperlink - A featureful,
immutable, and correct URL for Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753925-python-hyperlink/python-hyperlink.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/python-twisted/fedora-rawhide-x86_64/00753925-python-hyperlink/python-hyperlink-18.0.0-1.fc29.src.rpm

Description:
The humble, but powerful, URL runs everything around us. Chances are you've
used several just to read this text. Hyperlink is a featureful, pure-Python
implementation of the URL, with an emphasis on correctness.

Fedora Account System Username: eclipseo

-- 
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

[Bug 1578094] Review Request: rubygem-mini_mime - A lightweight mime type lookup toy

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578094

Pavel Valena  changed:

   What|Removed |Added

  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

[Bug 1578101] Review Request: python-h2 - HTTP/ 2 State-Machine based protocol implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578101

Corey Sheldon  changed:

   What|Removed |Added

 CC||sheldon.corey@openmailbox.o
   ||rg, zebo...@gmail.com
  Flags||needinfo?(zebo...@gmail.com
   ||)



--- Comment #1 from Corey Sheldon  ---
Willing to review. fasid: linuxmodder

-- 
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

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
keybinder-3.0-0.3.2-1.fc28 has been pushed to the Fedora 28 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-2018-d775377512

-- 
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

[Bug 1419379] Review Request: libefp - A full implementation of the Effective Fragment Potential (EFP) method

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1419379

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:44



--- Comment #17 from Fedora Update System  ---
libefp-1.5.0-2.fc28 has been pushed to the Fedora 28 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

[Bug 505154] Tracker: Review Requests for Science and Technology related packages

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=505154
Bug 505154 depends on bug 1419379, which changed state.

Bug 1419379 Summary: Review Request: libefp - A full implementation of the 
Effective Fragment Potential (EFP) method
https://bugzilla.redhat.com/show_bug.cgi?id=1419379

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1574287] Review Request: R-rvest - Easily Harvest (Scrape) Web Pages

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574287

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:34



--- Comment #8 from Fedora Update System  ---
R-rvest-0.3.2-1.fc28 has been pushed to the Fedora 28 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

[Bug 1558473] Review Request: xtl - QuantStack tools library

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1558473

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:37



--- Comment #9 from Fedora Update System  ---
xtl-0.4.7-1.fc28 has been pushed to the Fedora 28 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

[Bug 1574283] Review Request: R-whoami - Username, Full Name, Email Address, 'GitHub' Username of the Current User

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574283

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:32



--- Comment #8 from Fedora Update System  ---
R-whoami-1.1.2-1.fc28 has been pushed to the Fedora 28 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

[Bug 1573855] Review Request: editorconfig - Parser for EditorConfig files written in C

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573855

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:16



--- Comment #9 from Fedora Update System  ---
editorconfig-0.12.2-3.fc28 has been pushed to the Fedora 28 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

[Bug 1574284] Review Request: R-gmailr - Access the Gmail RESTful API

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1574284

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-05-14 13:53:29



--- Comment #8 from Fedora Update System  ---
R-gmailr-0.7.1-1.fc28 has been pushed to the Fedora 28 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

[Bug 1576879] Review Request: ignition - First boot installer and configuration tool

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576879



--- Comment #11 from Steve Milner  ---
Colin,

Agreed.

-- 
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

[Bug 1576879] Review Request: ignition - First boot installer and configuration tool

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576879



--- Comment #10 from Colin Walters  ---
 - Unbundle vendor, that might require lots of work packaging the missing deps.
Use gofed to package these.

I don't think we have an interest in doing this right now.  We have higher
priority things to do: for example, fixing Ignition to work more nicely with
SELinux.

-- 
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

[Bug 1576853] Review Request: mednafen - A multi-system emulator utilizing OpenGL and SDL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576853

Julian Sikorski  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2018-05-14 14:33:01



-- 
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

[Bug 1576853] Review Request: mednafen - A multi-system emulator utilizing OpenGL and SDL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1576853



--- Comment #4 from Julian Sikorski  ---
Thank you! Upstream have been notified:
https://forum.fobby.net/index.php?t=msg=1689
Regarding the desktop file and/or appdata: mednafen is a command-line-only
software. There is no GUI, and, as such, no .desktop files or appdata.

-- 
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

[Bug 1564500] Review Request: pipenv - The higher level Python packaging tool

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1564500



--- Comment #6 from Michal Cyprian  ---
The new version 11.10.4 containing the legal issue fixes was released, so I
rebased the spec file and updated all the patches.

The ./tests/pypi/odfpy/odfpy-1.3.6-py2.7.egg file is there only for testing
purpose.

I've tried to fix/workaround all the issues that were pointed out. The updated
files are here:
Spec URL: https://mcyprian.fedorapeople.org/pipenv.spec
SRPM URL: https://mcyprian.fedorapeople.org/pipenv-11.10.4-1.fc28.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

[Bug 1419379] Review Request: libefp - A full implementation of the Effective Fragment Potential (EFP) method

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1419379



--- Comment #18 from Fedora Update System  ---
libefp-1.5.0-2.fc26 has been pushed to the Fedora 26 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

[Bug 1567556] Review Request: keybinder-3.0 - A library for registering global keyboard shortcuts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1567556



--- Comment #7 from Fedora Update System  ---
keybinder-3.0-0.3.2-1.fc27 has been submitted as an update to Fedora 27.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a2a23aca78

-- 
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

[Bug 1432901] Review Request: restic - restic backup program

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1432901

Till Maas  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW)
 Resolution|--- |WONTFIX
Last Closed||2018-05-14 14:33:49



--- Comment #3 from Till Maas  ---
Philip, I close this review request for now. If you find time to work on this
again, please file a new request. Thank you!


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
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 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

[Bug 1578152] New: Review Request: perl-Minion - High performance job queue for the Perl programming language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578152

Bug ID: 1578152
   Summary: Review Request: perl-Minion - High performance job
queue for the Perl programming language
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: emman...@seyman.fr
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://people.parinux.org/~seyman/fedora/perl-Minion/perl-Minion.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-Minion/perl-Minion-9.03-1.fc28.src.rpm
Description:
Minion is a high performance job queue for the Perl programming language,
with support for multiple named queues, priorities, delayed jobs, job
dependencies, job progress, job results, retries with back-off, rate
limiting, unique jobs, statistics, distributed workers, parallel
processing, auto-scaling, remote control, Mojolicious admin UI, resource
leak protection and multiple backends (such as PostgreSQL).

Fedora Account System Username: eseyman
Rpmlint Output: 1 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
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

[Bug 1575991] Review Request: golang-github-jedisct1-dnsstamps - DNS Stamps library for Go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1575991

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  It builds cleanly in mock.

-- 
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

[Bug 1578151] New: Review Request: perl-Mojo-Pg - Mojolicious ♥ PostgreSQL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578151

Bug ID: 1578151
   Summary: Review Request: perl-Mojo-Pg - Mojolicious ♥
PostgreSQL
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: emman...@seyman.fr
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://people.parinux.org/~seyman/fedora/perl-Mojo-Pg/perl-Mojo-Pg.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-Mojo-Pg/perl-Mojo-Pg-4.08-1.fc28.src.rpm
Description:
Mojo::Pg is a tiny wrapper around DBD::Pg that makes PostgreSQL a lot of
fun to use with the Mojolicious real-time web framework. Perform queries
blocking and non-blocking, use all SQL features PostgreSQL has to offer,
generate CRUD queries from data structures, manage your database schema
with migrations and build scalable real-time web applications with the
publish/subscribe pattern.

Fedora Account System Username: eseyman
Rpmlint Output:
perl-Mojo-Pg.noarch: W: spelling-error Summary(en_US) Mojolicious -> Malicious
perl-Mojo-Pg.src: W: spelling-error Summary(en_US) Mojolicious -> Malicious
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

-- 
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

[Bug 1578152] Review Request: perl-Minion - High performance job queue for the Perl programming language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578152

Emmanuel Seyman  changed:

   What|Removed |Added

 Depends On||1578151
 Whiteboard||NotReady




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1578151
[Bug 1578151] Review Request: perl-Mojo-Pg - Mojolicious ♥ PostgreSQL
-- 
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

[Bug 1578151] Review Request: perl-Mojo-Pg - Mojolicious ♥ PostgreSQL

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578151

Emmanuel Seyman  changed:

   What|Removed |Added

 Blocks||1578152




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1578152
[Bug 1578152] Review Request: perl-Minion - High performance job queue for
the Perl programming 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

[Bug 1573999] Review Request: golang-github-pquerna-cachecontrol - Golang HTTP Cache-Control Parser and Interpretation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573999

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  It builds cleanly in mock, and 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1571619] Review Request: python-sphinxcontrib-trio - Make Sphinx better at documenting Python functions and methods

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1571619



--- Comment #2 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-sphinxcontrib-trio

-- 
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

[Bug 1575992] Review Request: golang-github-k-sone-critbitgo - Crit-bit trees in golang and its applications

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1575992

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
This package is approved.  It conforms to the Fedora packaging guidelines, has
an allowed license, and builds cleanly in mock.

-- 
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

[Bug 1573991] Review Request: golang-github-aead-chacha20 - ChaCha20 and XChaCha20 stream ciphers

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573991

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  It builds cleanly in mock.  It 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1573993] Review Request: golang-github-aead-poly1305 - The poly1305 message authentication code

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573993

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST



-- 
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

[Bug 1573993] Review Request: golang-github-aead-poly1305 - The poly1305 message authentication code

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573993

Jared Smith  changed:

   What|Removed |Added

 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  It builds cleanly in mock.  It 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1388294] Review Request: pyflame - Ptracing Profiler For Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1388294



--- Comment #21 from Evan Klitzke  ---
Created attachment 1436628
  --> https://bugzilla.redhat.com/attachment.cgi?id=1436628=edit
Spec file for pyflame.

This is the new current spec file I am using, which is in git at
https://github.com/eklitzke/pyflame-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

[Bug 1388294] Review Request: pyflame - Ptracing Profiler For Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1388294

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #22 from Zbigniew Jędrzejewski-Szmek  ---
OK, this still looks good. rpmlint has only bogus spelling corrections.

-- 
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

[Bug 1573997] Review Request: golang-github-jedisct1-xsecretbox - Go implementation of crypto_secretbox_xchacha20poly1305

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573997

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package.

 * It conforms to the Fedora Packaging Guidelines.
 * It has an acceptible licnese
 * It build cleanly in mockchain (with its listed dependencies)

It 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1388294] Review Request: pyflame - Ptracing Profiler For Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1388294

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|ASSIGNED|POST



-- 
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

[Bug 1573984] Review Request: golang-github-facebookgo-atomicfile - Provides an atomically written/ replaced file

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573984

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package, and it conforms to the Fedora Packaging
Guidelines.  The license is fine, and the package builds cleanly in mock.  It
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1573995] Review Request: golang-github-jedisct1-dlog - A super simple logger for Go

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573995

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package.

 * It conforms to the Fedora Packaging Guidelines
 * It has an acceptible license.
 * It builds cleanly in mock

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1573994] Review Request: golang-github-facebookgo-pidfile - Pid files management

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1573994

Jared Smith  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jsmith.fed...@gmail.com
   Assignee|nob...@fedoraproject.org|jsmith.fed...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Jared Smith  ---
I have reviewed this package.

 * It conforms to the Fedora Packaging Guidelines
 * It builds cleanly in mockchain (with its dependency)
 * It is under an approved license.

It 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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1577011] Review Request: wiringpi - PIN based GPIO access library for BCM283x SoC devices

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1577011



--- Comment #9 from Vaughan  ---
(In reply to Peter Oliver from comment #8)
> Best exclude aarch64 until it can be tested.  Apart from that, I approve the
> review.  Thanks for submitting the package!
> 
Done.
https://github.com/agrez/wiringpi/commit/48f9e6d2d93fb3dbe2df21109d37ba26e37bc1ac

> You'll now need to wait for a sponsor to sponsor you as a new packager. 
> I'll draw to the attention of any potentials sponsors your work over at
> https://github.com/fedberry/.

OK, no problem. Thanks for your time with this review. In regards to Fedora
packaging guidelines / policies, it has been a good learning experience for me
(I have been living in the packaging 'wild west' for too long). ;-)

Cheers!
Vaughan

-- 
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

[Bug 1578223] New: Review Request: python-yapf - A formatter for Python code

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578223

Bug ID: 1578223
   Summary: Review Request: python-yapf - A formatter for Python
code
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: e...@eklitzke.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eklitzke/python-yapf/fedora-rawhide-x86_64/00754180-python-yapf/yapf.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eklitzke/python-yapf/fedora-rawhide-x86_64/00754180-python-yapf/python-yapf-0.21.0-3.fc29.src.rpm

Description: YAPF is Yet Another Python Formatter. It automatically reformats
source files to conform to conform to PEP 8 and other Python coding best
practices.

FAS account: eklitzke

-- 
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

[Bug 1578238] New: Review Request: python-scan-build - A Clang scan-build reimplementation in Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578238

Bug ID: 1578238
   Summary: Review Request: python-scan-build - A Clang scan-build
reimplementation in Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: e...@eklitzke.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eklitzke/python-scan-build/fedora-rawhide-x86_64/00754183-python-scan-build/scan-build.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eklitzke/python-scan-build/fedora-rawhide-x86_64/00754183-python-scan-build/python-scan-build-2.0.13-1.fc29.src.rpm

This package is a little weird and I need some help on it. I've broken this
down into a few parts below.

scan-build binary
~~~

Right now there is an official utility called scan-build which is part of LLVM.
One of the LLVM developers has been working on a Python reimplementation of
scan-build, which also has some extra features. This is likely to become the
official scan-build implementation in a future LLVM release.

The existing scan-build implementation is packaged in the clang-analyzer
package, which includes the following binaries:

/usr/bin/scan-build
/usr/bin/scan-view

My new package includes the following binaries:

/usr/bin/analyze-build
/usr/bin/analyze-c++
/usr/bin/analyze-cc
/usr/bin/intercept-build
/usr/bin/intercept-c++
/usr/bin/intercept-cc
/usr/bin/scan-build

As you can see there are new (and useful!) binaries in this Python
implementation. However, I will need to rename the /usr/bin/scan-build to avoid
conflicting with clang-analyzer. I'm open to suggestions on how to handle this.

License
~~

The license file used by this project is the same as LLVM, which is a custom
NCSA-like license. I looked at the llvm source package, and they just declare
the license there to be NCSA, so I did the same. The actual license can be
found here: https://github.com/rizsotto/scan-build/blob/master/LICENSE.txt

noarch
~~

pyp2rpm thought this was a binary package because the setup.py file says it's
not zip-safe. Actually it works in kind of a strange way: the package includes
a file named ear.c, and the intercept-build binary compiles this on-the-fly
when wrapping builds. The ear.c file isn't actually built by setup.py. I
changed the spec file to be noarch which I think is correct, but I would like
someone to double check this.

My FAS username: eklitzke

-- 
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

[Bug 1578093] Review Request: python-constantly - Symbolic constants in Python

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1578093

Evan Klitzke  changed:

   What|Removed |Added

 CC||e...@eklitzke.org



--- Comment #1 from Evan Klitzke  ---
I am new to package review, so bear with me. The spec file looks good to me,
passes rpmlint, and I was able to build the SRPM.

However, I noticed the project on GitHub has tests, but the spec file does not
run them. It looks like PyPI doesn't include the tests file. What's the
protocol regarding 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