[PATCH] Cleanup: consolidate all /etc/fstab mentions to a define

Tito farmatito at tiscali.it
Sun Feb 6 20:12:22 UTC 2011


On Sunday 06 February 2011 18:43:48 Denys Vlasenko wrote:
> On Sunday 06 February 2011 17:00, Lauri Kasanen wrote:
> > Hi
> > 
> > /etc/mtab is a define, so lets clean up fstab to be one too.
> 
> Those string defines hail from the times when linker was too dumb
> to merge identical strings. Now it does that by its own (not perfectly,
> but good enough). Therefore the reason for them disappeared.
> 
> Maybe we need to remove them altogether.
> 
> In any case, I don't see the reason why we'd want to add more.
> 

Hi,
one reason for define s is that it allows to setup a test environment
where this files are manipulated on a running box without risking
to break the real files. It also allows to test codechanges 
which need root privileges as normal user. 
I'm for defines.

Ciao,
Tito


More information about the busybox mailing list