W dniu 29 listopada 2008 16:47 użytkownik Daniel Mróz <[EMAIL PROTECTED]> napisał: > 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ń :)
Ale ja się ciesze z takiej konstruktywnej krytyki. Nikomu nie zaszkodzi, a z pownością pomoże projektowi. > >> 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. W takim razie musisz pogadać z naszymi RMami na ten temat. > >> 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? Publicznie nie. Niestety bot już usunął o 1 w nocy zaległe maile, ale mogę Ci dać obecne. Tam wprawdzie jest tylko mail z informacją OK, ale chyba ławo będziesz umiał sobie spreparować takiego z informacja FAILED. http://stbr(dot)pld-linux(dot)org/beorn.txt (zeby boty nie przechwyciły jakieś) Jak nie będzie ci już to potrzebne to daj znać i to usunę. > >> 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. Masz. O ile mi wiadomo to wszyscy regularni developerzy maja dostęp zarówno do CVS jak i SVN (ten sam login, to samo hasło). > >> - 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. Cieszę się z każdej poprawy. A i tak, kiedyś sporo w C pisałem. > >> 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. Przeznaczenie jest dla http://stbr.pld-linux.org/ (stąd też tam wylądowało w SVNie). Konkretnie, żeby dostać informację: http://stbr.pld-linux.org/?show=pawelz&date=2008-11-28%2013:30:51 zamiast błędnego "not filled". _______________________________________________ pld-devel-pl mailing list [email protected] http://lists.pld-linux.org/mailman/listinfo/pld-devel-pl
