kicad.spec

[Bartosz Świątek]

W dniu 3 stycznia 2011 14:57 użytkownik Bartosz Świątek
<shadzik w gmail.com> napisał:
> W dniu 3 stycznia 2011 09:48 użytkownik Daniel Dawid Majewski
> <light-i w o2.pl> napisał:
>> W odpowiedzi na wiadomość z dnia 03.01.2011 09:20, od Bartosz Świątek:
>>>>> +%{__cmake} -DCMAKE_VERBOSE_MAKEFILE=ON \
>>>>> +-DCMAKE_BUILD_TYPE=Release \
>>>>> +-DCMAKE_CXX_FLAGS="-m64" \
>>>>> +-DwxUSE_UNICODE=ON \
>>>>> +-DKICAD_KEEPCASE=ON \
>>>>> +-DKICAD_MINIZIP=OFF \
>>>>> +-DCMAKE_INSTALL_PREFIX=%{_prefix} \
>>>>> +-DCMAKE_SYSTEM_LIBRARY_PATH=%{_libdir} \
>>>>> +-DKICAD_INTERNAT="%{_localedir}" \
>>>>> +-DKICAD_PLUGINS="%{_datadir}/%{name}/plugins" \
>>>>> +-DKICAD_DATA="%{_datadir}/%{name}" \
>>>>> +-DKICAD_DOCS="%{_datadir}/%{name}/doc" \
>>>>> +-DKICAD_HELP="%{_datadir}/%{name}/help" \
>>>>> +-DKICAD_MODULES="%{_datadir}/%{name}/modules" \
>>>>> +-DKICAD_LIBRARY="%{_datadir}/%{name}/library" \
>>>>> + ../
>>> To jest w ogóle pomieszanie z poplątaniem, bo makro cmake jest źle
>>> użyte i nadpisywane są wartości jakimiś bzdurami.
>>> Na twardo podany buildtype mimo ze makro definiuje już rodzaj builda itp. itd.
>> Drobne pytanie : Czy PLD jest zgodne z FHS ? Te definicje porządkują
>> chyba ten temat bardziej logicznie, minizip jest w distro w oddzielnej
>> paczce... Jakieś konkrety ?
>
> A nie podałem konkretnie? No to może wylistuję:
>
> 1) -DCMAKE_VERBOSE_MAKEFILE=ON - zbędne lub powinno być tylko w trybie debug
> 2) -DCMAKE_BUILD_TYPE=Release - totalnie źle i zbędne, dlaczego
> pisałem już wyżej
> 3) -DCMAKE_CXX_FLAGS="-m64" - meeeega źle i w ogóle zbędne, a jeśli
> już to powinno być ustawione na nasze standardowe flagi, co chyba już
> też makro samo w sobie załatwia
> 4) -DCMAKE_INSTALL_PREFIX=%{_prefix}  - definiowane przez makro, więc źle
> 5) -DCMAKE_SYSTEM_LIBRARY_PATH=%{_libdir} - nie chce mi się powtarzać
>
> Innych rzeczy mi się nawet nie chce sprawdzać czy są poprawne. Jak na
> początek to ja widzę 5 błędów w samym wywołaniu cmake, ciekawe ile
> błędów ogólnie się znajdzie.

Aha, jeszcze jeden fajny kwiatek:

+%prep
+%setup -q -a1 -a2 -n %{name}-sources-%{snapshoot}-stable
+%patch0 -p0
+%patch1 -p0
+mkdir build

raz że mkdir w %prep to już dziwna sprawa, jak dobrze pamiętam to u
nas nietolerowana.
Ale jak już będziesz to przesuwał do %build tam gdzie jest miejsce na
takie rzeczy, to proszę zastanów się 500 razy czy użyć mkdir i
dlaczego jednak go nie używać. Sprawdź też w innych specach co jest
używane do stworzenia katalogu "build".



-- 
"I'm living proof if you do one thing right in your career, you can
coast for a long time. A LOOOOONG time." -Guy Kawasaki