Roy Golan has posted comments on this change. Change subject: osinfo: Add 64-bit versions of linux systems. ......................................................................
Patch Set 1: hi and thanks for this patch. some issues to resolve: - if we want those OSs we need to add also relevant icon files to the frontend. (try git grep rhel_5 for example and see the results) - you added also linux_other_x64 as an OS (although its deriviated its still listed as an OS) maybe we should just convert all those OSs to 64 instead of 32? do they really have supported for 32 still? is it still relevant? @Omer you need to decide if adding those extra OSs is something we want. we can leave it as a file users can add to their setup meanwhile -- To view, visit http://gerrit.ovirt.org/35579 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5c0b01094854e7b0de03b885e2f85cc41e57128 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Zuzana Svetlíková <[email protected]> Gerrit-Reviewer: Martin Betak <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
