[PATCH] Makefile.in: set outdir for TIC explicitly

Changqing Li changqing.li at windriver.com
Fri Jan 5 08:34:49 CET 2024


On 12/22/23 16:10, Emanuele Giaquinta wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> Hi,
>
> On Thu, Dec 21, 2023 at 01:46:14PM +0800, changqing.li at windriver.com wrote:
>> When doing cross-compile, native tic's TERMINFO usually
>> set as a native dir, which could not the target install dir,
>> which will cause rxvt-unicode terminfo will be wrongly installed.
>> set the outdir explicitly to install them to correct dir.
> the patch does not look correct to me, as it hardcodes the terminfo
> location, with no guarantee that it is the one used by ncurses in the
> target environment. The correct solution should be to use the tic
> binary from the target enviroment with
>
> TIC=<path/to/tic> ./configure

Hi,

Thanks.  You are correct,  the hardcodes path is not right.  But during 
cross compile,  the target arch maybe

different with the host arch. so we cannot use the tic from target env.

So I think maybe it can be changed like this:

if TERMINFO_PATH ="":

     @TIC@ -x $(srcdir)/etc/rxvt-unicode.terminfo || \
     @TIC@ $(srcdir)/etc/rxvt-unicode.terminfo

else:

     @TIC@ -x $(srcdir)/etc/rxvt-unicode.terminfo -o $(TERMINFO_PATH) || \
     @TIC@ $(srcdir)/etc/rxvt-unicode.terminfo -o $(TERMINFO_PATH)


by default, TERMINFO_PATH is empty,  it works like current way.  if cross compile need
to set it's own path,  just set TERMINFO_PATH, and it have responsibility to ensure the path
is the same as what will be used in target env.


if you don't have other concerns about above way, I will send an v2 patch.

Thanks
Changqing

>
> Emanuele



More information about the rxvt-unicode mailing list