From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mout.kundenserver.de ([212.227.17.24]) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1XEgJE-0008Pe-Fz for ptxdist@pengutronix.de; Tue, 05 Aug 2014 17:05:59 +0200 Received: from [192.168.102.37] (herbert.entwicklung.er.corscience.de [192.168.102.37]) by mail.er.corscience.de (Postfix) with ESMTPSA id 4079C347A050 for ; Tue, 5 Aug 2014 17:05:43 +0200 (CEST) Message-ID: <1407251151.4441.44.camel@herbert.er.corscience.de> From: Tim Niemeyer Date: Tue, 05 Aug 2014 17:05:51 +0200 In-Reply-To: <53CF540A.6030105@optimeas.de> References: <1406042531-15648-1-git-send-email-bernhard@bwalle.de> <53CF540A.6030105@optimeas.de> Mime-Version: 1.0 Subject: Re: [ptxdist] [PATCH] ptxdist: Create symlinks for toolchain-related tools Reply-To: ptxdist@pengutronix.de List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist@pengutronix.de Hello Am Mittwoch, den 23.07.2014, 08:19 +0200 schrieb Matthias Klein: > Hello Bernhard, > > until now I did not test you patch, but I also have to link all these > toolchain programs into the sysroot-host/lib/wrapper/ directory for > using everything with QtCreator. I had problems with QtCreator, too. So i tested this patch and it solved my problems. So.. Tested-by: Tim Niemeyer Best regards, Tim > > Am 22.07.2014 17:22, schrieb Bernhard Walle: > > This patch adds symlinks in the toolchain wrapper dir for > > toolchain-related tools like as, nm, etc. Currently, that symlinks are > > only created for tools for which ptxdist provides the wrapper mechanism. > > > > The primary use case is our internal build system (application that > > resides in local_src) that expects all programs to be at the same > > location (because some very complicated magic...). > > > > Apart from that, I think the solution to have all tools at the same > > place is cleaner and the patch is very small. So it would be nice if it > > could be accepted. > > > > Signed-off-by: Bernhard Walle > > --- > > bin/ptxdist | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/bin/ptxdist b/bin/ptxdist > > index 45b3540..d9758e3 100755 > > --- a/bin/ptxdist > > +++ b/bin/ptxdist > > @@ -610,6 +610,12 @@ check_compiler() { > > } > > done > > > > + for tool in ar as cpp nm objcopy objdump ranlib readelf size strip ; do > > + tool_abs="$(which "${tool}" 2>/dev/null)" > > + ptxd_replace_link "${tool_abs}" "${wrapper_dir}/${tool}" || \ > > + ptxd_bailout "Unable to create host ${tool} wrapper link" > > + done > > + > > if ptxd_get_ptxconf PTXCONF_BUILD_TOOLCHAIN >/dev/null; then > > PATH="${wrapper_dir}:${PATH}" > > return > > @@ -712,6 +718,9 @@ check_compiler() { > > ptxd_replace_link "$(which ${compiler_prefix}${cc})" "${wrapper_dir}/real/${compiler_prefix}${cc}" && > > ptxd_replace_link "${SCRIPTSDIR}/wrapper/${cc}-wrapper" "${wrapper_dir}/${compiler_prefix}${cc}" > > done && > > + for tool in ar as cpp nm objcopy objdump ranlib readelf size strip ; do > > + ptxd_replace_link "$(which ${compiler_prefix}${tool})" "${wrapper_dir}/${compiler_prefix}${tool}" > > + done && > > PATH="${wrapper_dir}:${PATH}" && > > if [ -n "${PTXDIST_ICECC}" ]; then > > local icecc_dir="${sysroot_host}/lib/icecc" > > -- ptxdist mailing list ptxdist@pengutronix.de