After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 741953 - Rygel MediaExport continues to list deleted media files
Rygel MediaExport continues to list deleted media files
Status: RESOLVED FIXED
Product: rygel
Classification: Applications
Component: MediaExport plugin
0.24.x
Other Linux
: Normal normal
: ---
Assigned To: Jens Georg
rygel-maint
Depends on:
Blocks:
 
 
Reported: 2014-12-24 22:49 UTC by Nicholas Miell
Modified: 2015-01-02 17:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
media-export: Move file information query (2.37 KB, patch)
2015-01-02 17:06 UTC, Jens Georg
committed Details | Review

Description Nicholas Miell 2014-12-24 22:49:36 UTC
If you delete (or rename) a media file exported by Rygel, the now non-existent file continues to be reported to clients.
Comment 1 Jens Georg 2014-12-29 20:02:24 UTC
Can you somehow check whether this works with 0.22?
Comment 2 Nicholas Miell 2014-12-29 20:06:29 UTC
Rygel 0.20 shipped with Fedora 20, it broke when I updated to Fedora 21 and Rygel 0.24.

I have no idea of Rygel 0.22 works or not, is that packaged somewhere?
Comment 3 Jens Georg 2014-12-31 11:45:22 UTC
Nah, I suppose that's enough information. I think this might be the fix for 734303
Comment 4 Jens Georg 2014-12-31 11:46:42 UTC
I meant Bug 734303
Comment 5 Jens Georg 2015-01-02 17:06:53 UTC
Created attachment 293609 [details] [review]
media-export: Move file information query

The code would also try to query file information on removed files which
would silently fail on the remove event and never propagate the change
information to the parent folders.

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 6 Jens Georg 2015-01-02 17:09:44 UTC
Attachment 293609 [details] pushed as 0816317 - media-export: Move file information query
Comment 7 Jens Georg 2015-01-02 17:11:18 UTC
The fix for bug 694455 caused this.