[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