Bug 463848 - easyfix: spec file portability
Summary: easyfix: spec file portability
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Fedora
Classification: Fedora
Component: cups-pdf
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Remi Collet
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-09-24 21:03 UTC by R P Herrold
Modified: 2008-09-25 20:27 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-09-25 20:27:19 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description R P Herrold 2008-09-24 21:03:07 UTC
Description of problem:

The spec file contains a incompletely specified conditional test.
It needs a leading '0' to avoid a failed compare; trifvial patch in teh Additional Information section



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

cups-pdf-2.4.8-1.fc10.src.rpm

How reproducible:

   87  cp cups-pdf.spec-ORIG cups-pdf.spec
   88  rpmbuild -ba  cups-pdf.spec


Actual results:

a failed build

Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
Checking for unpackaged file(s): /usr/lib/rpm/check-files /var/tmp/cups-pdf-2.4.8-1-root-herrold
error: Installed (but unpackaged) file(s) found:
   /etc/cups/cups-pdf.conf
   /usr/lib/cups/backend/cups-pdf
   /usr/share/cups/model/CUPS-PDF.ppd


RPM build errors:
    /home/herrold/build/gnomint/cups-pdf.spec:149: parseExpressionBoolean returns -1
    Installed (but unpackaged) file(s) found:
   /etc/cups/cups-pdf.conf
   /usr/lib/cups/backend/cups-pdf
   /usr/share/cups/model/CUPS-PDF.ppd
[herrold@dhcp-233 gnomint]$


Expected results:

a clean build


Additional info:


patch inline:
[herrold@dhcp-233 gnomint]$ diff -u cups-pdf.spec-ORIG cups-pdf.spec
--- cups-pdf.spec-ORIG  2008-09-24 16:57:46.000000000 -0400
+++ cups-pdf.spec       2008-09-24 16:57:03.000000000 -0400
@@ -146,7 +146,7 @@
 # First install : create the printer if cupsd is running
 if [ "$1" -eq "1" -a -f "%{_var}/run/cupsd.pid" ]
 then
-%if %{?fedora}%{?rhel} < 5
+%if 0%{?fedora}%{?rhel} < 5
     # Restart CUPS on old Fedora/RHEL to discover the new "backend"
     /sbin/service cups condrestart
 %endif
[herrold@dhcp-233 gnomint]$

Comment 1 Remi Collet 2008-09-25 20:27:19 UTC
This issue doesn't affect Fedora nor Epel build.

I will fix it on a next build (new upstream release).


Note You need to log in before you can comment on or make changes to this bug.