[Bug 1336159] Review Request: golang-github-rubyist-tracerx - Output tracing information in your Go app based on environment variables

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

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



--- Comment #3 from Elliott Sales de Andrade  ---
So is this going to get pushed into Rawhide any time soon?

-- 
You are receiving this mail because:
You are on the CC 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 1336166] Review Request: golang-github-technoweenie-assert - Asserts to Go testomg

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

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



--- Comment #2 from Elliott Sales de Andrade  ---
The bmizerany repo has a note that it's unmaintained and the Go testing package
should be used instead. I don't know what that means for the technoweenie fork.

It appears to me that git-lfs is using https://github.com/stretchr/testify now,
so this package might not be needed.

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

[Bug 1421346] Review Request: extracturl - perl script for url extraction

[bugzilla]

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



--- Comment #1 from klaatu  ---
Spec URL: https://klaatu.fedorapeople.org/extracturl/extracturl.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 1421413] New: Review Request: sysusage - System monitoring based on perl, rrdtool, and sysstat

[bugzilla]

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

Bug ID: 1421413
   Summary: Review Request: sysusage - System monitoring based on
perl, rrdtool, and sysstat
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fr...@crawford.emu.id.au
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fedorapeople.org/~frankcrawford/sysusage.spec

SRPM URL: https://fedorapeople.org/~frankcrawford/sysusage-5.5-1.fc26.src.rpm

Description: SysUsage is a system monitoring and alarm reporting tool. 
It can generate historical graph views of CPU, memory, IO,
network and disk usage, and very much more.

Fedora Account System Username: frankcrawford

This is a request to re-review the package to being it back from retirement as
I am willing to over as the maintainer.  It was previously retired from F24.

-- 
You are receiving this mail because:
You are on the CC 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 1421406] New: Review Request: python-libtmux - Scripting library for tmux

[bugzilla]

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

Bug ID: 1421406
   Summary: Review Request: python-libtmux - Scripting library for
tmux
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rob...@electronsweatshop.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fedorapeople.org/~robled/python-libtmux.spec

SRPM URL: https://fedorapeople.org/~robled/python-libtmux-0.6.0-1.fc25.src.rpm

Description: libtmux is the tool behind tmuxp, a tmux workspace manager in
python.  It builds upon tmux' target and formats to create an object mappings
to traverse, inspect and interact with live tmux sesssions.

Fedora Account System Username: robled

-- 
You are receiving this mail because:
You are on the CC 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 1419330] Review Request: deepin-menu - deepin menu service

[bugzilla]

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



--- Comment #1 from Robin Lee  ---
I am not a sponsor. Here is my package review, but I am not able to approve it.
You should fix the failed items after all.

Package Review
==

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


Issues:
===
- Package contains BR: python2-devel or python3-devel
- Package installs a %{name}.desktop using desktop-file-install or desktop-
  file-validate if there is such a file.


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[-]: 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.
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "GPL (v3 or later)", "Unknown or generated". 25 files have
 unknown license. Detailed output of licensecheck in
 /home/cheese/Downloads/temp/deepin-menu/licensecheck.txt
[-]: 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/dbus-1,
 /usr/share/dbus-1/services
[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.
[!]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: 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.
[!]: Requires correct, justified where necessary.
[!]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[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

Python:
[x]: Python eggs must not download any dependencies during the build
 process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[!]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see att

[Bug 1379165] Review Request: libslz - StateLess Zip

[bugzilla]

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



--- Comment #14 from Christoph Junghans  ---
(In reply to Dridi Boukelmoune from comment #13)
> Spec URL: https://dridi.fedorapeople.org/review/libslz.spec
> SRPM URL: https://dridi.fedorapeople.org/review/libslz-1.1.0-1.fc25.src.rpm
> 
> Hello,
> 
> Sorry for the long delay, I'm back with an updated spec. Please note that
> I'm not disabling strip and that only programs are effectively stripped.
I would leave the stripping to rpmbuild for a better debug info and run make
install with STRIP=/bin/true as suggested in the guideline:

Or do you have a special reason to strip them yourself?

Second minor nitpick, submit your patch upstream (if you haven't done it
already) and make a comment in the spec.

Otherwise here is the review:

Package Review
==

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



= MUST items =

C/C++:
[-]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

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". 12 files have
 unknown license. Detailed output of licensecheck in /travis/libslz
 /review-libslz/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[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.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[!]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: 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 must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: If the source p

[Bug 1415311] Review Request: elementary-theme - elementary GTK+ Stylesheet

[bugzilla]

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

Fabio Valentini  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2017-02-11 16:22:21



-- 
You are receiving this mail because:
You are on the CC 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 1379174] Review Request: vmod-querystring - QueryString module for Varnish Cache

[bugzilla]

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



--- Comment #5 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/vmod-querystring

-- 
You are receiving this mail because:
You are on the CC 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 1419272] Review Request: ck - Library for high performance concurrent programming

[bugzilla]

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



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/ck

-- 
You are receiving this mail because:
You are on the CC 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 1379778] Review Request: python-diff-match-patch - Algorithms for synchronizing plain text

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
python-diff-match-patch-20121119-1.fc25 has been pushed to the Fedora 25
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-2017-5448ceeb3c

-- 
You are receiving this mail because:
You are on the CC 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 1421366] Review Request: MSearch - Mandatory library for MediathekView

[bugzilla]

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

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

[Bug 1379778] Review Request: python-diff-match-patch - Algorithms for synchronizing plain text

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
python-diff-match-patch-20121119-1.fc24 has been pushed to the Fedora 24
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-2017-33ab2cbca7

-- 
You are receiving this mail because:
You are on the CC 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 1421366] Review Request: MSearch - Mandatory library for MediathekView

[bugzilla]

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

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

[Bug 1421366] Review Request: MSearch - Mandatory library for MediathekView

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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 1420953] Container Review Request: redis - A persistent key-value database

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #2 from gil cattaneo  ---
is available @ https://admin.fedoraproject.org/pkgdb/package/rpms/redis/

-- 
You are receiving this mail because:
You are on the CC 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 1418208] Review Request: perl-Class-Std-Fast - Faster but less secure replacement for Class::Std

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
perl-Class-Std-Fast-0.0.8-2.el7 has been pushed to the Fedora EPEL 7 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-EPEL-2017-963e4ee3c5

-- 
You are receiving this mail because:
You are on the CC 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 1421366] Review Request: MSearch - Mandatory library for MediathekView

[bugzilla]

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

MartinKG  changed:

   What|Removed |Added

 CC||punto...@libero.it
 Depends On||1145303




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1145303
[Bug 1145303] RFE: JavaFX
-- 
You are receiving this mail because:
You are on the CC 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 1421366] New: Review Request: MSearch - Mandatory library for MediathekView

[bugzilla]

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

Bug ID: 1421366
   Summary: Review Request: MSearch - Mandatory library for
MediathekView
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mgans...@alice.de
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/MSearch.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/MSearch-2.1.1-2.fc25.src.rpm
Description: Mandatory library for MediathekView
Fedora Account System Username: martinkg

MSearch depends on openjfx https://bugzilla.redhat.com/show_bug.cgi?id=1145303
For MSearch to be built, the shared object and jar files must be symlinked into
somehwere where the JDK/JRE can find it.

#!/bin/bash

openjfx_dir=/usr/lib/jvm/openjfx

jdk_files=$(find $openjfx_dir/{bin,lib} -type f -print)
for file in $jdk_files; do
  relative=${file#$openjfx_dir/}
  ln -sfv "$openjfx_dir/$relative" "/usr/lib/jvm/java/$relative"
done

jre_files=$(find $openjfx_dir/rt -type f -print)
for file in $jre_files; do
  relative=${file#$openjfx_dir/rt/}
  ln -sfv "$openjfx_dir/rt/$relative" "/usr/lib/jvm/jre/$relative"
done


%changelog
* Fri Feb 10 2017 Martin Gansser  - 2.1.1-2
- delete *.jar files
- enable javadoc
- fix groupId
- fix non ASCII chars
- add subpkg javadoc
- add %%license macro to javadoc subpkg

* Thu Feb 09 2017 Martin Gansser  - 2.1.1-1
- Initial build

-- 
You are receiving this mail because:
You are on the CC 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 1409363] Review Request: llvm3.9 - Versioned LLVM 3.9

[bugzilla]

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



--- Comment #22 from Milan Bouchet-Valat  ---
I don't like the idea of putting everything under /lib64/llvm3.9. That probably
doesn't follow the FHS, even if we add symlinks. It took me quite some work to
improve LLVM's CMake build system, so why not make Clang use that? All the
required support is there.

For example, it looks like Clang should use LLVM_CMAKE_DIR to find where CMake
files have been installed. If you need to get that information from llvm-config
instead, we can add an argument for it. Or you could just hardcode it since
it's not going to change and the LLVM version is hardcoded 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 1421358] New: Review Request: js-jquery-noty - A jQuery notification plugin

[bugzilla]

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

Bug ID: 1421358
   Summary: Review Request: js-jquery-noty - A jQuery notification
plugin
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ra...@electronsweatshop.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://bowlofeggs.fedorapeople.org/js-jquery-noty.spec
SRPM URL:
https://bowlofeggs.fedorapeople.org/js-jquery-noty-2.3.11-1.fc26.src.rpm
Description: This is a jQuery notification plugin that makes it easy to create
alert - success - error - warning - information - confirmation messages as an
alternative the standard alert dialog.
Fedora Account System Username: bowlofeggs

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

[Bug 1420605] Review Request: nodejs-grunt-wrap - A Grunt plugin for wrapping project text files

[bugzilla]

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

Randy Barlow  changed:

   What|Removed |Added

 Blocks||1421358




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1421358
[Bug 1421358] Review Request: js-jquery-noty - A jQuery notification plugin
-- 
You are receiving this mail because:
You are on the CC 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 1421358] Review Request: js-jquery-noty - A jQuery notification plugin

[bugzilla]

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

Randy Barlow  changed:

   What|Removed |Added

 Depends On||1420605




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1420605
[Bug 1420605] Review Request: nodejs-grunt-wrap - A Grunt plugin for
wrapping project text 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 1379165] Review Request: libslz - StateLess Zip

[bugzilla]

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



--- Comment #13 from Dridi Boukelmoune  ---
Spec URL: https://dridi.fedorapeople.org/review/libslz.spec
SRPM URL: https://dridi.fedorapeople.org/review/libslz-1.1.0-1.fc25.src.rpm

Hello,

Sorry for the long delay, I'm back with an updated spec. Please note that I'm
not disabling strip and that only programs are effectively stripped.

-- 
You are receiving this mail because:
You are on the CC 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 1385237] Review Request: python-pythonz-bd - Manage python installations in your system

[bugzilla]

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



--- Comment #3 from Mathieu Bridon  ---
New upstream version.

I've remove the OS X patches as requested.

As for the /usr/lib/python3.6/site-packages/pythonz/etc stuff:

- one of the file is an example bashrc, which doesn't really have its place in
/etc, except maybe in /etc/skel, but... no. :)

- one of the file is the equivalent of bashrc for the fish shell, which I also
don't think should end up in /etc.

- the third one is a bash completion file, which I've installed into
/usr/share/bash-completion/completions/ with this new package.

Spec URL: https://bochecha.fedorapeople.org/packages/python-pythonz-bd.spec
Spec URL:
https://bochecha.fedorapeople.org/packages/python-pythonz-bd-1.11.4-1.fc25.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 1421346] New: Review Request: extracturl - perl script for url extraction

[bugzilla]

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

Bug ID: 1421346
   Summary: Review Request: extracturl - perl script for url
extraction
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: kla...@straightedgelinux.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: extracturl.spec
SRPM URL:
https://klaatu.fedorapeople.org/extracturl/extracturl-1.6.1-1.fc25.src.rpm
Description: A Perl script that extracts URLs from correctly-encoded MIME email
messages or plain text. This can be used either as a pre-parser for urlview, or
to replace urlview entirely, notably in the mutt email client.
Fedora Account System Username: klaatu

Apparently there was a package for this, but it appeoars to have been
abandoned:
https://bugzilla.redhat.com/show_bug.cgi?id=1323471

Successful Koji build --scratch: 
https://koji.fedoraproject.org/koji/taskinfo?taskID=17765361

Not my first RPM, but my first Fedora package. Need 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 1409884] Review Request: dayjournal - Digital journal that uses plain text files

[bugzilla]

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



--- Comment #28 from Fedora Update System  ---
dayjournal-23.0.6-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-a15061e4e9

-- 
You are receiving this mail because:
You are on the CC 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 1409884] Review Request: dayjournal - Digital journal that uses plain text files

[bugzilla]

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



--- Comment #27 from Fedora Update System  ---
dayjournal-23.0.6-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-4ad8a1bcc8

-- 
You are receiving this mail because:
You are on the CC 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 1409884] Review Request: dayjournal - Digital journal that uses plain text files

[bugzilla]

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

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 1409884] Review Request: dayjournal - Digital journal that uses plain text files

[bugzilla]

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



--- Comment #26 from Fedora Update System  ---
dayjournal-23.0.6-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-19667c7540

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