On 14/11/14 13:23, Martin Lucina wrote: > I have added some commits to the PR: > > - Renamed "xr" to "rumprun-xen".
I'd still like to see it called "rumprun" and e.g. take a currently mandatory "-t xen" parameter. That way the intent for the tool to be a universal frontend is clear from the get-go. > - Added -lrumpnet_config -lrumpnet_net -lrumpnet to > app-tools/specs.configure.in, needed by the rumpconfig module. I don't understand this change. Why does configure need to know about it, and especially why does it need to be hardcoded in specs? Reading the diff: Not sure if ":" is the easiest-to-parse delimiter for your netspec. > Given that no one has spoken up with "no, no, this is all wrong!", this > looks like a good starting point to merge? > > I would consider the "rumprun-xen" script status as "playground, in flux" > for now; there will be more changes which may include a review of the > syntax if we find that the current one is not sufficient for everything we > want to do with it, or if we decide to reimplement it in something less > hairy than shell :-) Sure, "in flux" mode is fine. There's prior art for the syntax required of such a tool, so surveying the prior art instead of going straight for reinventing the wheel might be the best approach. ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk _______________________________________________ rumpkernel-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/rumpkernel-users
