On Sat, Nov 29, 2008 at 04:12:08PM +0100, Bartosz Świątek wrote: > >> Added: > >> toys/stbr/src-builder-status.py (contents, props changed) > >> Log: > >> - get spec status from src-builder > > [CIAP...diff] > > Ałaaa.... Nie lepiej tak: [CIAP...kod]
> pewnie i lepiej, ale jak na 10min, które spędziłem nad tym po > zgłoszeniu zapotrzebowania na taką funkcję i tak jest nieźle i co > najważniejsze, program robi co ma robić. Nie krytykuję, tylko proponuję wprowadzenie małych usprawnień :) > 1. Wyślij dwa zlecenia testowe na buildery z Twojego maila: > a) pierwsze zlecenie niech się zbuduje > b) drugie zlecenie niech się nie zbuduje wcale, niech nawet nie trafi > do queue - celem tego jest dostać mail z src-buildera z błędem, że nie > mógł zbudować src.rpm. Hmmm... Czy ja wogólę mam taką moc sprawczą? Buildery mają raczej w nosie mój adres email i to, co z niego przychodzi. > 2. Uruchom skrypt i sprawdź czy w pierwszym wypadku dostaniesz OK lub > nic (tak naprawdę status poprawny nas w ogóle nie interesuje) i czy w > drugim przypadku wyskoczy FAILED (to i tylko to nas interesuje). > Oczywiście musisz też sprawdzić czy w razie podania bzdurnego speca > program nie wywali jakiegoś błędu - najlepiej żeby nic nie wywalił. Czy dostępne są takie wiadomości gdzieś publicznie? Z nagłówkami? Ewentualnie, czy mógłbyś mi podesłać takowe? Z nagłowkami? > 3. Jak zadziała to nic nie stoi na przeszkodzie byś mnie poprawił w > naszym SVNie Again, mam dostęp do CVS. Nic mi nie wiadomo o moich uprawnieniach do grzebania w repo SVN. > - ja pythona nie umiem programować, tak tylko czasami z > doskoku coś próbuję, także na pewno popełniam błędy, które Tobie mogą > wydawać się zabawne. Nie wydają mi się zabawne, ani głupie, ani nic z tych rzeczy. Kod jest dobry i działa. Zaproponowałem tylko zmianę na bardziej "pythonic way", gdyż rozpoznaję w nim silne wpływy języka C, które mogą nieco obniżyć wydajność. Nic strasznego, żadnych błędów, tylko usprawnienie. > 4. Bot ściągnie nową wersje z repozytorium o pełnej godzinie (kiedyś > było to tylko o 1 w nocy, ale po tym jak się zaczęli > wpisywać/wypisywać developerzy z projektu to lepiej częściej > odświeżać). Hmmm... To w takim razie przydałaby się jeszcze informacja o przeznaczeniu tego skryptu, żebym nie walnął jakiejś głupoty, która sprawdza się w jednym środowisku, a w innym niekoniecznie. Pozdrawiam Beorn -- Daniel 'Beorn' Mróz <[EMAIL PROTECTED]> http://127.0.0.1/beorn [GIT d s:- a-@ C++++ UL++++$ P+ L++++ E--- W+ N+++ o? K- w---] [O- M- V! PS+ PE++ Y+ PGP++ t- 5 X R !tv b+ DI D++ G++ e h*] [ r++ y+ ] _______________________________________________ pld-devel-pl mailing list [email protected] http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl
