On Tue, Feb 25, 2014 at 01:19:05PM +0100, Vratislav Podzimek wrote:
On Tue, 2014-02-25 at 04:30 -0500, Samantha N. Bueno wrote:
This includes fixes suggested from the first review. The flow of this is also a bit different from when I first posted as well, thanks to discussions with dlehman and mizmo (read: simpler and less clunky). The text install is basically the same, but for an updated screencast of the graphical version here's a link: http://sbueno.fedorapeople.org/1001070-graphical-25feb.webm
jstodola mentioned the "zerombr" kickstart command last time; support for that is here; if "zerombr" is in a ks file and unformatted DASDs are specified, they automatically have dasdfmt run against them.
A few points that are sort of minor but which bug me, all graphical:
- The link on the dialog box back to the summary hub doesn't work, so I sort of force it myself in what's probably a less correct way. I'm not sure what format the URI should be, so if anyone knows, let me know.
I'm afraid the way you do it is the only working one.
Eh, well that's fine. Just figured there's likely another way.
- Again following a return to the summary hub, while letting dasdfmt run in the background, the UI sort of freezes up; I'm guessing I need some sort of decorator somewhere, maybe?
Does it freeze for the time DASDs being formated?
Yes.
- I think my thread handling is jumbled and awkward in places (GUI storage), then again trying to cleanly work in the "zerombr" command was no picnic. I'd be greatly in favor of only allowing "zerombr" as valid if in text mode; the graphical code is hideous IMO.
I think it's not that bad even in the GUI code. :)
Ah you should've seen it before I'd trimmed things and squashed tmp patches. ;) The graphical "zerombr" stuff is still not perfect with the status notifications though so I'll likely re-post that once I figure it out.
Samantha
-- Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
anaconda-patches mailing list anaconda-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches