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

["Slava Semushin" <slava.semushin@gmail.com>]

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