Review request Bug comment: https://bugzilla.gnome.org/show_bug.cgi?id=709368#c4 Commit to review: https://git.gnome.org/browse/tracker/commit/?h=gnome-screenshot&id=2e0f0f6e9fe3c2284974f915258072b1575e3779 Example usage: pvanhoof sput-debian:~/repos/gnome/tracker$ gnome-screenshot -f /home/pvanhoof/screenshot.png ** Message: Unable to use GNOME Shell's builtin screenshot interface, resorting to fallback X11. pvanhoof sput-debian:~/repos/gnome/tracker$ tracker-control -f /home/pvanhoof/screenshot.png (Re)indexing file was successful pvanhoof sput-debian:~/repos/gnome/tracker$ tracker-sparql -q "SELECT ?u { ?f nie:isPartOf nfo:image-category-screenshot; nie:url ?u }" Results: file:///home/pvanhoof/screenshot.png file:///home/pvanhoof/Pictures/Screenshot%20from%202013-10-08%2010:05:25.png pvanhoof sput-debian:~/repos/gnome/tracker$ ps. A review from Debarshi on the example usage is also welcome. Kind regards, Philip Debarshi Ray schreef op 7/10/2013 14:57: Hey Martyn, On Mon, Oct 07, 2013 at 12:28:32PM +0100, Martyn Russell wrote:Currently albums or collections are implemented using nfo:DataContainer, and screenshots are identified by checking the tEXt::Software option for 'gnome-screenshot'.I don't follow regarding the "tEXt::Software option" bit?See this: https://git.gnome.org/browse/gnome-control-center/tree/panels/background/bg-pictures-source.c#n167I am wondering if the filesystem miner could look at tEXt::Software and create a nfo:DataContainer for all photos that have it. Ofcourse it could be under a optional --enable-gnome flag because this might be specific to GNOME.Makes sense. The hard part is knowing what's a photo and what's a screenshot? By filename? By location? By metadata in the file itself?The "tEXt::Software option" is embedded into the file if you took your screenshot using gnome-shell or gnome-screenshot. Cheers, Debarshi |