Re: [Libreoffice] [PATCH] Fix curl location in download script

Francois Tigeot <ftigeot -AT- wolfpond.org>
Sun, 27 Mar 2011 07:50:06 +0200

Hi Miklos,

On Sun, Mar 27, 2011 at 12:14:51AM +0100, Miklos Vajna wrote:

On Sat, Mar 26, 2011 at 11:41:38PM +0100, Francois Tigeot <ftigeot@wolfpond.org> wrote:


The attached patch replaces the strange and broken lookup logic
present in the script by a single 'which' command.


Won't this case a warning

bash: curl: command not found

on machines without curl? Maybe a better approarch is to see if `type -p
curl` returns anything.


No, it will just set the curl variable to 'curl not found'

curl is required at the autogen.sh stage: if I try to run ./autogen.sh
without any curl package installed, I get this error:

configure: error: install curl to run this script

-- 
Francois Tigeot

Context

[Libreoffice] [PATCH] Fix curl location in download script · Francois Tigeot
- Re: [Libreoffice] [PATCH] Fix curl location in download script · Miklos Vajna
  - Re: [Libreoffice] [PATCH] Fix curl location in download script · Francois Tigeot
    - Re: [Libreoffice] [PATCH] Fix curl location in download script · Miklos Vajna
      - Re: [Libreoffice] [PUSHED] [PATCH] Fix curl location in download script · Francois Tigeot

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.