In message <13191.848853061@rubble.west.ora.com>, Jerry Peek writes:
>> @echo "Copying manual pages to $(MAN1) and $(MAN8)"
>> @test -w $(MAN1) || (mkdir -p $(MAN1); \
>> chown $(W_USER) $(MAN1); chgrp $(W_GROUP) $(MAN1) )
>> @test -w $(MAN8) || (mkdir -p $(MAN8); \
>> chown $(W_USER) $(MAN8); chgrp $(W_GROUP) $(MAN1) )
>
>I haven't tested the new Makefile patch yet. (Sorry.) But it looks
>like these chown's may be run by a non-root user? If that's true,
>they will fail on systems that only let root do a chown. How about
>replacing these with calls to $(INSTALL)?
They probably should, though the install.sh I included doesn't support
install -d. I suppose it could be added pretty easily.
However, since 'make install' is run as root anyway, this probably isn't
a big deal.
as far a config-test goes, perhaps wrapper.c should be hard-coded to
exit and print an explanatory error message if it tries to run
"config-test" is run as root or $W_USER?
--Dave
Follow-Ups:
References:
|
|