Current version of 1.11Beta2 can't play iTunes songs on Winows. Error in path programming?

Bug #1047178 reported by Matt White
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
RJ Skerry-Ryan

Bug Description

Alright, so I have been trying the Mixxx 1.11.0 Beta Pre-releases in hopes that someone gets the Auto Recue (Bug #1022713) script to work. The problem is with testing the recent Windows builds, I've noticed two problems. x64 straight up crashes and doesn't even open, period. As for x86, the path was programmed as "//localhost/C:/.." instead of "C:/.." or even understandably "file://C:/..", however it seems it was a slight path error between compiling for Linux and then compiling for Windows.

I just wanted to give you guys a heads up on this so it doesn't go unnoticed. That aside, you guys have been doing an outstanding job and should really give yourselves a pat on the back for all your hard work.

Related branches

Revision history for this message
Sean M. Pappalardo (pegasus-renegadetech) wrote :

We're still ironing out some issues with the build server. Which revision number(s) did you last test? Can you try again with builds produced today or later and let us know?

Revision history for this message
Sean M. Pappalardo (pegasus-renegadetech) wrote :

Oh, and where did you see the errant file paths?

Revision history for this message
Matt White (djzemar) wrote :

Well I noticed the paths when I attempted to run them out of iTunes or from within the program. I DID, however, get them to load if I externally dragged them into a playlist or the AutoDJ. The lastest build I have tried were yesterday's and today's builds and they would either not run or they would have that path problem. For the latest, I believe it doesn't even show anything before it crashes. But I'll be happy to test any new builds and let you know when this problem is solved.

I tried multiple recent builds and I believe I've had the previously mentioned problems in 3374, 3372, 3371, and I think 3369.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
milestone: none → 1.11.0
importance: Undecided → Critical
Revision history for this message
Matt White (djzemar) wrote :

First of all, I apologize for the late reply, it took a while to do all this testing... Alright, so I tested as many builds as I had to from recent to the older builds until I found the most recent, error-free build; here are my findings.

3374-x64, 3372-x64: Does not start; crashes before anything shows.

3371-x64: Shows "Mixxx has stopped working!" error before anything shows.

3369-x86, 3361-x64, 3360-x64, 3358-x86, 3358-x64, 3352-x86, 3352-x64, 3345-x86, 3345-x64, 3344-x64, 3315-x64, 3314-x86, 3313-x64, 3311-x86, 3311-x64, 3310-x86, 3310-x64: Produces the "//localhost/C:/.." link error when linking from itunes, "could not be found."

3306-x86, 3306-x64: Working; no problems starting up or linking from the iTunes section.
Note: This is the most recent build that I didn't have either errors with.

3110-x86 (Current stable build): Working; no problems starting up or linking from the iTunes section.

My findings are that the proper path should be "C:/.." but it's instead using "//localhost/C:/.." and this seems to occur with the iTunes section in Mixxx. I also noticed that it seems the 3 most recent 64-bit binaries for Windows will simply crash. Lastly, my original purpose for checking the recent builds was to see if the AutoDJ ReCue (Bug #1022713) function was working yet, but so far it's not.. At least not on Windows it isn't. I hope all my bug-testing helped! I really appreciate all your hard work, the development of Mixxx is going quite well.

I will go ahead and continue using build r3306-x64 to make sure I don't find any errors, that way 1.11.0 is a cleaner release.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Thanks so much for putting the time in to track down the revision this broke in Matt!

The commit in question is r3307 which changed iTunes support to support loading iTunes libraries on Linux better. It seems we might have broken something in the process! Could you post your "iTunes Music Library.xml" file that should be located in the "iTunes" folder in your "My Music" folder? That should give us a hint for what is going wrong.

Thanks,
RJ

summary: - Current version of 1.11Beta2 can't play songs on Winows. Error in path
- programming?
+ Current version of 1.11Beta2 can't play iTunes songs on Winows. Error in
+ path programming?
Changed in mixxx:
assignee: nobody → RJ Ryan (rryan)
importance: Critical → High
Revision history for this message
Matt White (djzemar) wrote :

The exact Windows 7 path is "C:\Users\USERNAME\Music\iTunes\iTunes Music Library.xml"
Here's an upload of my library xml: http://up.ht/SeMJMm
Best of luck and let me know if there's anything else I can do to help!

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

My current suspicion is that not all versions of iTunes have a "Music Folder" path in their XML file so we are not able to detect the location of your music (and strip out the //localhost/ in the filenames). I have a patch that might fix this I just need to see your "iTunes Media Library.xml" file to confirm that's the problem.

Revision history for this message
Matt White (djzemar) wrote :

Ah okay, and here I'll upload the library directly to launchpad since I saw that feature after my last comment..

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Thanks -- I found the problem and hopefully have committed a fix in r3404.

We may also have fixed the x64 launch problem (see Bug #1047412) so could you please give the x64 binary a try again?

Changed in mixxx:
status: New → In Progress
Revision history for this message
Matt White (djzemar) wrote :

Alright, in a moment I'll try out the x64 build to assure it both runs as well as finds the iTunes paths... I'll post my results asap.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Oops, looks like I broke the Windows build. The fix is in r3405.

Builds should appear soon in http://builds.mixxx.org/builds/release-1.11.x/r3405/

Revision history for this message
Matt White (djzemar) wrote :

I do apologize greatly for the delay! The x64 binary took a while to compile and I had too much homework this week in college... I just checked the build and it both starts up and loads songs from the iTunes section correctly. You did a great job, best of luck with 1.11!

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Good to hear -- thanks!

Changed in mixxx:
status: In Progress → Fix Committed
Revision history for this message
Matt White (djzemar) wrote :

No problem, glad I could help in some form.. If you ever need someone to test any binaries on Windows 7 (64-bit) to ensure the Windows users are working well, let me know. I'm normally quick with my test results and I really love seeing how well the project is improving!

RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/6623

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.