On 2018-12-22, Bram Moolenaar wrote:
> Gary Johnson wrote:
> 
> > On 2018-12-22, Tony Mechelynck wrote:
> > > Setting $CONF_ARGS in the environment doesn't work anymore: it is
> > > overridden (unless make is called with the -e switch, which is not
> > > normal practice) by the line
> > > 
> > > CONF_ARGS = $(CONF_ARGS1) $(CONF_ARGS2) $(CONF_ARGS3) $(CONF_ARGS4)
> > > $(CONF_ARGS5)
> > > 
> > > at line 326 of src/Makefile. Setting $CONF_ARGS1 to $CONF_ARGS5 works,
> > > since lines 321-325 of the same Makefile are commented-out.
> > > 
> > > So if e.g. you had
> > > 
> > > export CONF_ARGS='--with-vim-name=vi'
> > > 
> > > in the bash startup script in your shadow directory for building a
> > > Tiny Vim, that line must now be replaced by
> > > 
> > > export CONF_ARGS2='--with-vim-name=vi'
> > > 
> > > otherwise your Tiny Vim will be built with the executable name "vim",
> > > and "make installvimbin" will copy and strip it into
> > > /usr/local/bin/vim, which is not what you want.
> > > 
> > > N.B. I don't edit the src/shadow-*/Makefile or the src/Makefile;
> > > instead, I create soft links Makefile -> ../Makefile in each shadow
> > > directory and set my configure arguments by means of environment
> > > variables. This way I always use the latest Makefile. But I have to
> > > keep watch for incompatible changes like the one above.
> 
> Sorry about that, I thought values passed in would override values set
> in the Makefile.
> 
> > That's a good short-term work-around, but wouldn't it be better to
> > fix the problem by replacing that CONF_ARGS assignment in
> > src/Makefile with this:
> > 
> >     CONF_ARGS = $(CONF_ARGS) $(CONF_ARGS1) $(CONF_ARGS2) $(CONF_ARGS3) 
> > $(CONF_ARGS4) $(CONF_ARGS5)
> 
> That doesn't work, my Make complains about a recursive value.
> Let me comment-out that line, that should work.  And still avoid the
> confusion when someone un-comments more then one CONF_ARGS line.

I wasn't sure that would work, but assumed that it would from this
commented-out code around line 770:

    ### (9) Solaris 2.x with cc (SunPro), using Athena.
    ###     Only required for compiling gui_at_sb.c.
    ###     Symptom: "identifier redeclared: vim_XawScrollbarSetThumb"
    ###     Use one of the lines (either Full ANSI or no ANSI at all)
    #CFLAGS = $(CFLAGS) -Xc
    #CFLAGS = $(CFLAGS) -Xs

Regards,
Gary

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui