On 2 Aug, 2010, at 8:58 PM, Hanspeter Niederstrasser wrote:

> An option like that is generally not kept in Makefiles.  If makefiles 
> are involved, it's more likely as a defines that is then used by the c 
> code.  This command "grep -r default_sound_plugin_name *" suggests that 
> client/options.c is the file to modify.  I see 2 occurrences of 
> default_sound_plugin_name there.  The first one (line 316) sets the 
> default value for the -P option when freeciv is run via the command line 
> (defaults to 0).  The second (line 475) is for the actual checkbox used 
> in the Options dialog.  Presumably setting the first NULL to "sdl" 
> (including the "") changes the setting.  I don't know which of the two 
> locations would need to be changed.  It might be just one or both.
> 
> The following two lines modify each of the two instances.  Make a 
> PatchScript field in the .info file, add them both, and then comment out 
> (prefix with #) as needed until you get the right combination (don't 
> forget to remove ~/.freeciv-client-rc-2.2 between runs to make sure you 
> get a clean start).

        After reading [but perhaps not comprehending?] options.c myself, i have 
the opinion that the first one is irrelevant.  It looks like the '\0' is 
supposed to be overwritten by the list of choices found in the second part.  A 
developer upstream thinks there is a bug there, though i'm waiting to enter it 
into their tracker until i gather some more data.  Anyway, i've built a new 
.deb using your second patch.  The interesting thing is, the new version 
doesn't install a new .freeciv-client-rc-2.2 .  When i 

$ dpkg -x 
/sw/fink/dists/local/main/binary-darwin-i386/freeciv_2.2.2-8_darwin-i386.deb 
~/temp/

        i don't see the rc file at all, just the stuff that goes into /sw .  Is 
there a better way to check this?  I have tinkered with a few other 
ConfigureParams between then and now so it's a possibility that it's not being 
built anymore...

Sent from my MacBookPro

Never trust anyone who says "trust me".  Except just this once.
------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
_______________________________________________
Fink-devel mailing list
Fink-devel@lists.sourceforge.net
http://news.gmane.org/gmane.os.apple.fink.devel
Subscription management:
https://lists.sourceforge.net/lists/listinfo/fink-devel

Reply via email to