Re: [MFC REQUEST] Filename completion in sh(1)

On Tue, Jun 15, 2010 at 9:08 PM, jhell <jhell_at_dataix.net> wrote:
> On 06/15/2010 22:00, jhell wrote:
>> On 06/15/2010 21:14, Brandon Gooch wrote:
>>> I discovered a few moments ago that filename completion had been
>>> committed to HEAD[1]!!!
>>
>>> This is a (seemingly) small, yet VERY useful addition, and, of course
>>> I'm so grateful to Guy Yur and Jilles for getting this into the tree
>>> :)
>>
>>> I would like to make an "official" request that this feature be MFC'd
>>> to 8-STABLE as soon as reasonably possible.
>>
>>> Again, my thanks to you both for your work!
>>
>>> -Brandon
>>
>>> [1] http://svn.freebsd.org/viewvc/base?view=revision&revision=209221
>>
>> Here is a diff from stable/8/bin/sh r209146 -> head/bin/sh. This is
>> quite a large difference among the two with the following diffstats.
>>
>>  44 files changed, 1214 insertions(+), 728 deletions(-)
>>
>> This is untested, use at your own risk.
>>
>
> Actually, I lied. I have just tested it and it does not compile for the
> following errors:
>
> /usr/src/bin/sh/histedit.c: In function 'histedit':
> /usr/src/bin/sh/histedit.c:124: error: '_el_fn_sh_complete' undeclared
> (first use in this function)
> /usr/src/bin/sh/histedit.c:124: error: (Each undeclared identifier is
> reported only once
> /usr/src/bin/sh/histedit.c:124: error: for each function it appears in.)
>
> But I do not have time to look into this further until tommorow.

Thanks for this jhell!

It appears that a C header file is missing from /usr/include; the
function declaration is "histedit.h".

No problem, just `make install` in /usr/src/include and it should build.

-Brandon