[Bug 497827] New: Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: shortrpm - Short-circuit binary RPM package build

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

Summary: Review Request: shortrpm - Short-circuit binary RPM
                    package build
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: low
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail], [e-mail]
   Estimated Hours: 0.0
    Classification: Fedora

SPEC: http://v3.sk/~lkundrak/SPECS/shortrpm.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/shortrpm-1.0-1.fc11.src.rpm

Description:

shortrpm tricks rpmbuild(1) into thinking certain scriptlets in SPEC file
are empty, effectively skipping their run and proceeding to build the binary
package from files from previous %%install scriptlet run in BuildRoot. The
trickery only takes place if both -bb and --short-circuit options are
specified.

[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

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

Mamoru Tasaka  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review+

--- Comment #1 from Mamoru Tasaka   2009-04-27 10:17:08 EDT ---
Assiging to myself as I definitely want this package :)
I tried with cairo-dock.spec and this actually works!

- Please specify full URL for Source:
  https://fedoraproject.org/wiki/Packaging/SourceURL

Please fix above before importing.
-------------------------------------------------------------
      This package (shortrpm) is APPROVED by mtasaka
-------------------------------------------------------------

[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

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

Lubomir Rintel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

--- Comment #2 from Lubomir Rintel   2009-04-27 16:17:03 EDT ---
(In reply to comment #1)
> Please specify full URL for Source:
> https://fedoraproject.org/wiki/Packaging/SourceURL
>
> Please fix above before importing.

Will do. Thanks a lot for review!

New Package CVS Request
=======================
Package Name: shortrpm
Short Description: Short-circuit binary RPM package build
Owners: lkundrak
Branches: EL-5 F-10

[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

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

Kevin Fenzi  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

--- Comment #3 from Kevin Fenzi   2009-04-28 23:16:06 EDT ---
cvs done (with F11 branch added).

[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

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

Lubomir Rintel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

--- Comment #4 from Lubomir Rintel   2009-04-29 03:24:07 EDT ---
Hm, testsuite was failing since %a seems to be ignored with most recent glibcs
even when the source is not an ISO C source, contrary to what was documented.
The package was updated to 1.1, imported and built.

[Bug 497827] Review Request: shortrpm - Short-circuit binary RPM package build

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

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

--- Comment #5 from Lubomir Rintel   2009-05-01 06:21:57 EDT ---
*** Bug 478463 has been marked as a duplicate of this bug. ***