Re: config(8) does not add post-processing for source file with compile-with command in sys/conf/files

From: Paul Ambrose <ambrosehua_at_gmail.com>
Date: Fri, 21 Oct 2011 20:39:08 +0800
There are many other compile-with  not started with ${NORMAL_C}, your
patch adds
${NORMAL_CTFCONVERT}  to them too, which could not be suitable for this.

2011/10/19 Ryan Stone <rysto32_at_gmail.com>:
> I have run into the same issue recently.  I have been testing the
> following patch(on 8.2-RELEASE) and it seems to have worked for me:
>
> --- mkmakefile.c 11:09:30.000000000 -0400
> +++ mkmakefile.c        2011-10-06 11:13:31.000000000 -0400
> _at__at_ -742,15 +742,16 _at__at_
>                                break;
>                        }
>                        snprintf(cmd, sizeof(cmd),
> -                           "${%s_%c%s}\n.if defined(NORMAL_CTFCONVERT) && "
> -                           "!empty(NORMAL_CTFCONVERT)\n"
> -                           "\t${NORMAL_CTFCONVERT}\n.endif", ftype,
> +                           "${%s_%c%s}\n", ftype,
>                            toupper(och),
>                            ftp->f_flags & NOWERROR ? "_NOWERROR" : "");
>                        compilewith = cmd;
>                }
>                *cp = och;
> -               fprintf(f, "\t%s\n\n", compilewith);
> +               fprintf(f, "\t%s\n", compilewith);
> +               fprintf(f, ".if defined(NORMAL_CTFCONVERT) && "
> +                            "!empty(NORMAL_CTFCONVERT)\n"
> +                            "\t${NORMAL_CTFCONVERT}\n.endif\n\n");
>        }
>  }
>
Received on Fri Oct 21 2011 - 10:39:10 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:19 UTC