Re: [devel] [git update] packages/prometeus: heads/master

[Avramenko Andrew <liksx@mail.ru>]

Спасибо, воспользуюсь.


Slava Semushin пишет:
> 2007/9/20, Avramenko Andrew <liks / altlinux.org>:
>> 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 = <FILE>;
>> +close(FILE);
>> +unlink("/tmp/people-packages-list");
> [...]
> 
>