Date: Fri, 21 Oct 2011 20:39:08 +0800 From: Paul Ambrose <ambrosehua@gmail.com> To: Ryan Stone <rysto32@gmail.com> Cc: freebsd-current <freebsd-current@freebsd.org>, freebsd-questions <freebsd-questions@freebsd.org> Subject: Re: config(8) does not add post-processing for source file with compile-with command in sys/conf/files Message-ID: <CAMwoQQ7D3Sk-Ka49Mcau2=V9Uz9dkC_pY2D-72kA8OWS5twZFw@mail.gmail.com> In-Reply-To: <CAFMmRNxKRqC=d-fwBPO=OD%2B1A7QmhsUdrQAYqCv33SdF_zDYtA@mail.gmail.com> References: <CAMwoQQ7Qa29eZnEKDmeVjx723AOejSWm%2BffgffEP3ZUUUuvzGA@mail.gmail.com> <CAFMmRNxKRqC=d-fwBPO=OD%2B1A7QmhsUdrQAYqCv33SdF_zDYtA@mail.gmail.com>
index | next in thread | previous in thread | raw e-mail
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@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
> @@ -742,15 +742,16 @@
> 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");
> }
> }
>
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMwoQQ7D3Sk-Ka49Mcau2=V9Uz9dkC_pY2D-72kA8OWS5twZFw>