[Bug 1149566] Review Request: slurm - Simple LinUx Resource Manager

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1149566



--- Comment #30 from Adam Huffman  ---
Hi David,

Could you update to 15.08.11?

Thanks,
Adam

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1326277] Review Request: ocitools - Collection of tools for working with the OCI runtime specification

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1326277



--- Comment #5 from Lokesh Mandvekar  ---
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: "Apache (v2.0)", "MIT/X11 (BSD like)", "Unknown or generated".
 83 files have unknown license. Detailed output of licensecheck in
 /home/lsm5/1326277-ocitools/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[!]: Package requires other packages for directories it uses.
 Note: No known owner of
 /usr/share/gocode/src/github.com/opencontainers/ocitools/cmd

--- gotta own this one

[!]: Package must own all directories that it creates.
 Note: Directories without known owners:
 /usr/share/gocode/src/github.com/opencontainers/ocitools/cmd,
 /usr/share/gocode/src, /usr/share/gocode,
 /usr/share/gocode/src/github.com

--- /usr/share/gocode{/src{/github.com}} should be owned by golang-devel
package I think.
I'm thinking we can waive this and file a bug against the main golang package
itself.



[!]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by:
 /usr/share/gocode/src/github.com/opencontainers(golang-github-
 opencontainers-specs-devel, golang-github-opencontainers-runtime-spec-
 devel, runc-devel)

--- some upper level dirs are common to devel subpackages in other packages.

[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[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.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1326277] Review Request: ocitools - Collection of tools for working with the OCI runtime specification

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1326277

Lokesh Mandvekar  changed:

   What|Removed |Added

  Flags|needinfo?(l...@redhat.com)  |



--- Comment #4 from Lokesh Mandvekar  ---
This is a review *template*. Besides handling the [ ]-marked tests you are
also supposed to fix the template before pasting into bugzilla:
- Add issues you find to the list of issues on top. If there isn't such
  a list, create one.
- Add your own remarks to the template checks.
- Add new lines marked [!] or [?] when you discover new things not
  listed by fedora-review.
- Change or remove any text in the template which is plain wrong. In this
  case you could also file a bug against fedora-review
- Remove the "[ ] Manual check required", you will not have any such lines
  in what you paste.
- Remove attachments which you deem not really useful (the rpmlint
  ones are mandatory, though)
- Remove this text



Package Review
==

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



= MUST items =

Generic:
[ ]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[ ]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Apache (v2.0)", "MIT/X11 (BSD like)", "Unknown or generated".
 83 files have unknown license. Detailed output of licensecheck in
 /home/lsm5/1326277-ocitools/licensecheck.txt
[ ]: License file installed when any subpackage combination is installed.
[ ]: Package requires other packages for directories it uses.
 Note: No known owner of
 /usr/share/gocode/src/github.com/opencontainers/ocitools/cmd
[ ]: Package must own all directories that it creates.
 Note: Directories without known owners:
 /usr/share/gocode/src/github.com/opencontainers/ocitools/cmd,
 /usr/share/gocode/src, /usr/share/gocode,
 /usr/share/gocode/src/github.com
[ ]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by:
 /usr/share/gocode/src/github.com/opencontainers(golang-github-
 opencontainers-specs-devel, golang-github-opencontainers-runtime-spec-
 devel, runc-devel)
[ ]: %build honors applicable compiler flags or justifies otherwise.
[ ]: Package contains no bundled libraries without FPC exception.
[ ]: Changelog in prescribed format.
[ ]: 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
[ ]: Package uses nothing in %doc for runtime.
[ ]: Package consistently uses macros (instead of hard-coded directory
 names).
[ ]: Package is named according to the Package Naming Guidelines.
[ ]: Package does not generate any conflict.
[ ]: Package obeys FHS, except libexecdir and /usr/target.
[ ]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[ ]: Requires correct, justified where necessary.
[ ]: Spec file is legible and written in American English.
[ ]: Package contains systemd file(s) if in need.
[ ]: Useful -debuginfo package or justification otherwise.
[ ]: Package is not known to require an ExcludeArch tag.
[ ]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[ ]: 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]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: 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 mu

[Bug 1336255] Review Request: vimwiki - A personal wiki For Vim

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336255

Brad Hubbard  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336255] New: Review Request: vimwiki - A personal wiki For Vim

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336255

Bug ID: 1336255
   Summary: Review Request: vimwiki - A personal wiki For Vim
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: bhubb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://people.redhat.com/bhubbard/vimwiki.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/badone/vimwiki/fedora-23-x86_64/00226318-vimwiki/vimwiki-2.3-1.fc23.src.rpm
Description: First package, looking for a sponsor.

Vimwiki is a personal wiki for Vim
-a number of linked text files that have their own syntax highlighting.

With Vimwiki you can:

-organize notes and ideas
-manage to do-lists
-write documentation
-maintain a diary

Features:

-three markup syntaxes supported: Vimwiki's own syntax, Markdown, MediaWiki
-export everything to HTML
-link to other wiki pages and external files
-search through all wiki pages
-outline notes and tasks in indented lists
-quickly manipulate numbered and bulleted lists
-tag wiki pages or arbitrary places and quickly jump to tags
-tables

Fedora Account System Username: badone

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=14095996

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336218] Review Request: golang-github-cockroachdb-cmux - Connection mux for serving different services on the same port

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336218

Lokesh Mandvekar  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid
- only sources installed.

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336217] Review Request: golang-github-dustin-go-humanize - Formatters for units to human friendly sizes

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336217

Lokesh Mandvekar  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid
- only sources installed.

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336141] Review Request: lxqt-config-randr - GUI interface to RandR extension

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336141



--- Comment #5 from Kevin Kofler  ---
Please note that the linked Qt review will NOT make lxqt-config-randr compile.
It actually REMOVES the last remains of 2 long-dead functions that were already
not actually implemented (and even the declaration was not built, it was
#ifdef-ed out). Even reverting that commit (which was apparently merged in some
form, you are looking at the patch backwards) will not fix the problem, because
the functions are only declared and not actually implemented.

lxqt-config-randr needs to stop using that obsolete API instead. There is no
other fix.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1326875] Review Request: keepassx2 - Cross-platform password manager

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1326875



--- Comment #7 from Mukundan Ragavan  ---
Just these two issues as mentioned earlier -

Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://fedoraproject.org/wiki/Packaging:Guidelines

---> This can be ignored. Local issue.

- Package does not contain duplicates in %files.
  Note: warning: File listed twice:
  /usr/share/keepassx2/translations/keepassx_cs.qm
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#DuplicateFiles


---> I think this is because of %{_datadir}/keepassx2 and correct usage of
%find_lang macro.

The %files section could be more descriptive to avoid these clashes.

- Package installs a %{name}.desktop using desktop-file-install or desktop-
  file-validate if there is such a file.


---> Yes, this does seem to be missing (not present in post or posttrans).
Please fix 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1203380] Review Request: gherkin2-java - A fast lexer and parser for the Gherkin language based on Ragel

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1203380

gil cattaneo  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |gherkin-java - A fast lexer |gherkin2-java - A fast
   |and parser for the Gherkin  |lexer and parser for the
   |language based on Ragel |Gherkin language based on
   ||Ragel



--- Comment #7 from gil cattaneo  ---
Spec URL: https://gil.fedorapeople.org/gherkin2-java.spec
SRPM URL: https://gil.fedorapeople.org/gherkin2-java-2.12.2-2.fc23.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336141] Review Request: lxqt-config-randr - GUI interface to RandR extension

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336141

Raphael Groner  changed:

   What|Removed |Added

External Bug ID||Qt Bug Tracker QTBUG-29333



--- Comment #4 from Raphael Groner  ---
FTR, Qt5 API issue may be related to 
https://codereview.qt-project.org/#/c/105285/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336218] Review Request: golang-github-cockroachdb-cmux - Connection mux for serving different services on the same port

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336218

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336217] Review Request: golang-github-dustin-go-humanize - Formatters for units to human friendly sizes

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336217

Jan Chaloupka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336217] New: Review Request: golang-github-dustin-go-humanize - Formatters for units to human friendly sizes

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336217

Bug ID: 1336217
   Summary: Review Request: golang-github-dustin-go-humanize -
Formatters for units to human friendly sizes
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: jchal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-dustin-go-humanize/golang-github-dustin-go-humanize.spec

SRPM URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-dustin-go-humanize/golang-github-dustin-go-humanize-0-0.1.git8929fe9.fc20.src.rpm

Description: Formatters for units to human friendly sizes

Fedora Account System Username: jchaloup

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=14089966

$ rpmlint golang-github-dustin-go-humanize-0-0.1.git8929fe9.fc20.src.rpm
golang-github-dustin-go-humanize-devel-0-0.1.git8929fe9.fc20.noarch.rpm
golang-github-dustin-go-humanize-unit-test-devel-0-0.1.git8929fe9.fc20.x86_64.rpm
golang-github-dustin-go-humanize.src: W: spelling-error Summary(en_US)
Formatters -> For matters, For-matters, Formatted
golang-github-dustin-go-humanize.src: W: spelling-error %description -l en_US
Formatters -> For matters, For-matters, Formatted
golang-github-dustin-go-humanize-devel.noarch: W: spelling-error Summary(en_US)
Formatters -> For matters, For-matters, Formatted
golang-github-dustin-go-humanize-devel.noarch: W: spelling-error %description
-l en_US Formatters -> For matters, For-matters, Formatted
3 packages and 0 specfiles checked; 0 errors, 4 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336217] Review Request: golang-github-dustin-go-humanize - Formatters for units to human friendly sizes

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336217

Jan Chaloupka  changed:

   What|Removed |Added

 Blocks||1333988




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1333988
[Bug 1333988] etcd-3.0.0-beta.0 is available
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336218] New: Review Request: golang-github-cockroachdb-cmux - Connection mux for serving different services on the same port

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336218

Bug ID: 1336218
   Summary: Review Request: golang-github-cockroachdb-cmux -
Connection mux for serving different services on the
same port
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: jchal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-cockroachdb-cmux/golang-github-cockroachdb-cmux.spec

SRPM URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-cockroachdb-cmux/golang-github-cockroachdb-cmux-0-0.1.git112f050.fc20.src.rpm

Description: Connection mux for serving different services on the same port

Fedora Account System Username: jchaloup

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=14090006

$ rpmlint golang-github-cockroachdb-cmux-0-0.1.git112f050.fc20.src.rpm
golang-github-cockroachdb-cmux-devel-0-0.1.git112f050.fc20.noarch.rpm
golang-github-cockroachdb-cmux-unit-test-devel-0-0.1.git112f050.fc20.x86_64.rpm
golang-github-cockroachdb-cmux.src: W: spelling-error Summary(en_US) mux ->
mix, mu, mus
golang-github-cockroachdb-cmux.src: W: spelling-error %description -l en_US mux
-> mix, mu, mus
golang-github-cockroachdb-cmux-devel.noarch: W: spelling-error Summary(en_US)
mux -> mix, mu, mus
golang-github-cockroachdb-cmux-devel.noarch: W: spelling-error %description -l
en_US mux -> mix, mu, mus
3 packages and 0 specfiles checked; 0 errors, 4 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336218] Review Request: golang-github-cockroachdb-cmux - Connection mux for serving different services on the same port

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336218

Jan Chaloupka  changed:

   What|Removed |Added

 Blocks||1333988




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1333988
[Bug 1333988] etcd-3.0.0-beta.0 is available
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1203380] Review Request: gherkin-java - A fast lexer and parser for the Gherkin language based on Ragel

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1203380



--- Comment #6 from Raphael Groner  ---
> Not sure about backward compatibility, and no, i am not interested @ the 
> moment
…
> moved @ https://github.com/cucumber/gherkin2/archive/v2.12.2.tar.gz

Hmm, maybe then better use gherkin2-java as the package name to use it for
compatibility?
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#MultiplePackages

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336141] Review Request: lxqt-config-randr - GUI interface to RandR extension

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336141

Raphael Groner  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
Summary|Review Request: |Review Request:
   |lxqt-config-randr -GUI  |lxqt-config-randr - GUI
   |interface to RandR  |interface to RandR
   |extension   |extension



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1333970] Review Request: plasma-applet-redshift-control - Plasma 5 applet for redshift

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1333970



--- Comment #6 from Fedora Update System  ---
plasma-applet-redshift-control-1.0.17-1.fc23 has been submitted as an update to
Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-97b9e0ad10

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1333970] Review Request: plasma-applet-redshift-control - Plasma 5 applet for redshift

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1333970

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1333970] Review Request: plasma-applet-redshift-control - Plasma 5 applet for redshift

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1333970



--- Comment #5 from Fedora Update System  ---
plasma-applet-redshift-control-1.0.17-1.fc24 has been submitted as an update to
Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-8198362676

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1333970] Review Request: plasma-applet-redshift-control - Plasma 5 applet for redshift

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1333970



--- Comment #7 from Fedora Update System  ---
plasma-applet-redshift-control-1.0.17-1.fc22 has been submitted as an update to
Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-b2e29dc70b

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1333970] Review Request: plasma-applet-redshift-control - Plasma 5 applet for redshift

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1333970



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/plasma-applet-redshift-control

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1336008] Review Request: keycloak-httpd-client-install - Tools to configure Apache HTTPD as Keycloak client

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1336008



--- Comment #1 from John Dennis  ---
I thought I would address up front what I suspect might be an issue that will
be raised during package review.

The name of the package is keycloak-httpd-client-install, but the package build
produces only 2 python packages, python2-keycloak-httpd-client-install and
python3-keycloak-httpd-client-install. One wonders if the package name
shouldn't be python-keycloak-httpd-client-install instead of just
keycloak-httpd-client-install without the python prefix.

Omitting the python prefix on the package name was a conscious and deliberate
choice. We expect the keycloak-httpd-client-install package to include more
tools down the road, especially components written in other languages (e.g.
java). Therefore there is nothing that makes the contents of this package
exclusively Python, it just happens that at this moment it only produces two
python subpackages just like any other package might also have a python
subpackage for it's Python components.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 976442] Review Request: bugwarrior - Sync github, bitbucket, and trac issues with taskwarrior

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=976442

Marcelo Barbosa "firemanxbr"  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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 976468] Review Request: rubygem-teamocil - Easy session, window and pane layouts for tmux

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=976468

Marcelo Barbosa "firemanxbr"  changed:

   What|Removed |Added

 CC||mr.marcelo.barb...@gmail.co
   ||m
  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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1291391] Review Request: nodejs-has-ansi - Check if a string has ANSI escape codes

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1291391

Jared Smith  changed:

   What|Removed |Added

  Flags|needinfo?(jsmith.fedora@gma |
   |il.com) |



--- Comment #12 from Jared Smith  ---
Pushed to epel7, f22, and f23.  I'm not going to push to EL6 or lower, as I
think there are still missing dependencies there.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1233240] Review Request: orocos-bfl - A framework for inference in Dynamic Bayesian Networks

2016-05-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1233240

Till Hofmann  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-05-11 21:29:55 |2016-05-15 05:00:09



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org