Re: Feedback request: Possible patch for fastestmirror plugin

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

 



Great! Thanks for accepting the patch.

Kris

On Sat, Apr 24, 2010 at 1:33 PM, Luke Macken <lmacken@xxxxxxxxxx> wrote:
On Sat, 2010-04-24 at 11:50 +0200, Kris Ven wrote:
> Dear all,
>
> A while ago, I wrote a patch for the Yum fastestmirror plugin. It
> further builds upon the "exclude" functionality offered by the plugin.
> However, instead of blacklisting certain hosts in the mirror list, my
> patch allows to whitelist specific hosts by listing them in a new
> "include_only" setting in the config file. Another improvement over the
> original plugin is that it allows to use regular expressions in both the
> "exclude" and "include_only" settings for more powerful filtering.
>
> For example, by specifying "include_only=\.de$, \.fr$, \.ie$" in the
> fastestmirror.conf file, only mirrors that end in the specified TLDs are
> considered for downloading. If no mirrors match the given search string
> in the include_only setting, then all mirrors are used. If the
> "include_only" setting is not used (i.e., the setting is outcommented in
> the config file), it defaults to the original behavior by processing the
> "exclude" setting.
>
> I developed this patch since I find it convenient to make use of a few
> selected mirrors that are known to perform well. This patch avoids
> having to blacklist a large number of mirrors. I included a diff against
> the current git repository in attachment.
>
> I have been using this patch for personal use for a while. However, I
> would like to donate this patch to the community if it may be of
> interest for other users as well.
>
> Any questions or comments with respect to this patch?

Hi Kris,

This patch makes sense and seems to nicely complement our 'exclude'
option.  I tested it locally and it works as expected, so I went ahead
and applied your patch upstream.

Thanks!

luke


_______________________________________________
Yum mailing list
Yum@xxxxxxxxxxxxxxxxx
http://lists.baseurl.org/mailman/listinfo/yum

_______________________________________________
Yum mailing list
Yum@xxxxxxxxxxxxxxxxx
http://lists.baseurl.org/mailman/listinfo/yum

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux