From owner-svn-ports-head@FreeBSD.ORG Mon Dec 16 14:45:40 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 333CCDAA; Mon, 16 Dec 2013 14:45:40 +0000 (UTC) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 1D74D15A7; Mon, 16 Dec 2013 14:45:40 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.7/8.14.7) with ESMTP id rBGEjehv077010; Mon, 16 Dec 2013 14:45:40 GMT (envelope-from wg@svn.freebsd.org) Received: (from wg@localhost) by svn.freebsd.org (8.14.7/8.14.7/Submit) id rBGEjdQj077008; Mon, 16 Dec 2013 14:45:39 GMT (envelope-from wg@svn.freebsd.org) Message-Id: <201312161445.rBGEjdQj077008@svn.freebsd.org> From: William Grzybowski Date: Mon, 16 Dec 2013 14:45:39 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r336641 - in head/net/minidlna: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Dec 2013 14:45:40 -0000 Author: wg Date: Mon Dec 16 14:45:39 2013 New Revision: 336641 URL: http://svnweb.freebsd.org/changeset/ports/336641 Log: net/minidlna: use newer kqueue patch - Use newer kqueue patch - Use autotools to regenerate configure required for the patch Modified: head/net/minidlna/Makefile head/net/minidlna/files/patch-kqueue Modified: head/net/minidlna/Makefile ============================================================================== --- head/net/minidlna/Makefile Mon Dec 16 14:27:40 2013 (r336640) +++ head/net/minidlna/Makefile Mon Dec 16 14:45:39 2013 (r336641) @@ -3,7 +3,7 @@ PORTNAME= minidlna PORTVERSION= 1.1.1 -PORTREVISION= 2 +PORTREVISION= 3 PORTEPOCH= 1 CATEGORIES= net multimedia www MASTER_SITES= SF @@ -21,11 +21,13 @@ LIB_DEPENDS= libsqlite3.so:${PORTSDIR}/d libavformat.so:${PORTSDIR}/multimedia/ffmpeg GNU_CONFIGURE= yes -CONFIGURE_ARGS+=--with-db-path=/var/db/minidlna \ +CONFIGURE_ARGS= --with-db-path=/var/db/minidlna \ --with-os-url="http://www.FreeBSD.org" CFLAGS+= -I${LOCALBASE}/include LDFLAGS+= -L${LOCALBASE}/lib USES= iconv +USE_AUTOTOOLS= autoconf aclocal automake +AUTOMAKE_ARGS= --add-missing PKGMESSAGE= ${WRKSRC}/pkg-message SUB_FILES+= pkg-message Modified: head/net/minidlna/files/patch-kqueue ============================================================================== --- head/net/minidlna/files/patch-kqueue Mon Dec 16 14:27:40 2013 (r336640) +++ head/net/minidlna/files/patch-kqueue Mon Dec 16 14:45:39 2013 (r336641) @@ -1,6 +1,5 @@ -diff -ruN configure.ac.orig configure.ac ---- configure.ac.orig 2013-11-02 02:06:41.000000000 +0100 -+++ configure.ac 2013-11-21 13:22:46.000000000 +0100 +--- ./configure.ac.orig 2013-11-02 05:06:41.000000000 +0400 ++++ ./configure.ac 2013-11-13 17:25:27.000000000 +0400 @@ -453,7 +453,7 @@ ################################################################################################################ ### Header checks @@ -10,11 +9,13 @@ diff -ruN configure.ac.orig configure.ac AC_CHECK_FUNCS(inotify_init, AC_DEFINE(HAVE_INOTIFY,1,[Whether kernel has inotify support]), [ AC_MSG_CHECKING([for __NR_inotify_init syscall]) -diff -ruN inotify.c.orig inotify.c ---- inotify.c.orig 2013-11-02 02:06:41.000000000 +0100 -+++ inotify.c 2013-11-21 13:49:58.000000000 +0100 -@@ -17,7 +17,7 @@ +--- ./inotify.c.orig 2013-11-02 05:06:41.000000000 +0400 ++++ ./inotify.c 2013-11-13 17:25:27.000000000 +0400 +@@ -15,9 +15,10 @@ + * You should have received a copy of the GNU General Public License + * along with MiniDLNA. If not, see . */ ++ #include "config.h" -#ifdef HAVE_INOTIFY @@ -22,7 +23,7 @@ diff -ruN inotify.c.orig inotify.c #include #include #include -@@ -31,11 +31,16 @@ +@@ -31,11 +32,16 @@ #include #include #include @@ -40,13 +41,15 @@ diff -ruN inotify.c.orig inotify.c #endif #include "libav.h" -@@ -49,11 +54,13 @@ +@@ -49,11 +55,13 @@ #include "playlist.h" #include "log.h" +-#define EVENT_SIZE ( sizeof (struct inotify_event) ) +-#define BUF_LEN ( 1024 * ( EVENT_SIZE + 16 ) ) +#ifdef HAVE_INOTIFY - #define EVENT_SIZE ( sizeof (struct inotify_event) ) - #define BUF_LEN ( 1024 * ( EVENT_SIZE + 16 ) ) ++#define EVENT_SIZE ( sizeof (struct inotify_event) ) ++#define BUF_LEN ( 1024 * ( EVENT_SIZE + 16 ) ) #define DESIRED_WATCH_LIMIT 65536 #define PATH_BUF_SIZE PATH_MAX @@ -54,58 +57,55 @@ diff -ruN inotify.c.orig inotify.c struct watch { -@@ -86,6 +93,7 @@ +@@ -86,12 +94,34 @@ struct watch *nw; int wd; -+ #ifdef HAVE_INOTIFY ++#ifdef HAVE_INOTIFY wd = inotify_add_watch(fd, path, IN_CREATE|IN_CLOSE_WRITE|IN_DELETE|IN_MOVE); if( wd < 0 ) { -@@ -93,6 +101,25 @@ + DPRINTF(E_ERROR, L_INOTIFY, "inotify_add_watch(%s) [%s]\n", path, strerror(errno)); return -1; } - +#else /*HAVE_INOTIFY*/ -+ wd = open(path, O_RDONLY); -+ if (wd == -1) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "inotify_add_watch[kqueue,open](%s) [%s]\n", path, strerror(errno)); -+ return -1; -+ } -+ struct kevent ke; -+ EV_SET(&ke, wd, -+ EVFILT_VNODE, -+ EV_ADD | EV_ENABLE | EV_CLEAR, -+ NOTE_DELETE | NOTE_WRITE | NOTE_EXTEND /*| NOTE_ATTRB*/, -+ 0, NULL); -+ if( kevent(fd, &ke, 1, NULL, 0, NULL) == -1 ) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "inotify_add_watch[kqueue](%s) [%s]\n", path, strerror(errno)); -+ return -1; -+ } -+#endif ++ wd = open(path, O_RDONLY); ++ if (wd == -1) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "inotify_add_watch[kqueue,open](%s) [%s]\n", path, strerror(errno)); ++ return -1; ++ } ++ ++ struct kevent ke; ++ EV_SET(&ke, wd, ++ EVFILT_VNODE, ++ EV_ADD | EV_ENABLE | EV_CLEAR, ++ NOTE_DELETE | NOTE_WRITE | NOTE_EXTEND /*| NOTE_ATTRB*/, ++ 0, NULL); ++ ++ if( kevent(fd, &ke, 1, NULL, 0, NULL) == -1 ) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "inotify_add_watch[kqueue](%s) [%s]\n", path, strerror(errno)); ++ return -1; ++ } ++#endif + nw = malloc(sizeof(struct watch)); if( nw == NULL ) - { -@@ -125,9 +152,15 @@ +@@ -125,7 +155,12 @@ for( w = watches; w; w = w->next ) { if( strcmp(path, w->path) == 0 ) -+ #ifdef HAVE_INOTIFY ++#ifdef HAVE_INOTIFY return(inotify_rm_watch(fd, w->wd)); -- } -+ #else ++#else + close(w->wd); /* kqueue cleans up events when handle dies*/ + return(0); -+ #endif -+ } ++#endif + } -+ return 1; - } - -@@ -145,8 +178,7 @@ +@@ -145,15 +180,14 @@ int inotify_create_watches(int fd) { @@ -115,406 +115,681 @@ diff -ruN inotify.c.orig inotify.c char **result; int i, rows = 0; struct media_dir_s * media_path; -@@ -165,6 +197,9 @@ + + for( media_path = media_dirs; media_path != NULL; media_path = media_path->next ) + { +- DPRINTF(E_DEBUG, L_INOTIFY, "Add watch to %s\n", media_path->path); ++ DPRINTF(E_DEBUG, L_INOTIFY, "Setiing up monitoring on %s\n", media_path->path); + add_watch(fd, media_path->path); + num_watches++; + } +@@ -165,7 +199,11 @@ num_watches++; } sqlite3_free_table(result); -+ #ifdef HAVE_INOTIFY -+ FILE * max_watches; -+ unsigned int watch_limit; - +- ++ ++#ifdef HAVE_INOTIFY ++ FILE * max_watches; ++ unsigned int watch_limit; ++ max_watches = fopen("/proc/sys/fs/inotify/max_user_watches", "r"); if( max_watches ) -@@ -204,7 +239,7 @@ - DPRINTF(E_WARN, L_INOTIFY, "WARNING: Could not read inotify max_user_watches! " - "Hopefully it is enough to cover %u current directories plus any new ones added.\n", num_watches); + { +@@ -194,16 +232,17 @@ + else + { + DPRINTF(E_WARN, L_INOTIFY, "WARNING: Inotify max_user_watches [%u] is low or close to the number of used watches [%u] " +- "and I do not have permission to increase this limit. Please do so manually by " +- "writing a higher value into /proc/sys/fs/inotify/max_user_watches.\n", watch_limit, num_watches); ++ "and I do not have permission to increase this limit. Please do so manually by " ++ "writing a higher value into /proc/sys/fs/inotify/max_user_watches.\n", watch_limit, num_watches); + } + } + } + else + { +- DPRINTF(E_WARN, L_INOTIFY, "WARNING: Could not read inotify max_user_watches! " +- "Hopefully it is enough to cover %u current directories plus any new ones added.\n", num_watches); ++ DPRINTF(E_WARN, L_INOTIFY, "WARNING: Could not read inotify max_user_watches! " ++ "Hopefully it is enough to cover %u current directories plus any new ones added.\n", num_watches); } -- +#endif + return rows; } - -@@ -218,7 +253,11 @@ +@@ -218,7 +257,11 @@ while( w ) { last_w = w; -+ #ifdef HAVE_INOTIFY ++#ifdef HAVE_INOTIFY inotify_rm_watch(fd, w->wd); -+ #else -+ close(w->wd); /*kqueue cleans up after fhandle dies*/ -+ #endif ++#else ++ close(w->wd); /*kqueue cleans up after fhandle dies*/ ++#endif free(w->path); rm_watches++; w = w->next; -@@ -294,6 +333,8 @@ +@@ -261,10 +304,10 @@ + while( (e = readdir(ds)) ) + { + if( strcmp(e->d_name, ".") == 0 || +- strcmp(e->d_name, "..") == 0 ) ++ strcmp(e->d_name, "..") == 0 ) + continue; + if( (e->d_type == DT_DIR) || +- (e->d_type == DT_UNKNOWN && resolve_unknown_type(dir, NO_MEDIA) == TYPE_DIR) ) ++ (e->d_type == DT_UNKNOWN && resolve_unknown_type(dir, NO_MEDIA) == TYPE_DIR) ) + i += add_dir_watch(fd, dir, e->d_name); + } + } +@@ -294,6 +337,8 @@ struct media_dir_s * media_path = media_dirs; struct stat st; + DPRINTF(E_DEBUG, L_INOTIFY, "inotify_insert_file: %s @ %s\n", name, path); -+ ++ /* Is it cover art for another file? */ if( is_image(path) ) update_if_album_art(path); -@@ -421,7 +462,7 @@ +@@ -314,25 +359,25 @@ + { + case ALL_MEDIA: + if( !is_image(path) && +- !is_audio(path) && +- !is_video(path) && +- !is_playlist(path) ) ++ !is_audio(path) && ++ !is_video(path) && ++ !is_playlist(path) ) + return -1; + break; + case TYPE_AUDIO: + if( !is_audio(path) && +- !is_playlist(path) ) ++ !is_playlist(path) ) + return -1; + break; + case TYPE_AUDIO|TYPE_VIDEO: + if( !is_audio(path) && +- !is_video(path) && +- !is_playlist(path) ) ++ !is_video(path) && ++ !is_playlist(path) ) + break; + case TYPE_AUDIO|TYPE_IMAGES: + if( !is_image(path) && +- !is_audio(path) && +- !is_playlist(path) ) ++ !is_audio(path) && ++ !is_playlist(path) ) + return -1; + break; + case TYPE_VIDEO: +@@ -341,14 +386,14 @@ + break; + case TYPE_VIDEO|TYPE_IMAGES: + if( !is_image(path) && +- !is_video(path) ) ++ !is_video(path) ) + return -1; + break; + case TYPE_IMAGES: + if( !is_image(path) ) + return -1; + break; +- default: ++ default: + return -1; + break; + } +@@ -371,11 +416,11 @@ + inotify_remove_file(path); + } + +- /* Find the parentID. If it's not found, create all necessary parents. */ ++ /* Find the parentID. If it's not found, create all necessary parents. */ + len = strlen(path)+1; + if( !(path_buf = malloc(len)) || +- !(last_dir = malloc(len)) || +- !(base_name = malloc(len)) ) ++ !(last_dir = malloc(len)) || ++ !(base_name = malloc(len)) ) + return -1; + base_copy = base_name; + while( depth ) +@@ -386,9 +431,9 @@ + + do + { +- //DEBUG DPRINTF(E_DEBUG, L_INOTIFY, "Checking %s\n", parent_buf); ++ /*DEBUG DPRINTF(E_DEBUG, L_INOTIFY, "Checking %s\n", parent_buf);*/ + id = sql_get_text_field(db, "SELECT OBJECT_ID from OBJECTS o left join DETAILS d on (d.ID = o.DETAIL_ID)" +- " where d.PATH = '%q' and REF_ID is NULL", parent_buf); ++ " where d.PATH = '%q' and REF_ID is NULL", parent_buf); + if( id ) + { + if( !depth ) +@@ -421,13 +466,13 @@ if( !depth ) { - //DEBUG DPRINTF(E_DEBUG, L_INOTIFY, "Inserting %s\n", name); -+ DPRINTF(E_DEBUG, L_INOTIFY, "Inserting %s\n", name); ++ DPRINTF(E_DEBUG, L_INOTIFY, "Inserting %s\n", name); insert_file(name, path, id+2, get_next_available_id("OBJECTS", id)); sqlite3_free(id); if( (is_audio(path) || is_playlist(path)) && next_pl_fill != 1 ) -@@ -446,6 +487,8 @@ + { + next_pl_fill = time(NULL) + 120; // Schedule a playlist scan for 2 minutes from now. +- //DEBUG DPRINTF(E_WARN, L_INOTIFY, "Playlist scan scheduled for %s", ctime(&next_pl_fill)); ++ //DEBUG DPRINTF(E_WARN, L_INOTIFY, "Playlist scan scheduled for %s", ctime(&next_pl_fill)); + } + } + return depth; +@@ -446,6 +491,8 @@ struct media_dir_s* media_path; struct stat st; + DPRINTF(E_DEBUG, L_INOTIFY, "inotify_insert_directory: %s @ %s\n", name, path); -+ ++ if( access(path, R_OK|X_OK) != 0 ) { DPRINTF(E_WARN, L_INOTIFY, "Could not access %s [%s]\n", path, strerror(errno)); -@@ -537,6 +580,7 @@ - char **result; +@@ -459,7 +506,7 @@ + + parent_buf = strdup(path); + id = sql_get_text_field(db, "SELECT OBJECT_ID from OBJECTS o left join DETAILS d on (d.ID = o.DETAIL_ID)" +- " where d.PATH = '%q' and REF_ID is NULL", dirname(parent_buf)); ++ " where d.PATH = '%q' and REF_ID is NULL", dirname(parent_buf)); + if( !id ) + id = sqlite3_mprintf("%s", BROWSEDIR_ID); + insert_directory(name, path, BROWSEDIR_ID, id+2, get_next_available_id("OBJECTS", id)); +@@ -538,9 +585,12 @@ int64_t detailID; int rows, playlist; -+ DPRINTF(E_DEBUG, L_INOTIFY, "inotify_remove_file: %s\n", path); ++ DPRINTF(E_DEBUG, L_INOTIFY, "inotify_remove_file: %s\n", path); ++ if( ends_with(path, ".srt") ) { -@@ -582,6 +626,8 @@ +- return sql_exec(db, "DELETE from CAPTIONS where PATH = '%q'", path); ++ rows = sql_exec(db, "DELETE from CAPTIONS where PATH = '%q'", path); ++ return rows; + } + /* Invalidate the scanner cache so we don't insert files into non-existent containers */ + valid_cache = 0; +@@ -554,27 +604,25 @@ + { + sql_exec(db, "DELETE from PLAYLISTS where ID = %lld", detailID); + sql_exec(db, "DELETE from DETAILS where ID =" +- " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s$%llX')", +- MUSIC_PLIST_ID, detailID); ++ " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s$%llX')", ++ MUSIC_PLIST_ID, detailID); + sql_exec(db, "DELETE from OBJECTS where OBJECT_ID = '%s$%llX' or PARENT_ID = '%s$%llX'", +- MUSIC_PLIST_ID, detailID, MUSIC_PLIST_ID, detailID); ++ MUSIC_PLIST_ID, detailID, MUSIC_PLIST_ID, detailID); + } + else + { + /* Delete the parent containers if we are about to empty them. */ +- snprintf(sql, sizeof(sql), "SELECT PARENT_ID from OBJECTS where DETAIL_ID = %lld" +- " and PARENT_ID not like '64$%%'", +- (long long int)detailID); ++ snprintf(sql, sizeof(sql), "SELECT PARENT_ID from OBJECTS where DETAIL_ID = %lld", (long long int)detailID); + if( (sql_get_table(db, sql, &result, &rows, NULL) == SQLITE_OK) ) + { + int i, children; +- for( i = 1; i <= rows; i++ ) ++ for( i=1; i <= rows; i++ ) + { + /* If it's a playlist item, adjust the item count of the playlist */ + if( strncmp(result[i], MUSIC_PLIST_ID, strlen(MUSIC_PLIST_ID)) == 0 ) + { + sql_exec(db, "UPDATE PLAYLISTS set FOUND = (FOUND-1) where ID = %d", +- atoi(strrchr(result[i], '$') + 1)); ++ atoi(strrchr(result[i], '$') + 1)); + } + + children = sql_get_int_field(db, "SELECT count(*) from OBJECTS where PARENT_ID = '%s'", result[i]); +@@ -582,6 +630,8 @@ continue; if( children < 2 ) { + sql_exec(db, "DELETE from DETAILS where ID =" -+ " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s')", result[i]); ++ " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s')", result[i]); sql_exec(db, "DELETE from OBJECTS where OBJECT_ID = '%s'", result[i]); ptr = strrchr(result[i], '$'); -@@ -589,7 +635,9 @@ +@@ -589,6 +639,8 @@ *ptr = '\0'; if( sql_get_int_field(db, "SELECT count(*) from OBJECTS where PARENT_ID = '%s'", result[i]) == 0 ) { -- sql_exec(db, "DELETE from OBJECTS where OBJECT_ID = '%s'", result[i]); -+ sql_exec(db, "DELETE from DETAILS where ID =" -+ " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s')", result[i]); -+ sql_exec(db, "DELETE from OBJECTS where OBJECT_ID = '%s'", result[i]); ++ sql_exec(db, "DELETE from DETAILS where ID =" ++ " (SELECT DETAIL_ID from OBJECTS where OBJECT_ID = '%s')", result[i]); + sql_exec(db, "DELETE from OBJECTS where OBJECT_ID = '%s'", result[i]); } } - } -@@ -612,7 +660,7 @@ - char **result; +@@ -613,11 +665,13 @@ int64_t detailID = 0; int rows, i, ret = 1; -- + + DPRINTF(E_DEBUG, L_INOTIFY, "inotify_remove_directory: %s\n", path); ++ /* Invalidate the scanner cache so we don't insert files into non-existent containers */ valid_cache = 0; remove_watch(fd, path); -@@ -638,7 +686,7 @@ - + sql = sqlite3_mprintf("SELECT ID from DETAILS where (PATH > '%q/' and PATH <= '%q/%c')" +- " or PATH = '%q'", path, path, 0xFF, path); ++ " or PATH = '%q'", path, path, 0xFF, path); + if( (sql_get_table(db, sql, &result, &rows, NULL) == SQLITE_OK) ) + { + if( rows ) +@@ -639,6 +693,7 @@ return ret; } -- + +#ifdef HAVE_INOTIFY void * start_inotify() { -@@ -755,4 +803,218 @@ +@@ -649,7 +704,7 @@ + int length, i = 0; + char * esc_name = NULL; + struct stat st; +- ++ + pollfds[0].fd = inotify_init(); + pollfds[0].events = POLLIN; + +@@ -664,13 +719,13 @@ + } + inotify_create_watches(pollfds[0].fd); + if (setpriority(PRIO_PROCESS, 0, 19) == -1) +- DPRINTF(E_WARN, L_INOTIFY, "Failed to reduce inotify thread priority\n"); ++ DPRINTF(E_WARN, L_INOTIFY, "Failed to reduce inotify thread priority\n"); + sqlite3_release_memory(1<<31); + av_register_all(); +- ++ + while( !quitting ) + { +- length = poll(pollfds, 1, timeout); ++ length = poll(pollfds, 1, timeout); + if( !length ) + { + if( next_pl_fill && (time(NULL) >= next_pl_fill) ) +@@ -682,14 +737,14 @@ + } + else if( length < 0 ) + { +- if( (errno == EINTR) || (errno == EAGAIN) ) +- continue; +- else ++ if( (errno == EINTR) || (errno == EAGAIN) ) ++ continue; ++ else + DPRINTF(E_ERROR, L_INOTIFY, "read failed!\n"); + } + else + { +- length = read(pollfds[0].fd, buffer, BUF_LEN); ++ length = read(pollfds[0].fd, buffer, BUF_LEN); + } + + i = 0; +@@ -707,12 +762,12 @@ + sprintf(path_buf, "%s/%s", get_path_from_wd(event->wd), event->name); + if ( event->mask & IN_ISDIR && (event->mask & (IN_CREATE|IN_MOVED_TO)) ) + { +- DPRINTF(E_DEBUG, L_INOTIFY, "The directory %s was %s.\n", ++ DPRINTF(E_DEBUG, L_INOTIFY, "The directory %s was %s.\n", + path_buf, (event->mask & IN_MOVED_TO ? "moved here" : "created")); + inotify_insert_directory(pollfds[0].fd, esc_name, path_buf); + } + else if ( (event->mask & (IN_CLOSE_WRITE|IN_MOVED_TO|IN_CREATE)) && +- (lstat(path_buf, &st) == 0) ) ++ (lstat(path_buf, &st) == 0) ) + { + if( S_ISLNK(st.st_mode) ) + { +@@ -726,7 +781,7 @@ + else if( event->mask & (IN_CLOSE_WRITE|IN_MOVED_TO) && st.st_size > 0 ) + { + if( (event->mask & IN_MOVED_TO) || +- (sql_get_int_field(db, "SELECT TIMESTAMP from DETAILS where PATH = '%q'", path_buf) != st.st_mtime) ) ++ (sql_get_int_field(db, "SELECT TIMESTAMP from DETAILS where PATH = '%q'", path_buf) != st.st_mtime) ) + { + DPRINTF(E_DEBUG, L_INOTIFY, "The file %s was %s.\n", + path_buf, (event->mask & IN_MOVED_TO ? "moved here" : "changed")); +@@ -755,4 +810,234 @@ return 0; } -+ +#else +void * +start_kqueue() +{ -+ int global_kqueue_handle = -1; ++ int global_kqueue_handle = -1; + -+ global_kqueue_handle = kqueue(); -+ if ( global_kqueue_handle < 0 ) -+ DPRINTF(E_ERROR, L_INOTIFY, "kqueue() failed: %s\n", strerror(errno)); -+ -+ while( scanning ) -+ { -+ if( quitting ) -+ goto quitting; -+ sleep(1); -+ } -+ -+ inotify_create_watches(global_kqueue_handle); -+ if (setpriority(PRIO_PROCESS, 0, 19) == -1) -+ DPRINTF(E_WARN, L_INOTIFY, "Failed to reduce kqueue thread priority\n"); -+ sqlite3_release_memory(1<<31); -+ av_register_all(); -+ -+ while( !quitting ) -+ { -+ struct kevent ke; -+ if ( kevent(global_kqueue_handle, NULL, 0, &ke, 1, NULL) == -1 ) -+ { -+ DPRINTF(E_WARN, L_INOTIFY, "kevent polling failure: %s\n", strerror(errno)); -+ } -+ -+ /*DPRINTF(E_DEBUG, L_INOTIFY, "GOT KEVENT:\n" -+ "ident=0x%X, filter=0x%X, flags=0x%X, fflags=0x%X, data=0x%X, udata=0x%X\n", -+ ke.ident, ke.filter, ke.flags, ke.fflags, ke.data, ke.udata);*/ -+ char* dir_path = get_path_from_wd(ke.ident); -+ if (dir_path == NULL) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "Path with FD=0x%X can't be resolved.\n", ke.ident); -+ continue; -+ } -+ -+ -+ -+ if (ke.fflags & NOTE_DELETE) -+ { -+ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] deleted.\n", dir_path); -+ inotify_remove_directory(ke.ident, dir_path); -+ } -+ else if ((ke.fflags & (NOTE_WRITE | NOTE_LINK)) == (NOTE_WRITE | NOTE_LINK)) -+ { -+ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] content updated (directory).\n", dir_path); -+ char * sql; -+ char **result; -+ int i, rows; -+ sql = sqlite3_mprintf("SELECT PATH from DETAILS where (PATH > '%q/' and PATH <= '%q/%c')" -+ " and SIZE = ''", dir_path, dir_path, 0xFF); -+ if( (sql_get_table(db, sql, &result, &rows, NULL) != SQLITE_OK) ) -+ { -+ DPRINTF(E_WARN, L_INOTIFY, "Read state [%s]: Query failed, not updating\n", dir_path); -+ sqlite3_free(sql); -+ continue; -+ } -+ -+ -+ for( i=1; i <= rows; i++ ) -+ { -+ DPRINTF(E_DEBUG, L_INOTIFY, "Indexed content: %s\n", result[i]); -+ if (access(result[i], R_OK) == -1) -+ { -+ /* actually, global_kqueue_handle is not used here */ -+ inotify_remove_directory(global_kqueue_handle, result[i]); -+ } -+ } -+ DIR* d; -+ struct dirent * entry; -+ d = opendir(dir_path); -+ if (!d) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "Can't list [%s] (%s)\n", dir_path, strerror(errno)); -+ continue; -+ } -+ -+ for ( entry = readdir(d); entry != NULL; entry = readdir(d) ) -+ { -+ if ( (entry->d_type == DT_DIR) && -+ (strcmp(entry->d_name, "..") != 0) && -+ (strcmp(entry->d_name, ".") != 0) ) -+ { -+ -+ char tmp_path[PATH_MAX]; -+ int result_path_len; -+ result_path_len = snprintf(tmp_path, PATH_MAX, -+ "%s/%s", dir_path, entry->d_name); -+ if (result_path_len >= PATH_MAX) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "File path too long for %s!", entry->d_name); -+ continue; -+ } -+ DPRINTF(E_DEBUG, L_INOTIFY, "Walking %s\n", tmp_path); -+ char found_flag = 0; -+ for( i=1; i <= rows; i++ ) -+ { -+ if (strcmp(result[i], tmp_path) == 0) -+ { -+ found_flag = 1; -+ break; -+ } -+ } -+ if ( !found_flag ) -+ { -+ char * esc_name = NULL; -+ esc_name = modifyString(strdup(entry->d_name), "&", "&amp;"); -+ inotify_insert_directory(global_kqueue_handle, esc_name, tmp_path); -+ free(esc_name); -+ } -+ } -+ } -+ -+ sqlite3_free_table(result); -+ sqlite3_free(sql); -+ } -+ else if (ke.fflags & NOTE_WRITE) -+ { -+ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] content updated (file).\n", dir_path); -+ char * sql; -+ char **result; -+ int i, rows; -+ sql = sqlite3_mprintf("SELECT PATH from DETAILS where (PATH > '%q/' and PATH <= '%q/%c')" -+ " and SIZE <> ''", dir_path, dir_path, 0xFF); -+ if( (sql_get_table(db, sql, &result, &rows, NULL) != SQLITE_OK) ) -+ -+ { -+ DPRINTF(E_WARN, L_INOTIFY, "Read state [%s]: Query failed, not updating\n", dir_path); -+ sqlite3_free(sql); -+ continue; -+ } -+ for( i=1; i <= rows; i++ ) -+ { -+ DPRINTF(E_DEBUG, L_INOTIFY, "Indexed content: %s\n", result[i]); -+ if (access(result[i], R_OK) == -1) /*oops, our file is gone*/ -+ { -+ inotify_remove_file(result[i]); -+ } -+ } -+ -+ DIR* d; -+ struct dirent * entry; -+ d = opendir(dir_path); -+ if (!d) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "Can't list [%s] (%s)\n", dir_path, strerror(errno)); -+ continue; -+ } -+ -+ for ( entry = readdir(d); entry != NULL; entry = readdir(d) ) -+ { -+ if ( (entry->d_type == DT_REG) || -+ (entry->d_type == DT_LNK) ) -+ { -+ char tmp_path[PATH_MAX]; -+ int result_path_len; -+ result_path_len = snprintf(tmp_path, PATH_MAX, -+ "%s/%s", dir_path, entry->d_name); -+ if (result_path_len >= PATH_MAX) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "File path too long for %s!", entry->d_name); -+ continue; -+ } -+ -+ DPRINTF(E_DEBUG, L_INOTIFY, "Walking %s\n", tmp_path); -+ char found_flag = 0; -+ for( i=1; i <= rows; i++ ) -+ { -+ if (strcmp(result[i], tmp_path) == 0) -+ { -+ found_flag = 1; -+ break; -+ } -+ } -+ if ( !found_flag ) -+ { -+ char * esc_name = NULL; -+ struct stat st; -+ if( stat(tmp_path, &st) != 0 ) -+ { -+ DPRINTF(E_ERROR, L_INOTIFY, "'%s' disappeared!", tmp_path); -+ continue; -+ } -+ -+ esc_name = modifyString(strdup(entry->d_name), "&", "&amp;"); -+ if ( S_ISDIR(st.st_mode) ) -+ inotify_insert_directory(global_kqueue_handle, esc_name, tmp_path); -+ else -+ inotify_insert_file(esc_name, tmp_path); -+ free(esc_name); -+ } -+ } -+ } -+ -+ -+ -+ sqlite3_free_table(result); -+ sqlite3_free(sql); -+ } -+ } ++ global_kqueue_handle = kqueue(); ++ if ( global_kqueue_handle < 0 ) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "kqueue() failed: %s\n", strerror(errno)); ++ return 0; ++ } ++ ++ while( scanning ) ++ { ++ if( quitting ) ++ goto quitting; ++ ++ DPRINTF(E_DEBUG, L_INOTIFY, "..waiting for scanning to complete...\n"); ++ sleep(1); ++ } ++ ++ DPRINTF(E_DEBUG, L_INOTIFY, "GOING WILD!\n"); ++ ++ inotify_create_watches(global_kqueue_handle); ++ if (setpriority(PRIO_PROCESS, 0, 19) == -1) ++ DPRINTF(E_WARN, L_INOTIFY, "Failed to reduce kqueue thread priority\n"); ++ sqlite3_release_memory(1<<31); ++ av_register_all(); ++ ++ while( !quitting ) ++ { ++ struct kevent ke; ++ if ( kevent(global_kqueue_handle, NULL, 0, &ke, 1, NULL) == -1 ) ++ { ++ DPRINTF(E_WARN, L_INOTIFY, "kevent polling failure: %s\n", strerror(errno)); ++ continue; ++ } ++ ++ /*DPRINTF(E_DEBUG, L_INOTIFY, "GOT KEVENT:\n" ++ "ident=0x%X, filter=0x%X, flags=0x%X, fflags=0x%X, data=0x%X, udata=0x%X\n", ++ ke.ident, ke.filter, ke.flags, ke.fflags, ke.data, ke.udata);*/ ++ ++ char* dir_path = get_path_from_wd(ke.ident); ++ if (dir_path == NULL) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "Path with FD=0x%X can't be resolved.\n", ke.ident); ++ continue; ++ } ++ ++ if (ke.fflags & NOTE_DELETE) ++ { ++ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] deleted.\n", dir_path); ++ inotify_remove_directory(ke.ident, dir_path); ++ } ++ else if ((ke.fflags & (NOTE_WRITE | NOTE_LINK)) == (NOTE_WRITE | NOTE_LINK)) ++ { ++ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] content updated (directory).\n", dir_path); ++ ++ char * sql; ++ char **result; ++ int i, rows; ++ sql = sqlite3_mprintf("SELECT PATH from DETAILS where (PATH > '%q/' and PATH <= '%q/%c')" ++ " and SIZE = ''", dir_path, dir_path, 0xFF); ++ if( (sql_get_table(db, sql, &result, &rows, NULL) != SQLITE_OK) ) ++ { ++ DPRINTF(E_WARN, L_INOTIFY, "Read state [%s]: Query failed, not updating\n", dir_path); ++ sqlite3_free(sql); ++ continue; ++ } ++ ++ for( i=1; i <= rows; i++ ) ++ { ++ DPRINTF(E_DEBUG, L_INOTIFY, "Indexed content: %s\n", result[i]); ++ if (access(result[i], R_OK) == -1) ++ { ++ /* actually, global_kqueue_handle is not used here */ ++ inotify_remove_directory(global_kqueue_handle, result[i]); ++ } ++ } + -+ inotify_remove_watches(global_kqueue_handle); ++ DIR* d; ++ struct dirent * entry; ++ d = opendir(dir_path); ++ if (!d) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "Can't list [%s] (%s)\n", dir_path, strerror(errno)); ++ continue; ++ } ++ ++ for ( entry = readdir(d); entry != NULL; entry = readdir(d) ) ++ { ++ if ( (entry->d_type == DT_DIR) && ++ (strcmp(entry->d_name, "..") != 0) && ++ (strcmp(entry->d_name, ".") != 0) ) ++ { ++ char tmp_path[PATH_MAX]; ++ int result_path_len; ++ ++ result_path_len = snprintf(tmp_path, PATH_MAX, ++ "%s/%s", dir_path, entry->d_name); ++ if (result_path_len >= PATH_MAX) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "File path too long for %s!", entry->d_name); ++ continue; ++ } ++ ++ DPRINTF(E_DEBUG, L_INOTIFY, "Walking %s\n", tmp_path); ++ char found_flag = 0; ++ for( i=1; i <= rows; i++ ) ++ { ++ if (strcmp(result[i], tmp_path) == 0) ++ { ++ found_flag = 1; ++ break; ++ } ++ } ++ ++ if ( !found_flag ) ++ { ++ char * esc_name = NULL; ++ esc_name = modifyString(strdup(entry->d_name), "&", "&amp;"); ++ inotify_insert_directory(global_kqueue_handle, esc_name, tmp_path); ++ free(esc_name); ++ } ++ } ++ } ++ ++ closedir(d); ++ ++ sqlite3_free_table(result); ++ sqlite3_free(sql); ++ } ++ else if (ke.fflags & NOTE_WRITE) ++ { ++ DPRINTF(E_DEBUG, L_INOTIFY, "Path [%s] content updated (file).\n", dir_path); ++ ++ char * sql; ++ char **result; ++ int i, rows; ++ sql = sqlite3_mprintf("SELECT PATH from DETAILS where (PATH > '%q/' and PATH <= '%q/%c')" ++ " and SIZE <> ''", dir_path, dir_path, 0xFF); ++ if( (sql_get_table(db, sql, &result, &rows, NULL) != SQLITE_OK) ) ++ { ++ DPRINTF(E_WARN, L_INOTIFY, "Read state [%s]: Query failed, not updating\n", dir_path); ++ sqlite3_free(sql); ++ continue; ++ } ++ ++ for( i=1; i <= rows; i++ ) ++ { ++ DPRINTF(E_DEBUG, L_INOTIFY, "Indexed content: %s\n", result[i]); ++ if (access(result[i], R_OK) == -1) /*oops, our file is gone*/ ++ { ++ inotify_remove_file(result[i]); ++ } ++ } ++ ++ DIR* d; ++ struct dirent * entry; ++ d = opendir(dir_path); ++ if (!d) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "Can't list [%s] (%s)\n", dir_path, strerror(errno)); ++ continue; ++ } ++ ++ for ( entry = readdir(d); entry != NULL; entry = readdir(d) ) ++ { ++ if ( (entry->d_type == DT_REG) || ++ (entry->d_type == DT_LNK) ) ++ { ++ char tmp_path[PATH_MAX]; ++ int result_path_len; ++ ++ result_path_len = snprintf(tmp_path, PATH_MAX, ++ "%s/%s", dir_path, entry->d_name); ++ if (result_path_len >= PATH_MAX) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "File path too long for %s!", entry->d_name); ++ continue; ++ } ++ ++ DPRINTF(E_DEBUG, L_INOTIFY, "Walking %s\n", tmp_path); ++ ++ char found_flag = 0; ++ for( i=1; i <= rows; i++ ) ++ { ++ if (strcmp(result[i], tmp_path) == 0) ++ { ++ found_flag = 1; ++ break; ++ } ++ } ++ ++ if ( !found_flag ) ++ { ++ char * esc_name = NULL; ++ struct stat st; ++ ++ if( stat(tmp_path, &st) != 0 ) ++ { ++ DPRINTF(E_ERROR, L_INOTIFY, "'%s' disappeared!", tmp_path); ++ continue; ++ } ++ ++ esc_name = modifyString(strdup(entry->d_name), "&", "&amp;"); ++ if ( S_ISDIR(st.st_mode) ) ++ inotify_insert_directory(global_kqueue_handle, esc_name, tmp_path); ++ else ++ inotify_insert_file(esc_name, tmp_path); ++ free(esc_name); ++ } ++ } ++ } ++ ++ closedir(d); ++ ++ sqlite3_free_table(result); ++ sqlite3_free(sql); ++ } ++ } ++ inotify_remove_watches(global_kqueue_handle); +quitting: + -+ return 0; ++ return 0; +} -+ #endif ++ +#endif // defined(HAVE_INOTIFY) || defined(HAVE_SYS_EVENT_H) *** DIFF OUTPUT TRUNCATED AT 1000 LINES ***