We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 517

Summary: Default path \TIMIDITY is awkward, please add e.g. C:\Windows\TIMIDITY
Product: SDL_mixer Reporter: dank
Component: miscAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2    
Version: unspecified   
Hardware: x86   
OS: Windows (All)   
Attachments: Patch to implement desired behavior without breaking current users

Description dank 2007-11-18 05:32:54 UTC 
The default path \\TIMIDITY has two problems:
1) if the current drive is D:, and timidity.cfg is in C:\TIMIDITY,
it won't be found.  Relative drive names are almost never a good idea
when looking for config files.

2) On Wine, the default drive is usually Z:\, the whole Linux
drive.  And we can't really put a symlink /timidity -> /etc/timidity
just for Wine.  But we could very easily arrange for e.g.
C:\Windows\Timidity to point to /etc/timidity.

I'll attach a suggested patch.  (Hi Sam!)
Comment 1 dank 2007-11-18 05:34:40 UTC 
Created attachment 236 [details]
Patch to implement desired behavior without breaking current users

By the way, see http://bugs.winehq.org/show_bug.cgi?id=10492
for a recipe for reproducing the problem in real life.
Comment 2 dank 2007-11-18 05:39:08 UTC 
I should mention that patch is untested, it's just to show the idea.
Comment 3 Trygve Vea 2008-07-07 08:49:54 UTC 
A little advice

Instead of using C:, I'd suggest you fetch the environment variable %SYSTEMDRIVE%. Perhaps fall back on C: if the environment variable (for some reason) does not exist.

It is possible to remap the drives in Windows - a common process on Citrix-solutions. %SYSTEMDRIVE%\timidity\timidity.cfg should work regardless of the letter of the system drive.

Regards, Trygve
Comment 4 Sam Lantinga 2009-10-03 02:51:40 UTC 
Hi Dan! :)

This is fixed for the next release.  I default Windows to C:\TIMIDITY and added a TIMIDITY_CFG environment variable so you can fully specify the timidity.cfg file.

Thanks!