[Mono-devel-list] Re: [Mono-patches] r43792 - in trunk/mcs/build: . profiles

Miguel de Icaza miguel at ximian.com
Tue May 3 11:20:43 EDT 2005


> > > 	* profiles/basic.make: The test for mcs was broken on jails where
> > > 	we got a warning due to /proc not being mounted. Remove hackish wc
> > > 	usage.
> > 
> > ???  Why remove a working test that caught an actual problem?
> > 
> > What's wrong with adding a 'mount /proc' to the jail setup script?  We
> > are not worried about running untrusted code inside the jail.
> Because anything could create extra text. Say I wanted to run make with
> MONO_LOG_LEVEl set? Maybe I am running with some light weight profiler?

I agree with Raja that we should not remove tests that actually flag a
problem.   Lets discuss in the office.

Mounting /proc is something that is needed for the jails anyways.

> My test made sure the exit status of mono was correct (at least, I am
> pretty sure it does) -- this is what really matters. If mcs can run and
> create a binary that returns a 0 exit status, things are probably
> working OK.

More information about the Mono-devel-list mailing list