[virt-tools-list] [libosinfo] Match substring regex in the end

[Daniel P. Berrange]

On Mon, Feb 06, 2012 at 08:23:01PM +0100, Zeeshan Ali (Khattak) wrote:
> From: "Zeeshan Ali (Khattak)" <zeeshanak at gnome.org>
> 
> Sometimes volume-id expression of one architecture is a substring of
> volume-id expression of another architecture for the same OS. For
> example '.*G.*RMC.*' is volume-id expression for i386 media of win7,
> while '.*G.*RMC.*X.*' is for x86_64. To avoid incorrect matching, we
> need to ensure that matching is done against '.*G.*RMC.*X.*' first and
> then '.*G.*RMC.*'.
> ---
>  osinfo/osinfo_db.c |   20 ++++++++++++++++++++
>  1 files changed, 20 insertions(+), 0 deletions(-)
> 
> diff --git a/osinfo/osinfo_db.c b/osinfo/osinfo_db.c
> index 0e90f6e..b5ac506 100644
> --- a/osinfo/osinfo_db.c
> +++ b/osinfo/osinfo_db.c
> @@ -321,6 +321,24 @@ void osinfo_db_add_deployment(OsinfoDb *db, OsinfoDeployment *deployment)
>      osinfo_list_add(OSINFO_LIST(db->priv->deployments), OSINFO_ENTITY(deployment));
>  }
>  
> +static gint media_volume_compare (gconstpointer a, gconstpointer b)
> +{
> +    OsinfoMedia *media_a = OSINFO_MEDIA(a);
> +    OsinfoMedia *media_b = OSINFO_MEDIA(b);
> +    const gchar *volume_a = osinfo_media_get_volume_id(media_a);
> +    const gchar *volume_b = osinfo_media_get_volume_id(media_b);
> +
> +    if (volume_a == NULL || volume_b == NULL)
> +        /* Order doesn't matter then */
> +        return 0;
> +
> +    if (strstr(volume_a, volume_b) != NULL)
> +        return -1;
> +    else
> +        /* Sub-string comes later */
> +        return 1;
> +}
> +
>  /**
>   * osinfo_db_guess_os_from_media:
>   * @db: the database
> @@ -357,6 +375,8 @@ OsinfoOs *osinfo_db_guess_os_from_media(OsinfoDb *db,
>          GList *medias = osinfo_list_get_elements(OSINFO_LIST(media_list));
>          GList *media_iter;
>  
> +        medias = g_list_sort(medias, media_volume_compare);
> +
>          for (media_iter = medias; media_iter; media_iter = media_iter->next) {
>              OsinfoMedia *os_media = OSINFO_MEDIA(media_iter->data);
>              const gchar *os_volume = osinfo_media_get_volume_id(os_media);

ACK


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|