From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <46F26460.7020504@altlinux.com> Date: Thu, 20 Sep 2007 16:15:28 +0400 From: Anton Farygin User-Agent: Thunderbird 2.0.0.6 (X11/20070804) MIME-Version: 1.0 To: ALT Devel discussion list References: <20070920042934.5F69A1CB441C@ssh.git.local.altlinux.org> <75e139a00709200205u4a012460s6a0ad59596ecb2ad@mail.gmail.com> <46F23D4E.3050602@mail.ru> In-Reply-To: <46F23D4E.3050602@mail.ru> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [devel] [git update] packages/prometeus: heads/master X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.9 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Sep 2007 12:15:28 -0000 Archived-At: List-Archive: List-Post: Avramenko Andrew пишет: > Спасибо, воспользуюсь. Может быть стоит вместо wget использовать curl. В чём-то он удобнее. > > > Slava Semushin пишет: >> 2007/9/20, Avramenko Andrew : >>> Update of /people/liks/packages/prometeus.git >> [...] >>> --- a/prometeus/scripts/gitrepos2mysql >>> +++ b/prometeus/scripts/gitrepos2mysql >> [...] >>> +system("cd /tmp; wget http://git.altlinux.org/people-packages-list"); >> Поискал в мане и нашел, что у wget есть опция -P, которая позволяет >> указать каталог для сохранения файла. Думаю, что лучше будет >> использовать её: >> >> - system("cd /tmp; wget http://git.altlinux.org/people-packages-list"); >> +system("wget -P /tmp http://git.altlinux.org/people-packages-list"); >> >> Плюсы вижу следующие: >> - меньше команд, меньше процессов создаётся и быстрее работает >> - избавляет от возможной проблемы, когда cd /tmp не удаётся выполнить, >> тогда текущий каталог не изменится, файл сохранится не в /tmp и все >> последующие команды не смогут отработать, а временный файл останется >> где попало >> >> Также, возможно, стОит использовать опцию -q. >> >>> +open(FILE,"/tmp/people-packages-list"); >>> +@list = ; >>> +close(FILE); >>> +unlink("/tmp/people-packages-list"); >> [...] >> >> > > _______________________________________________ > Devel mailing list > Devel@lists.altlinux.org > https://lists.altlinux.org/mailman/listinfo/devel