[Bug 1186889] Review Request: geoipupdate - Update GeoIP2 and GeoIP Legacy binary databases from MaxMind

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #30 from Paul Howarth <paul@xxxxxxxxxxxx> ---
Requires:    %{name}%{?_isa} = %{version}-%{release}, crontabs(In reply to
Philip Prindeville from comment #29)
> (In reply to Paul Howarth from comment #28)
> > You can't have noarch sub-packages with %{_isa}-based dependencies, as a
> > noarch package is not arch-specific.
> 
> Wasn't point of comment #13 that the %{?_isa} was required?
> 
> [!]: Fully versioned dependency in subpackages if applicable.
>      Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
>      geoipupdate-cron

The cron package is noarch and must not have an arch-specific dependency.
The resulting package must be the same regardless of which arch it's built on,
which can't happen if there's an arch-specific dependency there. If guess that
comes under the "if applicable" part of the guideline.

> > I'm glad it failed though, because I think the IPv6 updates/cron issue needs
> > to be sorted first (comment #24).
> 
> Yeah, I'm still thinking about that issue.  I didn't like having a separate
> lastmod.pl script so I'm thinking of writing a Perl script that handles
> everything.
> 
> I'll try to get it done this week.

Please do that before posting any updates for non-Rawhide releases. Let's get
this update right first time.

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





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]