From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <46F23D4E.3050602@mail.ru> Date: Thu, 20 Sep 2007 13:28:46 +0400 From: Avramenko Andrew User-Agent: Thunderbird 1.5.0.4 (X11/20060613) MIME-Version: 1.0 To: ALT Linux Team development discussions References: <20070920042934.5F69A1CB441C@ssh.git.local.altlinux.org> <75e139a00709200205u4a012460s6a0ad59596ecb2ad@mail.gmail.com> In-Reply-To: <75e139a00709200205u4a012460s6a0ad59596ecb2ad@mail.gmail.com> 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 09:31:14 -0000 Archived-At: List-Archive: List-Post: Спасибо, воспользуюсь. 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"); > [...] > >