This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix fixincludes for canadian cross builds
- From: Bruce Korb <bkorb at gnu dot org>
- To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Biener <rguenther at suse dot de>
- Date: Fri, 17 Feb 2017 15:37:18 -0800
- Subject: Re: [PATCH] Fix fixincludes for canadian cross builds
- Authentication-results: sourceware.org; auth=none
- References: <AM4PR0701MB216279A16CA1B1D69DCFE1D7E4400@AM4PR0701MB2162.eurprd07.prod.outlook.com>
On 02/06/17 10:44, Bernd Edlinger wrote:
> I tested this change with different arm-linux-gnueabihf cross
> compilers, and verified that mkheaders still works on the host system.
>
> Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
> Is it OK for trunk?
As long as you certify that this is correct for all systems we care about:
+BUILD_SYSTEM_HEADER_DIR = `
+ echo $(CROSS_SYSTEM_HEADER_DIR) | \
+ sed -e :a -e 's,[^/]*/\.\.\/,,' -e ta`
that is pretty obtuse sed-speak to me. I suggest a comment
explaining what sed is supposed to be doing. What should
"$(CROSS_SYSTEM_HEADER_DIR)" look like?