gg.spec

[Tomek Orzechowski]

[29 maj, 2002] Tomasz Witek napisał:

>> >> Tak byc nie moze, poprzednie rozwiazanie orzecha bylo ok.
>> >> To bez sensu z ta zmienna, TAK BYC NIE MOZE
>> >> 
>> >> Oddawac to :)
>> >sie wytlumacz dlaczego? :)
>> >przeca jest znacznie krotszy i jasniejszy ten patch... 
>> >co najwyzej wrzucilbym zmienna homedir poza strukture GGConfig...
>> 
>> Ja też nie wiem, dlaczego Tiwkowi nie podoba się druga wersja... Jedna
>> uwaga: s/homedir/configdir/ homedir kojarzy się raczej z $HOME niż
>> $HOME/$CONFIG_DIR...
>> 
>
>Nie podoba mi sie reczna konfiguracja pliku konfiguracyjnego.
>i ot co.
>
>To co ty Zrobiles bylo jakies czytelniejsze.

końcówka home_etc.patch
---->
+       config.homedir = NULL;
+       if (getenv("CONFIG_DIR"))
+           config.homedir = g_strconcat(getenv("HOME"),
+               getenv("CONFIG_DIR"), "/gg", NULL);
+       else
+           config.homedir = g_strconcat(getenv("HOME"), "/.gg", NULL);
<----

Nie widzę tutaj nic ustawianego 'ręcznie'...

Ten kod jest IMVHO ładniejszy - zamiast kilku
if (getenv("CONFIG_DIR"))... jest zrobione raz i porządnie.

-- 
Tomek Orzechowski