And while I'm nitpicking only, I think the shortlog could be shorter to follow the git guidelines. I'd go for "rhel: Add AArch64 ISO" but up to you. On Wed, May 6, 2015 at 1:51 PM, Zeeshan Ali (Khattak) <zeeshanak@xxxxxxxxx> wrote: > Hi Cole, > > On Tue, May 5, 2015 at 6:22 PM, Cole Robinson <crobinso@xxxxxxxxxx> wrote: >> RHELSA will be its own release for a short while, but long term it's >> expected to just be another RHEL architecture. So piggyback on the rhel7.0 >> entry for now. If needed we can add a toplevel rhelsa entry later. >> >> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx> > > Patch looks good. Just one nit-pick below. > >> diff --git a/data/schemas/libosinfo.rng b/data/schemas/libosinfo.rng >> index db6e145..a129745 100644 >> --- a/data/schemas/libosinfo.rng >> +++ b/data/schemas/libosinfo.rng >> @@ -700,6 +700,7 @@ >> </define> >> <define name='archnames'> >> <choice> >> + <value>aarch64</value> > > I think this should be a separate patch. > > > -- > Regards, > > Zeeshan Ali (Khattak) > ________________________________________ > Befriend GNOME: http://www.gnome.org/friends/ -- Regards, Zeeshan Ali (Khattak) ________________________________________ Befriend GNOME: http://www.gnome.org/friends/ _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo
