Bug 10636 - Rescan w/ "Look for new and changed" doesn't noticed deleted flac w/ embedded cuesheet
: Rescan w/ "Look for new and changed" doesn't noticed deleted flac w/ embedded...
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.4.2
: PC Ubuntu Linux
: P1 normal (vote)
: 7.5.0
Assigned To: Andy Grundman
: TinySC
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-01-10 03:18 UTC by Manoj Kasichainula
Modified: 2010-04-25 02:31 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
cuesheet from one of the flacs in my ripped CD collection (657 bytes, text/plain)
2009-02-02 20:09 UTC, Manoj Kasichainula
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Manoj Kasichainula 2009-01-10 03:18:41 UTC
I've reproduced this successfully a couple of times now.

Much of my library is flacs of whole CDs w/ embedded cuesheets. When I delete one of those flacs, and do a rescan with "Look for new and changed music", the tracks from that album are still present in the database. This doesn't happen when I delete the more standard single-track mp3s.

A "Clear library and rescan everything" does work, of course, and I'm using that as a workaround for now.

Version: 7.3.2 - 24555 @ Thu Jan 8 03:13:03 PST 2009
Platform Architecture: x86_64-linux
Perl Version: 5.10.0 - x86_64-linux-gnu-thread-multi
MySQL Version: 5.0.67-0ubuntu6
Comment 1 Chris Owens 2009-01-30 10:18:12 UTC
Manoj, could you attach one of the affected cue files?  Thanks!
Comment 2 Ross Levine 2009-02-02 12:58:07 UTC
How did you create the files with embedded cuesheets? Using Foobar and a script? If you could attach that would be really great, if not maybe you could help me create one. :-)
Comment 3 Manoj Kasichainula 2009-02-02 20:07:59 UTC
Oops, sorry for the delay. I created the flacs w/ cuesheets myself, with some custom scripts using the flac and metaflac tools. I add title/artist/etc. metadata with another script that scoops the details out of my cddb cache, but that won't show up in the cuesheet.

I pasted a sample cuesheet below, and will also attach it (in case some formatting  aspect might be the problem).

Note that also changing title/artist/etc. metadata doesn't get recognized unless I  clear and rescan. And I have confirmed that the mtime of the flac file does indeed get changed when I retag it.

Because I don't have the affected cue files anymore (the flacs containing them were deleted after all). I just pasted another cuesheet from my collection instead. This does seem reproducible on every flac I've tried it on (maybe 5 deleted lately, and dozens have had metadata changed) so I don't think that'll be an issue.

FILE "/home/warez/audio/archive/ea12b50e/1.flac" FLAC
  TRACK 01 AUDIO
    INDEX 01 00:00:00
  TRACK 02 AUDIO
    INDEX 01 00:52:35
  TRACK 03 AUDIO
    INDEX 01 06:37:33
  TRACK 04 AUDIO
    INDEX 01 12:59:17
  TRACK 05 AUDIO
    INDEX 01 19:37:33
  TRACK 06 AUDIO
    INDEX 01 25:51:21
  TRACK 07 AUDIO
    INDEX 01 31:12:66
  TRACK 08 AUDIO
    INDEX 01 36:06:30
  TRACK 09 AUDIO
    INDEX 01 41:37:48
  TRACK 10 AUDIO
    INDEX 01 47:44:56
  TRACK 11 AUDIO
    INDEX 01 53:13:17
  TRACK 12 AUDIO
    INDEX 01 58:55:72
  TRACK 13 AUDIO
    INDEX 01 64:19:23
  TRACK 14 AUDIO
    INDEX 01 71:57:31
REM FLAC__lead-in 88200
REM FLAC__lead-out 170 211199604
Comment 4 Manoj Kasichainula 2009-02-02 20:09:01 UTC
Created attachment 4747 [details]
cuesheet from one of the flacs in my ripped CD collection
Comment 5 Ross Levine 2009-02-03 17:58:47 UTC
I'm afraid I'll need a sample file. I've created a flac file with embedded cuesheet and 10 times I've added and removed it from my music directory, chosen 'look for new and changed music' and every time it works exactly as it should. 
Comment 6 Manoj Kasichainula 2009-02-07 02:15:24 UTC
I can provide the methods I used to rip the CD and the scripts (they're pretty simple), or if you have a way for me to get you a sample, I can do that. An actual large sample might be a bad idea to attach to the issue, but let me know. 

Also, I removed another CD image today, and noticed that when it tries to play something from a deleted image, the player and squeezecenter think that the player is still playing even though it stopped. There's a playback progress bar and everything. Probably not directly relevant to this bug, but it does seem like another bug, and it's probably relevant to https://bugs-archive.lyrion.org/show_bug.cgi?id=9534 which also is a problem with failed detection of playback state. I'm adding a note there too.
Comment 7 Manoj Kasichainula 2009-02-19 01:33:04 UTC
URL to sample sent by private e-mail.
Comment 8 Ross Levine 2009-02-19 14:44:36 UTC
(In reply to comment #7)
> URL to sample sent by private e-mail.

Thanks Manoj, I received the file and I'm able to reproduce! I'm investigating and I hope to have something to share soon. Thank you again.
Comment 9 Manoj Kasichainula 2009-06-03 01:10:08 UTC
Hi. What ended up happening to this? Thanks.
Comment 10 Ross Levine 2009-06-03 13:10:58 UTC
Sorry for the delay. This is still easily reproduced but I haven't figured out what is going on. Alan or Andy, if you'd like I have his file here in the office.
Comment 11 Andy Grundman 2009-06-03 13:13:58 UTC
Will try and handle this part of the rescan work.
Comment 12 Andy Grundman 2009-07-29 14:58:55 UTC
Moving 7.4 bugs to 8.0.
Comment 13 Andy Grundman 2009-12-03 10:48:19 UTC
This is fixed in the embedded scanner code.
Comment 14 Manoj Kasichainula 2010-03-19 02:40:36 UTC
I finally got around to testing this (with latest deb out of the testing apt repo, 7.4.3~30375) and it's still broken. I rm -r'ed a directory with one of my whole-CD flacs, did a "Look for new and changed music" rescan, and the old flac still shows up.
Comment 15 Andy Grundman 2010-03-19 06:51:22 UTC
Maybe you missed where I said it was only fixed in embedded.  Please test with that version.
Comment 16 Chris Owens 2010-04-08 17:24:59 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!
Comment 17 Manoj Kasichainula 2010-04-25 02:31:46 UTC
Ah I see, when Andy said "embedded scanner code" I thought it meant the scanner code embedded in the system because I didn't know about the version called "embedded".

I have 7.5.1~30670 from the apt testing repo installed now, and have the same problem. "Look for new and changed music" doesn't notice that I deleted a directory and the flac w/ cuesheet inside.

I get the impression from the forums that "embedded" used to be 7.5 but is now 7.6? But the latest reply on this bug says this should be in something released, so I'm unsure if I should be reopening this or not.