[Bug 13] Review request: rpmfusion-package-config-smart - RPM Fusion configuration files for the Smart package manager
http://bugzilla.rpmfusion.org/show_bug.cgi?id=13
--- Comment #3 from David Timms <dtimms(a)iinet.net.au> 2008-11-21 14:24:12 ---
(In reply to comment #2)
> rpmfusion-package-config-smart.i386: E: no-binary
I fixed no-documentation. but no-binary is OK since we don't want one in the
first place.
OK.
> 2: should this package be noarch, since it has no executable ?
Smart doesn't handle multiarch very well, so it's best to keep this
arch-dependant so that users can install the i386 and x86_64 versions if they
wish to do so.
OK.
> [??] buildroot differs slightly from third most preferred
location.
Fixed
Correct. Now identical to form 2.
> [??] Provides: smart-config-rpmfusion. Is this a previous
package that you
> are trying to supersee ? Or why is this done ?
> If there is a smart config package for any of the three parent repos, then
> perhaps this should provide smart-config-[oldrepo], so that the old smart
> config is removed ?
I just copied this off of fedora-package-config-smart, it's a metaprovides for
other packages to use.
OK. makes sense.
> [!] Release is sane, but needs the %{?dist}
Because this is the repo release file, it's best to avoid %{?dist}.
Agreed.
> [??] The config presented in the smart config files refers to
URLs that don't
> exist. I assume these will get updated as rpmfusion nears public release
Yeah, we'll have to tweak those later ;)
"It's time"
I think we have final locations now.
Also:
[rpmfusion-free-updates-testing]: add disabled=yes as default ?
[rpmfusion-non-free-updates-testing]: add disabled=yes as default ?
Until we can put these in CVS, I have the individual sources here as
well:
http://downloads.diffingo.com/rpmfusion/rpmfusion-package-config-smart-9/
Can smart
handle mirrorlist's like yum ?
A quick look suggests not, which might limit smart download speed due to
hassling specific servers rather than in country rpmfusion mirrors etc.
If that is the case, perhaps it would be good to at least include multiple
baseurl entries from the repos mirror. Not that we want to re-issue the
-config-smart each time mirror are added/removed from the list. Would it be
possible for the postinstall to retrieve the mirrorlist and insert it as the
list of baseurl= ?
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.