[Bug 1328248] Review Request: python-pysword - Python bindings to read Sword bible files directly

[bugzilla]

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



--- Comment #23 from Robert-André Mauchin  ---
When I download
https://gitlab.com/tgc-dk/%{srcname}/repository/archive.tar.gz?ref=%{version}#/%{srcname}-%{version}.tar.gz,
the root folder of the archive I get is named %{srcname}-%{version}-%{commit},
not %{srcname}-%{version}.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1328248] Review Request: python-pysword - Python bindings to read Sword bible files directly

[bugzilla]

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



--- Comment #22 from TR Bentley  ---
https://trb143.fedorapeople.org/python-pysword-0.2.5-1.fc27.src.rpm

https://trb143.fedorapeople.org/python-pysword.spec

Rebuilt from scratch using fedpkg.

Unable to find any reference  to {commit}

if this is still wrong what commands are you using to test so I can test my
self.

-- 
You are receiving this mail because:
You are on the CC 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 1559200] Review Request: R-processx - Execute and Control System Processes

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
R-processx-2.0.0.1-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-e4b039096b

-- 
You are receiving this mail because:
You are on the CC 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 1483339] Review Request: kiwi - A flexible operating system image builder

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #26 from Fedora Update System  ---
kiwi-9.14.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-1e2a73f7f7

-- 
You are receiving this mail because:
You are on the CC 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 1559388] Review Request: dep - Go dependency management tool

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
dep-0.4.1-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-b31a1c55d5

-- 
You are receiving this mail because:
You are on the CC 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 1559249] Review Request: langpacks-install - Tool to get auto installed langpacks on GNOME session startup

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
langpacks-install-1.0.0-2.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-8c37954e73

-- 
You are receiving this mail because:
You are on the CC 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 1559786] Review Request: libvirt-dbus - This package provides integration between libvirt and the DBus

[bugzilla]

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

Cole Robinson  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |
  Flags||fedora-review+



--- Comment #9 from Cole Robinson  ---
Looks good to me, setting fedora-review+

I can sponsor Pavel, so removing NEEDSPONSOR

Admins, please also add virt-ma...@lists.fedoraproject.org to the package CC
list, that was pkgdb group::virtmaint-sig but I have no idea if that changed in
pagure world


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1540335] Review Request: primesieve - Fast C/ C++ prime number generator

[bugzilla]

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



--- Comment #8 from Kim Walisch  ---
I have released primesieve-6.4 on GitHub yesterday. Today I have updated the
primesieve spec file and corresponding SRPM package to version 6.4:

Spec URL:
https://raw.githubusercontent.com/kimwalisch/primesieve-rpm/v6.4-5/primesieve.spec
SRPM URL:
https://raw.githubusercontent.com/kimwalisch/primesieve-rpm/v6.4-5/primesieve-6.4-5.fc27.src.rpm

Hers's the link to the successful koji build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=25943211

I am still looking for a sponsor for the primesieve package. The development of
primesieve was started 8 years ago and it is now very mature i.e. primesieve
builds and runs successfully on every single CPU architecture supported by both
Fedora and Debian.

-- 
You are receiving this mail because:
You are on the CC 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 #20 from Neal Gompa  ---
Also, the spec and SRPM don't match, at least with the changelog, though it
looks like it's because you fixed the changelog entries in the spec...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
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 #19 from Neal Gompa  ---
The nodejs-yarn package has "npm()" Provides for all the bundled nodejs
modules, which would confuse and break things.

-- 
You are receiving this mail because:
You are on the CC 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 1424890] Review Request: domterm - terminal emulator based on web technologies

[bugzilla]

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



--- Comment #29 from Per Bothner  ---
Updated to deal with the problems you noted, plus some others.
The most important is that the Source0 URL is for a release tar-ball (as
produced
by 'make dist'), rather than a git snapshot:  The former includes "missing"
files added by 'make dist'.

http://per.bothner.com/tmp/domterm-0.99.2-1.fc27.src.rpm
http://per.bothner.com/tmp/domterm.spec

fedora-review reports some weird warnings/errors, including:

domterm.x86_64: W: only-non-binary-in-usr-lib

No idea what that means.

domterm.x86_64: E: zero-length /usr/share/domterm/help/domterm-attach.txt
(and more)

I don't know what this means - after rpm -i those files are installed but
they're not zero-length.

-- 
You are receiving this mail because:
You are on the CC 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 1559786] Review Request: libvirt-dbus - This package provides integration between libvirt and the DBus

[bugzilla]

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

Neal Gompa  changed:

   What|Removed |Added

 CC||ngomp...@gmail.com



--- Comment #8 from Neal Gompa  ---
(In reply to Iwicki Artur from comment #1)
> >%__make %{?_smp_mflags}
> Macro forms of system executables are discouraged.
> https://fedoraproject.org/wiki/Packaging:Guidelines#Macros
> 

Note, %make_build is preferred too.

-- 
You are receiving this mail because:
You are on the CC 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 1559786] Review Request: libvirt-dbus - This package provides integration between libvirt and the DBus

[bugzilla]

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



--- Comment #7 from Pavel Hrdina  ---
Thanks for the review, I've fixed all the issues and updated the files on
server, the links in Comment 1 should point to the fixed files.

-- 
You are receiving this mail because:
You are on the CC 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 1560111] Review Request: rubygem-crass - CSS parser based on the CSS Syntax Level 3 spec

[bugzilla]

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

Jaroslav Prokop  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||jar.pro...@volny.cz
   Assignee|nob...@fedoraproject.org|jar.pro...@volny.cz
  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jaroslav Prokop  ---
Seems good, 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 1288737] Review Request: openmeeg - Low-frequency bio-electromagnetism solving forward problems in the field of EEG and MEG

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

 CC||anto.tra...@gmail.com
   Assignee|nob...@fedoraproject.org|anto.tra...@gmail.com
  Flags||needinfo?(ignatenko@redhat.
   ||com)



--- Comment #9 from Antonio Trande  ---
@Igor

Are you still interested to OpenMEEG?
I'm preparing a new SRPM for reviewing...

-- 
You are receiving this mail because:
You are on the CC 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 1483339] Review Request: kiwi - A flexible operating system image builder

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |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 1483339] Review Request: kiwi - A flexible operating system image builder

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
kiwi-9.14.0-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-1e2a73f7f7

-- 
You are receiving this mail because:
You are on the CC 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