Re: [PATCH v2] win7: Put pro n sp1 regex under correct architecture

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

 



On Thu, Apr 24, 2014 at 2:05 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
>
>
> ----- Mail original -----
>> In commit 39f22d2 (win7: Generalize volume-id for professional), I
>> generalized the regex of windows 7 professional to include N SP1 but I
>> ended up adding the regex for both i386 and x86_64 architectures under
>> i386.
>>
>> This patch fixes that.
>>
>
> On second though, I'd probably not use a regex at all, but explicitly list every variant. Or we can start trying to decipher the label, I'm fairly sure all volume IDs can be generated using a few regexp.

I'd rather we go for the latter approach. We decided to use regex for a reason.


-- 
Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




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

  Powered by Linux