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

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

[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");
[...]


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

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

[Avramenko Andrew]

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


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");
> [...]
> 
> 

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

[Anton Farygin]

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

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

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 351 bytes --]

* Anton Farygin <rider@> [070920 16:15]:
> Может быть стоит вместо wget использовать curl.
> В чём-то он удобнее.
Вообще-то в перле удобнее использовать сразу LWP...

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

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

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 383 bytes --]

On Thu, Sep 20, 2007 at 04:15:28PM +0400, Anton Farygin wrote:
> Avramenko Andrew пишет:
> > Спасибо, воспользуюсь.
> 
> Может быть стоит вместо wget использовать curl.
> В чём-то он удобнее.

У меня был негативный опыт с wget.  Я его всегда считал "стандартным
гнутым инструментом", а он на поверку оказался довольно глюкавым.
По крайней мере таймауты одно время глючили.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

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

[Avramenko Andrew]

Alexey I. Froloff пишет:
> * Anton Farygin <rider@> [070920 16:15]:
>> Может быть стоит вместо wget использовать curl.
>> В чём-то он удобнее.
> Вообще-то в перле удобнее использовать сразу LWP...

Алексей, с точки зрения правильной программы - да. С точки зрения того, 
что нужно быстро что-то написать, гораздо проще использовать 
существующий системный инструмент, который мне достаточно хорошо 
понятен. Мне кажется пара Кб памяти здесь не критичны - запускается один 
раз в сутки по крону, тянет одну страничку.

На LWP писал один раз и 2 года назад. Как появится время - обязательно 
перепишу на нем.