[Bug 173053] Review Request: perl-Readonly-XS
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Readonly-XS
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=173053
paul(a)city-fan.org changed:
What |Removed |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778 |163779
nThis| |
------- Additional Comments From paul(a)city-fan.org 2005-12-08 13:49 EST -------
- Builds fine in mock (FC4 i386)
- Review issues addressed
Strictly speaking the buildreq should be perl(Readonly) >= 1.02 (see
Makefile.PL) but it doesn't really matter as no older version has ever been
released in Extras.
Approved.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
18 years, 4 months
[Bug 172677] Review Request: perl-Readonly
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Readonly
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172677
------- Additional Comments From mpeters(a)mac.com 2005-12-08 13:34 EST -------
(In reply to comment #14)
> Looks good. Approved.
>
> You should change the %{?_smp_mflags} in the changelog to %%{?_smp_mflags}
> though - try "rpm -q --changelog perl-Readonly-1.03-4.src.rpm" to see why. You
> can do this post-cvs-import if you prefer.
>
Oh yes - definitely. My bad - I know about that.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
18 years, 4 months
[Bug 172677] Review Request: perl-Readonly
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Readonly
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=172677
paul(a)city-fan.org changed:
What |Removed |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778 |163779
nThis| |
------- Additional Comments From paul(a)city-fan.org 2005-12-08 13:20 EST -------
Looks good. Approved.
You should change the %{?_smp_mflags} in the changelog to %%{?_smp_mflags}
though - try "rpm -q --changelog perl-Readonly-1.03-4.src.rpm" to see why. You
can do this post-cvs-import if you prefer.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
18 years, 4 months