* Re: [devel] [git update] packages/prometeus: heads/master @ 2007-09-20 9:05 ` Slava Semushin 2007-09-20 9:28 ` Avramenko Andrew 0 siblings, 1 reply; 6+ messages in thread From: Slava Semushin @ 2007-09-20 9:05 UTC (permalink / raw) To: ALT Devel discussion list 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/prometeus: heads/master 2007-09-20 9:05 ` [devel] [git update] packages/prometeus: heads/master Slava Semushin @ 2007-09-20 9:28 ` Avramenko Andrew 2007-09-20 12:15 ` Anton Farygin 0 siblings, 1 reply; 6+ messages in thread From: Avramenko Andrew @ 2007-09-20 9:28 UTC (permalink / raw) To: ALT Linux Team development discussions Спасибо, воспользуюсь. 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"); > [...] > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/prometeus: heads/master 2007-09-20 9:28 ` Avramenko Andrew @ 2007-09-20 12:15 ` Anton Farygin 2007-09-20 13:48 ` Alexey I. Froloff 2007-09-20 18:40 ` Alexey Tourbin 0 siblings, 2 replies; 6+ messages in thread From: Anton Farygin @ 2007-09-20 12:15 UTC (permalink / raw) To: ALT Devel discussion list 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/prometeus: heads/master 2007-09-20 12:15 ` Anton Farygin @ 2007-09-20 13:48 ` Alexey I. Froloff 2007-09-21 9:57 ` Avramenko Andrew 2007-09-20 18:40 ` Alexey Tourbin 1 sibling, 1 reply; 6+ messages in thread From: Alexey I. Froloff @ 2007-09-20 13:48 UTC (permalink / raw) To: ALT Devel discussion list [-- 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 --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/prometeus: heads/master 2007-09-20 13:48 ` Alexey I. Froloff @ 2007-09-21 9:57 ` Avramenko Andrew 0 siblings, 0 replies; 6+ messages in thread From: Avramenko Andrew @ 2007-09-21 9:57 UTC (permalink / raw) To: ALT Linux Team development discussions Alexey I. Froloff пишет: > * Anton Farygin <rider@> [070920 16:15]: >> Может быть стоит вместо wget использовать curl. >> В чём-то он удобнее. > Вообще-то в перле удобнее использовать сразу LWP... Алексей, с точки зрения правильной программы - да. С точки зрения того, что нужно быстро что-то написать, гораздо проще использовать существующий системный инструмент, который мне достаточно хорошо понятен. Мне кажется пара Кб памяти здесь не критичны - запускается один раз в сутки по крону, тянет одну страничку. На LWP писал один раз и 2 года назад. Как появится время - обязательно перепишу на нем. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/prometeus: heads/master 2007-09-20 12:15 ` Anton Farygin 2007-09-20 13:48 ` Alexey I. Froloff @ 2007-09-20 18:40 ` Alexey Tourbin 1 sibling, 0 replies; 6+ messages in thread From: Alexey Tourbin @ 2007-09-20 18:40 UTC (permalink / raw) To: ALT Devel discussion list [-- 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 --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-09-21 9:57 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2007-09-20 9:05 ` [devel] [git update] packages/prometeus: heads/master Slava Semushin 2007-09-20 9:28 ` Avramenko Andrew 2007-09-20 12:15 ` Anton Farygin 2007-09-20 13:48 ` Alexey I. Froloff 2007-09-21 9:57 ` Avramenko Andrew 2007-09-20 18:40 ` Alexey Tourbin
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git