[email protected] said: > 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.
Done. I used "rumprun STACK ..." rather than "rumprun -t STACK" as that saves us an extra option character and STACK is not optional. > > - 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? (After extensive discussion on IRC) This is due to the interdependencies between minios.o, rumprun.o and libc being a horrible mess. We agree that the mess is in dire need of a cleanup, just not right now. > Reading the diff: > Not sure if ":" is the easiest-to-parse delimiter for your netspec. Good point. Changed to comma for both BLKSPEC and NETSPEC, this will work well for both eg. NFS and IPv6. Martin ------------------------------------------------------------------------------ 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
