Bugzilla – Bug 376
Music folder path should be fixed per platform
Last modified: 2008-09-15 14:37:04 UTC
Pages.pm v 1.76 switched to using itempath instead of songurl as a paramter, so now the \'s aren't fixed and the comparison for the audiodir fails: sample data from Pages.pm after line 1076: curdir: d:/mp3 loc: D:\mp3\Professionals\Cherry Poppin' Daddies - Zoot Suit Riot.mp3 thus the downloadurl comes up as $loc and the web interface reports "d is not a recognised protocol"
What happens when your audiodir is set to: d:\mp3 instead of: d:/mp3 (On windows, d:\mp3 is more correct...) If this resolves the problem then we need to make sure that the audiodir has the right kind of slashes in it for the appropriate platform.
that does allow it to work. I've been away from that setup for a while, so I can't say for sure if I changed it or not. It used to work before. Perhaps d:/mp3 should fail if entered, or we normalise the whole / mess :)
So, the real bug is that if you enter: D:/mp3 as your path, it should get fixed to: D:\mp3
the bug is that it validates as ok, when in other tests it would fail. One solution would be to convert when it is entered, or ensure that all tests are slashflip independant.
Fixed in 6.0 tree using canonpath.
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006. I am setting them to targets of 6.2.1 to keep them from showing up in my queries.