[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #12 from Fedora Update System  ---
shrinkpdf-0-1.fc23 has been pushed to the Fedora 23 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-02-14 21:53:46



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
shrinkpdf-0-1.fc23 has been pushed to the Fedora 23 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-2016-98f34222a4

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #4 from Petr Pisar  ---
(In reply to Raphael Groner from comment #3)
> thanks for your hints. I do not see any blockers. Please approve the package.
> 
Good package is also mark of the reviewer and I will no pass package with these
simple to fix mistakes. Please correct the issues I marked as `FIX' and then I
will approve this package.

> Spelling errors
> They do not harm with packaging in general, therefore marked as warnings.
>
Though if spotted they should be fixed. Especially the "filesize" is not an
English word.

> coreutils
> That package can be assumed to be installed in every working system.
> Otherwise, we would have to add iit as dependency to every other packages as
> well. It's always present in a fresh mock build environment and so also on
> koji.

We have to add it to every package that needs it.
.
There is no exceptions list anymore.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #6 from Upstream Release Monitoring 
 ---
raphgro's scratch build of shrinkpdf-0-1.fc23.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12800827

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Petr Pisar  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #7 from Petr Pisar  ---
Spec file changes:

$ diff -u shrinkpdf.spec{.old,}
--- shrinkpdf.spec.old  2016-01-13 16:29:09.0 +0100
+++ shrinkpdf.spec  2016-02-03 11:58:37.0 +0100
@@ -12,13 +12,13 @@
 BuildArch:  noarch

 Requires:   ghostscript-core
+Requires:   coreutils

 %description
 A simple wrapper around Ghostscript to shrink PDFs (as in reduce
-filesize) under Linux. Inspired by some code found in an OpenOffice
-Python script (I think). The script feeds a PDF through Ghostscript,
-which performs lossy recompression by such methods as downsampling
-the images to 72dpi. The result should be (but not always is) a much
+file size) under Linux. The script feeds a PDF through Ghostscript,
+which performs lossy recompression by such methods own as downsampling
+the images to 72 DPI. The result should be (but not always is) a much
 smaller file.


> TODO: I would improve the description a little bit. I would spell "filesize"
> as two words "file size". I think the second sentence "Inspired by some code
> found in an OpenOffice Python script (I think)" is irrelevant and I would
> remove it. Also I think "72dpi" is spelled wrongly. I'd write it as "72 DPI".

TODO: I think an "own" word slipped into your text this time.

> FIX: Build-require "coreutils" (shrinkpdf.spec:30).
Not addressed. Please add the build-require.

> FIX: Run-require "coreutils" because the script executes "cp", "wc", and
> "cut" programs.
$ rpm -q --requires -p ../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm | sort -f |
uniq -c
  1 /bin/sh
  1 coreutils
  1 ghostscript-core
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

Package builds in F24
(http://koji.fedoraproject.org/koji/taskinfo?taskID=12802729). Ok.

Please add build-dependency on coreutils before building the package.
Resolution: 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #5 from Raphael Groner  ---
Fixed. Links are the same for SPEC and SRPM.

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=12800827

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
shrinkpdf-0-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-98f34222a4

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

 Status|ASSIGNED|POST



--- Comment #8 from Raphael Groner  ---
(In reply to Petr Pisar from comment #7)
> > FIX: Build-require "coreutils" (shrinkpdf.spec:30).
> Not addressed. Please add the build-require.

No. As bash (and mock installing bash, see below) depends on coreutils'
functionality and rpm package depends on it, we can guess a functional build
environment when rpm is installed by rpmbuild.
https://lists.fedoraproject.org/pipermail/devel/2015-June/211423.html


Thanks for 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



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

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #3 from Raphael Groner  ---
Hi Petr,

thanks for your hints. I do not see any blockers. Please approve the package.

Spelling errors
They do not harm with packaging in general, therefore marked as warnings.

coreutils
That package can be assumed to be installed in every working system. Otherwise,
we would have to add iit as dependency to every other packages as well. It's
always present in a fresh mock build environment and so also on koji.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Petr Pisar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ppi...@redhat.com
   Assignee|nob...@fedoraproject.org|ppi...@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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #2 from Petr Pisar  ---
URL and Source0 links are usable. Ok.
Source is original (SHA-256:
d68561bd674847d6b73961af571adf5720c9b525720afc399c3f7324d7af638f). Ok.
The summary is Ok.

TODO: I would improve the description a little bit. I would spell "filesize" as
two words "file size". I think the second sentence "Inspired by some code found
in an OpenOffice Python script (I think)" is irrelevant and I would remove it.
Also I think "72dpi" is spelled wrongly. I'd write it as "72 DPI".

License verified from shrinkpdf.sh. Ok.
No compiled code. noarch BuildArch is Ok.

FIX: Run-require "coreutils" because the script executes "cp", "wc", and "cut"
programs.

FIX: Build-require "coreutils" (shrinkpdf.spec:30).

$ rpmlint shrinkpdf.spec ../SRPMS/shrinkpdf-0-1.fc24.src.rpm
../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm 
shrinkpdf.src: W: spelling-error %description -l en_US filesize -> file size,
file-size, fissile
shrinkpdf.src: W: spelling-error %description -l en_US lossy -> loss, glossy,
flossy
shrinkpdf.src: W: spelling-error %description -l en_US recompression ->
decompression, re compression, re-compression
shrinkpdf.src: W: spelling-error %description -l en_US downsampling -> down
sampling, down-sampling, oversampling
shrinkpdf.noarch: W: spelling-error %description -l en_US filesize -> file
size, file-size, fissile
shrinkpdf.noarch: W: spelling-error %description -l en_US lossy -> loss,
glossy, flossy
shrinkpdf.noarch: W: spelling-error %description -l en_US recompression ->
decompression, re compression, re-compression
shrinkpdf.noarch: W: spelling-error %description -l en_US downsampling -> down
sampling, down-sampling, oversampling
shrinkpdf.noarch: W: no-documentation
shrinkpdf.noarch: W: no-manual-page-for-binary shrinkpdf
2 packages and 1 specfiles checked; 0 errors, 10 warnings.

FIX: Correct the "filesize" spelling.

$ rpm -q -lv -p ../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm 
-rwxr-xr-x1 rootroot 2979 Feb  2 16:17
/usr/bin/shrinkpdf
File permission and layout is Ok.

$ rpm -q --requires -p ../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm | sort -f |
uniq -c
  1 /bin/sh
  1 ghostscript-core
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
FIX: Run-require "coreutils".

$ rpm -q --provides -p ../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm | sort -f |
uniq -c
  1 shrinkpdf = 0-1.fc24
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/shrinkpdf-0-1.fc24.noarch.rpm 
Binary dependencies resolvable. Ok.

Package builds in F24
(http://koji.fedoraproject.org/koji/taskinfo?taskID=12788562). Ok.

Otherwise the package is in line with Fedora packaging guidelines.

Please correct all `FIX' items, consider fixing `TODO' items, and provide a new 
spec file.
Resolution: Package NOT 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

  Alias||shrinkpdf



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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



--- Comment #1 from Upstream Release Monitoring 
 ---
raphgro's scratch build of shrinkpdf-0-1.fc23.src.rpm for rawhide completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12534764

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1298251] Review Request: shrinkpdf - Simple wrapper around Ghostscript to shrink PDFs

[bugzilla]

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

Raphael Groner  changed:

   What|Removed |Added

 Whiteboard||Trivial



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review