[m-rev.] diff: disable mfilterjavac by default

Paul Bone paul at bone.id.au
Wed Oct 16 15:58:21 AEDT 2013


On Wed, Oct 16, 2013 at 02:55:58PM +1100, Julien Fischer wrote:
>
> On Wed, 16 Oct 2013, Paul Bone wrote:
>>
>> Are these really the steps you used to reproduce this problem?  I'm having
>> trouble accepting that they worked at all before I introduced mfilterjavac.
>> I'm also concerned that since it can't find mmc when I have none in my path,
>> that it isn't using the stage1 mmc to build stage2 which is a problem for
>> bootchecking in general.
>
> I think the problem I had was initially just a bootstrapping problem,
> i.e. rotd-2013-04-18 didn't have mfilterjavac but it was required to
> install from stage2 of rotd-2013-05-01 (suitably modified to remove the
> workaround that disables the use of mfilterjavac).

Okay, thanks.

>
> Ok, so now that all installations have mfilterjavac the problem should
> go away?  However, why is mmake install (technically install_grades)
> looking at the mfilterjavac in the PATH, not the one it has just built?
> (This is potentially a problem if mfilterjavac itself changes -- however
> unlikely -- in a non backwards compatible manner.)

I agree, this could be an issue.

> Note that for other utility programs, e.g. mfiltercc, mkinit etc, we set
> things up to look at the version in the workspace.  For example from the
> top-level Mmakefile:
>
>     SUBDIR_MMAKE =  PATH=../scripts:../util:$$PATH \
>                     MMAKE_VPATH=. \
>                     MMAKE_DIR=../scripts \
>                     DESTDIR=$(DESTDIR) \
>                    ../scripts/mmake $(MMAKEFLAGS)
>
>     SUBSUBDIR_MMAKE = PATH=../../scripts:../../util:$$PATH \
>                     MMAKE_VPATH=. \
>                     MMAKE_DIR=../../scripts \
>                     DESTDIR=$(DESTDIR) \
>                     ../../scripts/mmake $(MMAKEFLAGS)
>
> For me, the issue here is that you are not using the new mfilterjavac
> (the java grade fails to build for me from stage 2 -- I'm not sure about
> stage 1, but both should work anyway -- when the build compiler lacks
> mfilterjavac, even though the workspace has just built a copy.)
>
> In short, why do SUBDIR_MMAKE and SUBSUBDIR_MMAKE not also point at
> ../mfilterjavac and ../../mfilterjavac respectively.

The simple reason is "I forgot", so I can fix it and re-test.  What I
propose to test is to chmod 000 my installed mfilterjavac and then try to
run make install from my workspace's stage2 directory.

I never meant to disagree with your description of the path issue.  I just
couldn't reproduce the same result.

Thanks.


-- 
Paul Bone
http://www.bone.id.au



More information about the reviews mailing list