Not logged in. · Lost password · Register
Forum: Fedora › Programy RSS
Gnome-color-chooser
paczuszki ;-)

Announcement

2006-06-03, 10:40 by gajownik
Subject: Zanim coś napiszesz...
Witamy w naszych niskich progach ;-)

Zanim coś napiszesz na forum:
1. zapoznaj się z Wiki i FAQ a także Poradnikiem - są tam odpowiedzi na najczęściej pojawiające się pytania
2. skorzystaj z forumowej wyszukiwarki i/lub Google - jest spora szansa, że ktoś już rozwiązał Twój problem
3. uaktualnij cały system
4. przeczytaj regulamin oraz tekst na temat mądrego zadawania pytań
5. jeśli masz jakiś problem, przekaż nam o nim odpowiednią ilość informacji - jak na razie nie potrafimy jeszcze wróżyć z fusów
6. dla każdego nowego problemu twórz osobny wątek używając jednak zdrowego rozsądku
7. nie pisz niczego, czego wstydził(a)byś się przed swoją mamą ;-)

Życzymy miłej zabawy - zespół fedorapl.org
Create new thread · Page:  1  2  next
Avatar
Reply · Quote adios #1 (2007-01-27, 20:07)
User title: Antylopa GNU
since May 2006 · 189 posts · Location: Mielec
Group memberships: Administratorzy, Moderatorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Subject: Gnome-color-chooser
Przeglądając zasoby gnome-look.org natrafiłem na ciekawy program, dzięki któremu możemy zmienić wszystkie kolory w widgetach GTK i inne ciekawe rzeczy. Program tak mi się spodobał, że postanowiłem go zpaczkować ;-)

http://fedorapl.org/adios/paczki/gnome-color-chooser/

UWAGA ! To druga moja paczka RPM więc może działać wadliwie. Jakby coś działało źle to podziel się spostrzeżeniami w tym wątku.
Zapraszam do testowania...
Pierwsze kroki w Fedora Core - http://poradnik.fedorapl.org
adios.blog - http://adiospl.wordpress.com
Avatar
Reply · Quote gajownik #2 (2007-01-27, 22:04)
User title: GNU fanatic!
since May 2006 · 206 posts · Location: Bytom
Group memberships: Administratorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
W poniedziałek mam egzamin, więc teraz za bardzo nie mam czasu na robienie pełnej recenzji. Na pierwszy rzut oka:
- zła wartość w tagu Release: tylko dla wersji beta/alfa/z cvs-a zaczyna się od 0 ? http://fedoraproject.org/wiki/Packaging/NamingGuidelines#P…
- w polu Summary brakuje kilku spacji (to nie jest błąd, ale źle wygląda :P )
- nie odpalałem jeszcze programu, ale chyba jest błędna grupa: tu masz dostępne ? http://fedoraproject.org/wiki/RPMGroups
- w polu Source powinna być pełna ścieżka do tarballa (można korzystać z makr)
- do wywalenia: BuildRequires:  gcc-c++ ? http://fedoraproject.org/wiki/Packaging/Guidelines#Excepti…
- jeszcze nie sprawdzałem, ale reszta Requires jest pewnie też do wywalenia. rpmbuild samemu to robi. Jeśli potrzebne jest określenie minimalnej wersji biblioteki to robi się to raczej w BuildRequires, a SONAME biblioteki sam resztę załatwia
- %setup: parametr -n można usunąc - rpm samemu się z tym upora
- %makeinstall sux :P ? http://fedoraproject.org/wiki/Packaging/Guidelines#MakeIns…
- powinieneś zaprzyjaźnić się z rpmlint ? http://fedoraproject.org/wiki/Packaging/Guidelines#rpmlint

[rpm-build@X ~]$ rpmlint gnome-color-chooser-0.1.3-0.1.src.rpm
E: gnome-color-chooser description-line-too-long Gnome Color Chooser lets you customize your Gnome desktop to fit all your needs.
E: gnome-color-chooser description-line-too-long -> change many colors (e.g. background colors, window decoration colors, tooltips)
E: gnome-color-chooser description-line-too-long -> allow the use of light-colored wallpapers without getting the icons unreadable
E: gnome-color-chooser no-cleaning-of-buildroot %install
E: gnome-color-chooser no-cleaning-of-buildroot %clean
W: gnome-color-chooser no-%clean-section
[rpm-build@X ~]$

Reszta spostrzeżeń później. Teraz tylko coś mi się wydaje, że plik .desktop nie jest poprawnie zainstalowany. A, w adresie e-mail radziłbym zamienić małpę na coś innego, by nie mieć potem tyle spamu.

No, paczuszkę się dopieści i możesz zaczynać starać się o bycie oficjalnym fedorowym pakerem :]
Get counted!
Do rozwiązywania problemów z Fedorą służy forum!
SM/Fx/M + AdBlock Plus + Adblock Filterset.G Updater == [Image: http://browsehappy.com/buttons/bh_80x15.gif]
Avatar
Reply · Quote ecik #3 (2007-01-27, 22:08)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
In reply to post #1
W zasadzie to paczki nie testowałem, mogę tylko ocenić w sposób podobny jak się to robi w Extras ;)
Najpierw wyjście rpmlinta:
E: gnome-color-chooser description-line-too-long Gnome Color Chooser lets you customize your Gnome desktop to fit all your needs.
E: gnome-color-chooser description-line-too-long -> change many colors (e.g. background colors, window decoration colors, tooltips)
E: gnome-color-chooser description-line-too-long -> allow the use of light-colored wallpapers without getting the icons unreadable
W: gnome-color-chooser incoherent-version-in-changelog 0.1.3 0.1.3-0.1
E: gnome-color-chooser description-line-too-long Gnome Color Chooser lets you customize your Gnome desktop to fit all your needs.
E: gnome-color-chooser description-line-too-long -> change many colors (e.g. background colors, window decoration colors, tooltips)
E: gnome-color-chooser description-line-too-long -> allow the use of light-colored wallpapers without getting the icons unreadable
E: gnome-color-chooser no-cleaning-of-buildroot %install
E: gnome-color-chooser no-cleaning-of-buildroot %clean
W: gnome-color-chooser no-%clean-section
Tutaj już wiele jest widoczne - jedna linia w polu %description może mieć maksymalnie 79 znaków. Poza tym powinieneś dopisywać zawsze także aktualny release w changelogu, czyli zmienić 0.1.3 na 0.1.3-0.1. Poza tym nie wiem czemu dałeś 0.1 zamiast po prostu 1 ;) Musisz jeszcze na początku sekcji %install dopisać: rm -rf $RPM_BUILD_ROOT oraz stworzyć nową sekcję %clean:
%clean
rm -rf $RPM_BUILD_ROOT
A teraz kolejne rzeczy, których już rpmlint nie wyłapał:
 * W polu Source0 powinno się używać pełnego adresu URL
 * nie ma potrzeby dodawania BR gcc-c++, ponieważ należy do buildsystemu. Tutaj znajduje się lista paczek, których nie należy dodawać jako BuildRequires
 * jako Requires masz coś, co przez Fedorowców jest nazywane jako reinventing the wheel, czyli podajesz zależności, które zostały już automatycznie wykryte przez RPM-a. Jeśli chcesz poznać jakie zależności RPM sobie sam wykryje to wystarczy zapodać polecenie rpm -qpR gnome-color-chooser-0.1.3-0.1.x86_64.rpm na plik rpm, lub bez parametru -p jeśli na zainstalowany, aby zobaczyć:
gtkmm24 >= 2.8.0
libatk-1.0.so.0()(64bit)
libatkmm-1.6.so.1()(64bit)
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.4)(64bit)
libcairo.so.2()(64bit)
libcairomm-1.0.so.1()(64bit)
libdl.so.2()(64bit)
libgcc_s.so.1()(64bit)
libgcc_s.so.1(GCC_3.0)(64bit)
libgdk-x11-2.0.so.0()(64bit)
libgdk_pixbuf-2.0.so.0()(64bit)
libgdkmm-2.4.so.1()(64bit)
libglade-2.0.so.0()(64bit)
libglademm-2.4.so.1()(64bit)
libglademm24 >= 2.6.0
libglib-2.0.so.0()(64bit)
libglibmm-2.4.so.1()(64bit)
libgmodule-2.0.so.0()(64bit)
libgobject-2.0.so.0()(64bit)
libgtk-x11-2.0.so.0()(64bit)
libgtkmm-2.4.so.1()(64bit)
libm.so.6()(64bit)
libpango-1.0.so.0()(64bit)
libpangocairo-1.0.so.0()(64bit)
libpangomm-1.4.so.1()(64bit)
libsigc-2.0.so.0()(64bit)
libstdc++.so.6()(64bit)
libstdc++.so.6(CXXABI_1.3)(64bit)
libstdc++.so.6(CXXABI_1.3.1)(64bit)
libstdc++.so.6(GLIBCXX_3.4)(64bit)
libxml2 >= 2.6.0
libxml2.so.2()(64bit)
libz.so.1()(64bit)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rtld(GNU_HASH)
jak sam widzisz są tu wszystkie biblioteki z pakietów, które podałeś ręcznie. W rezultacie - niepotrzebnie się trudziłeś ;)
 * nie ma potrzeby podawania argumentu -n do makra %setup, gdyż to co jest tam podane jest domyślną ścieżką do której RPM próbuje przejść
 * nie należy używać makra %makeinstall, a tutaj jest wyjaśnione dlaczego
 * powinieneś użyć polecenia desktop-file-install do zainstalowania pliku .desktop. Więcej informacji tutaj
to by chyba było na tyle ;)

EDIT: no to się niestety spóźniłem ;-)
No, paczuszkę się dopieści i możesz zaczynać starać się o bycie oficjalnym fedorowym pakerem :]
Jakby co to ja jestem sponsorem i chętnie patrzę w kierunku paczkerów z ojczyzny ;)
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
This post was edited on 2007-01-27, 22:12 by ecik.
Avatar
Reply · Quote adios #4 (2007-01-27, 23:51)
User title: Antylopa GNU
since May 2006 · 189 posts · Location: Mielec
Group memberships: Administratorzy, Moderatorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Posiedziałem, poczytałem i się okazało, że nie taki diabeł straszny ;-) Paczki są już zgodne z zaleceniami Fedora Extras prawie w 100%.

Jedyna trudność to Source0. Nie mogę dać tam URL bo autor umieścił tak program, że jego url co jakiś czas może się zmieniać :/ Taki Tarball jest niezbyt pewny...

http://fedorapl.org/adios/paczki/gnome-color-chooser/

Co o tych paczuszkach sądzicie ?
Pierwsze kroki w Fedora Core - http://poradnik.fedorapl.org
adios.blog - http://adiospl.wordpress.com
Avatar
Reply · Quote gajownik #5 (2007-01-28, 12:21)
User title: GNU fanatic!
since May 2006 · 206 posts · Location: Bytom
Group memberships: Administratorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
  • nie powinieneś zmieniać nazwy pliku. Jako tag Source0 daj:
http://www.gnome-look.org/content/files/47349-gnome-color-chooser-0.1.3.tar.bz2

Wystarczyło użyć wgeta by zobaczyć poprawnego linka :P Stosowanie makra %{version} nie ma sensu, ze względu na ten numerek na początku pakietu.
  • błędne BuildRequires
Zamiast glibmm24-devel skrypt configure potrzebuje gtkmm24-devel. Ponieważ libglademm24-devel wymaga tego drugiego pakietu to nie zauważyłeś problemu. Skoro skrypt wymaga minimalnych wersji to warto to uwzględnić w pliku spec.
Brak "BuildRequires:  desktop-file-utils" (bez tego wysypie Ci się budowanie pakietu na fedorowych builderach)
  • skoro usunąłeś opcję -n z makrai setup, to trzeba było usunąć też parametr przekazywany przez tę opcję ;-)
Doczytaj:
http://rpm.org/max-rpm-snapshot/s1-rpm-specref-macros.html
http://rpm.org/max-rpm-snapshot/s1-rpm-inside-macros.html#…
  • do makra %configure dodaj opcję --disable-dependency-tracking
Przyspiesza to budowanie pakietu i zwiększa czytelność wypisywanych komunikatów
Zdecyduj się na jedne (polecam %{buildroot} i %{optflags} bo są krótsze)
  • nie dołączyłeś licencji i pliku THANKS
  • w seckji %files lepiej nie stosować makr (unika się zaciemnienia sytuacji). Oprócz tego gdy podajesz cały katalog to najlepiej podać na końcu znak / - dzięki temu wiesz, że to jest katalog a nie plik
  • w changelogu lepiej oddzielać poszczególne wpisy enterem (ja je dodaje w Emacsie - ctrl+c, ctrl+e)
Get counted!
Do rozwiązywania problemów z Fedorą służy forum!
SM/Fx/M + AdBlock Plus + Adblock Filterset.G Updater == [Image: http://browsehappy.com/buttons/bh_80x15.gif]
The author has attached one file to this post:
spec.diff 2.4 kBytes
You have no permission to open this file.
This post was edited 2 times, last on 2007-01-28, 12:26 by gajownik.
Edit reason: Dołączenie pliki z diffem
Avatar
Reply · Quote ecik #6 (2007-01-28, 12:28)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
Quote by gajownik:
Wystarczyło użyć wgeta by zobaczyć poprawnego linka :P Stosowanie makra %{version} nie ma sensu, ze względu na ten numerek na początku pakietu.
Można w takim razie zdefiniować dodatkowe makro, np. %{num} i w nim umieszczać ten numerek,
Quote by gajownik:
* w seckji %files lepiej nie stosować makr (unika się zaciemnienia sytuacji). Oprócz tego gdy podajesz cały katalog to najlepiej podać na końcu znak / - dzięki temu wiesz, że to jest katalog a nie plik
To zależy mocno od osoby, która recenzuje dany pakiet - ja ogólnie odnoszę wrażenie, że sugeruje się używanie makr wszędzie tam, gdzie to możliwe - czyli w tym przypadku raczej powinno zostać tak jak jest.
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
Avatar
Reply · Quote gajownik #7 (2007-01-28, 12:42)
User title: GNU fanatic!
since May 2006 · 206 posts · Location: Bytom
Group memberships: Administratorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Quote by ecik:
ja ogólnie odnoszę wrażenie, że sugeruje się używanie makr wszędzie tam, gdzie to możliwe - czyli w tym przypadku raczej powinno zostać tak jak jest.

Hehe, a ja wręcz przeciwnie :D Wiele osób mi doradzało niekorzystanie z makra %{name}, gdyż ono prawie nigdy się nie zmienia.
https://bugzilla.redhat.com/bugzilla/show_bug.…?id=17290… - podobnych głosów słyszałem już wiele. Podobnie doradza wylistowanie wszystkich plików, a nie stosowanie znaków globalnych (przy późniejszych aktualizacjach można wyłapać pewne błędy).
Get counted!
Do rozwiązywania problemów z Fedorą służy forum!
SM/Fx/M + AdBlock Plus + Adblock Filterset.G Updater == [Image: http://browsehappy.com/buttons/bh_80x15.gif]
Avatar
Reply · Quote ecik #8 (2007-01-28, 12:49)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
Quote by gajownik:
Hehe, a ja wręcz przeciwnie :D
Jak widać zależy na kogo się trafi ;) czy tak czy owak nie wydaje się, aby był to szczególny problem. Moim zdaniem i jedna i druga forma jest dobra i nie czepiałbym się tego jako recenzent paczki (i też nigdy się nie czepiam ;))
EDIT: sprawdziłem właśnie paczkę mockiem i wygląda na to, że faktycznie brakuje tylko tego desktop-file-install w BR.
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
This post was edited on 2007-01-28, 12:56 by ecik.
Avatar
Reply · Quote gajownik #9 (2007-01-28, 12:57)
User title: GNU fanatic!
since May 2006 · 206 posts · Location: Bytom
Group memberships: Administratorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Off Topic:
Ja tylko tak pisałem bazując na własnym doświadczeniu i późniejszych problemach - za coś takiego też bym nie zablokował pakietu ;)
Get counted!
Do rozwiązywania problemów z Fedorą służy forum!
SM/Fx/M + AdBlock Plus + Adblock Filterset.G Updater == [Image: http://browsehappy.com/buttons/bh_80x15.gif]
Avatar
Reply · Quote ecik #10 (2007-01-28, 13:01)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
Off Topic:
I zapewne masz rację, choć ja jeszcze problemów związanych z użyciem tego makra nie doświadczyłem ;)
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
Avatar
Reply · Quote gajownik #11 (2007-01-28, 13:15)
User title: GNU fanatic!
since May 2006 · 206 posts · Location: Bytom
Group memberships: Administratorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Off Topic:
Jak się siedzi po nocach i modyfikuje/przegląda kilka(naście) plików spec to może czasami coś się pokiełbasić. A tak to od razu widać jaki plik się edytuje ;-)
Get counted!
Do rozwiązywania problemów z Fedorą służy forum!
SM/Fx/M + AdBlock Plus + Adblock Filterset.G Updater == [Image: http://browsehappy.com/buttons/bh_80x15.gif]
Avatar
Reply · Quote ecik #12 (2007-01-28, 13:17)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
Off Topic:
No tak racja - mam w swoim dorobku trochę recenzji różnych pakietów, więc wiem jak to jest ;)
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
Avatar
Reply · Quote adios #13 (2007-01-28, 20:52)
User title: Antylopa GNU
since May 2006 · 189 posts · Location: Mielec
Group memberships: Administratorzy, Moderatorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
3 wersja paczuszki już jest...
Pierwsze kroki w Fedora Core - http://poradnik.fedorapl.org
adios.blog - http://adiospl.wordpress.com
Avatar
Reply · Quote ecik #14 (2007-01-28, 21:03)
User title: Paczker
since Sep 2006 · 49 posts · Location: Rybnik
Group memberships: Użytkownicy
Show profile · Link to this post
I wszystko byłoby idealnie (paczka buduje się ładnie w mocku), gdyby nie:
W: gnome-color-chooser mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 36)
Oznacza to, że raz w SPEC-u używasz wielospacji, a raz robisz tabulatory. Może to jest trochę śmieszne, ale powinieneś to poprawić, żeby uciszyć rpmlinta ;)
Poza tym, ale to już tylko moja osobista sugestia, radziłbym się troszkę bardziej rozpisywać w %changelogu, bo z niego tak naprawdę kompletnie nic nie wiadomo :)
Off Topic:
Pisząc komentarze w bugzilli przy wysyłaniu nowych paczek ładnie jest przy każdym releasie podawać nowy URL do SRPM-a ;)
Off Topic:
Jeśli będziesz tworzył następne paczki zawsze, oprócz oficjalnych Guidelines, możesz je "przepuścić" przez moje FAQ ;)
Paczker i fan Fedory
... moje zapiski...
... i strona na wiki
Avatar
Reply · Quote adios #15 (2007-01-28, 21:13)
User title: Antylopa GNU
since May 2006 · 189 posts · Location: Mielec
Group memberships: Administratorzy, Moderatorzy, Użytkownicy, Zespół fedorapl.org
Show profile · Link to this post
Quote by ecik:
Oznacza to, że raz w SPEC-u używasz wielospacji, a raz robisz tabulatory. Może to jest trochę śmieszne, ale powinieneś to poprawić, żeby uciszyć rpmlinta ;)

Łooo ;-) Dwa tabulatory czyta jako spacje + tabulator ;-) Tak z tego wynika ;-)
Cóż dopiszę coś do changelogu i paczki jak znalazł ;-)
Pierwsze kroki w Fedora Core - http://poradnik.fedorapl.org
adios.blog - http://adiospl.wordpress.com
Reply to this post:
Verification code: VeriCode Please note the verification code from the picture into the text field next to it.
Close Smaller – Larger +
Page:  1  2  next

Go to forum
Powered by Unclassified NewsBoard (© 2003-7 by Yves Goergen)
Current time: 2009-01-07, 05:25:00 (UTC +01:00)
Copyright © 2006 - 2007 fedorapl.org