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

[Anton Farygin <rider@altlinux.com>]

Avramenko Andrew пишет:
> Спасибо, воспользуюсь.

Может быть стоит вместо wget использовать curl.

В чём-то он удобнее.

> 
> 
> 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");
>> [...]
>>
>>
> 
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel