I'm experiencing the following error (have been for a while which is why I've been stalled):
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT buffy.o -MD -MP -MF .deps/buffy.Tpo -c -o buffy.o buffy.c buffy.c: In function ‘mutt_buffy_check’: buffy.c:602:21: error: ‘VirtIncoming’ undeclared (first use in this function) buffy.c:602:21: note: each undeclared identifier is reported only once for each function it appears in make[2]: *** [buffy.o] Error 1 make[2]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make: *** [all] Error 2
Has anybody else hit this?
On Thu, Mar 29, 2012 at 08:06:25AM -0400, Darryl L. Pierce wrote:
I'm experiencing the following error (have been for a while which is why I've been stalled):
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT buffy.o -MD -MP -MF .deps/buffy.Tpo -c -o buffy.o buffy.c buffy.c: In function ‘mutt_buffy_check’: buffy.c:602:21: error: ‘VirtIncoming’ undeclared (first use in this function) buffy.c:602:21: note: each undeclared identifier is reported only once for each function it appears in make[2]: *** [buffy.o] Error 1 make[2]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make: *** [all] Error 2
You need to run configure with --enable-notmuch. Somewhere it's used out of the #ifdef NOTMUCH/#endif block.
david
On Thu, Mar 29, 2012 at 02:23:02PM +0200, David Sterba wrote:
On Thu, Mar 29, 2012 at 08:06:25AM -0400, Darryl L. Pierce wrote:
I'm experiencing the following error (have been for a while which is why I've been stalled):
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT buffy.o -MD -MP -MF .deps/buffy.Tpo -c -o buffy.o buffy.c buffy.c: In function ‘mutt_buffy_check’: buffy.c:602:21: error: ‘VirtIncoming’ undeclared (first use in this function) buffy.c:602:21: note: each undeclared identifier is reported only once for each function it appears in make[2]: *** [buffy.o] Error 1 make[2]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make: *** [all] Error 2
You need to run configure with --enable-notmuch. Somewhere it's used out of the #ifdef NOTMUCH/#endif block.
Since part of our goal (I assumed) was integration of notmuch with the Mutt code, why do we not have this by default? It would seem to make more sense for regression tests to have to explicitly disable rather than enable notmuch.
On Thu, Mar 29, 2012 at 08:25:56AM -0400, Darryl L. Pierce wrote:
On Thu, Mar 29, 2012 at 02:23:02PM +0200, David Sterba wrote:
On Thu, Mar 29, 2012 at 08:06:25AM -0400, Darryl L. Pierce wrote:
I'm experiencing the following error (have been for a while which is why I've been stalled):
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT buffy.o -MD -MP -MF .deps/buffy.Tpo -c -o buffy.o buffy.c buffy.c: In function ‘mutt_buffy_check’: buffy.c:602:21: error: ‘VirtIncoming’ undeclared (first use in this function) buffy.c:602:21: note: each undeclared identifier is reported only once for each function it appears in make[2]: *** [buffy.o] Error 1 make[2]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make: *** [all] Error 2
You need to run configure with --enable-notmuch. Somewhere it's used out of the #ifdef NOTMUCH/#endif block.
Since part of our goal (I assumed) was integration of notmuch with the Mutt code, why do we not have this by default? It would seem to make more sense for regression tests to have to explicitly disable rather than enable notmuch.
We can still leave mutt-kz compilable entirely without notmuch, just for the case there are users out there not interested in notmuch enhancements, but I seriously doubt that :)
Here's the patch which compiles fine without --enable-notmuch ---
From: David Sterba dsterba@suse.cz Date: Mon, 2 Apr 2012 23:18:15 +0200 Subject: [PATCH] build: fix compilation errors without --enable-notmuch
Signed-off-by: David Sterba dsterba@suse.cz --- browser.c | 2 ++ buffy.c | 8 +++++++- curs_main.c | 4 ++++ sidebar.c | 4 ++++ 4 files changed, 17 insertions(+), 1 deletions(-)
diff --git a/browser.c b/browser.c index 2e1e390..89db8f8 100644 --- a/browser.c +++ b/browser.c @@ -272,10 +272,12 @@ folder_format_str (char *dest, size_t destlen, size_t col, char op, const char * break;
case 'n': +#ifdef USE_NOTMUCH if (mx_is_notmuch (folder->ff->name)) { snprintf (tmp, sizeof (tmp), "%%%sd", fmt); snprintf (dest, destlen, tmp, folder->ff->all); } +#endif break;
case 'N': diff --git a/buffy.c b/buffy.c index 7daf734..33a5b92 100644 --- a/buffy.c +++ b/buffy.c @@ -599,7 +599,11 @@ int mutt_buffy_check (int force) #endif
/* fastest return if there are no mailboxes */ - if (!Incoming && !VirtIncoming) +#ifdef USE_NOTMUCH + if (!Incomming && !VirtIncoming) + return 0; +#endif + if (!Incoming) return 0; t = time (NULL); if (!force && (t - BuffyTime < BuffyTimeout)) @@ -624,8 +628,10 @@ int mutt_buffy_check (int force) for (tmp = Incoming; tmp; tmp = tmp->next) buffy_check(tmp, &contex_sb);
+#ifdef USE_NOTMUCH for (tmp = VirtIncoming; tmp; tmp = tmp->next) buffy_check(tmp, &contex_sb); +#endif
BuffyDoneTime = BuffyTime; return (BuffyCount); diff --git a/curs_main.c b/curs_main.c index 653b226..21ebcb8 100644 --- a/curs_main.c +++ b/curs_main.c @@ -758,7 +758,9 @@ int mutt_index_menu (void) mutt_curs_set (1); /* fallback from the pager */ }
+#ifdef USE_NOTMUCH nm_debug_check(Context); +#endif
switch (op) { @@ -2366,7 +2368,9 @@ int mutt_index_menu (void) km_error_key (MENU_MAIN); }
+#ifdef USE_NOTMUCH nm_debug_check(Context); +#endif
if (menu->menu == MENU_PAGER) { diff --git a/sidebar.c b/sidebar.c index 6eff2f0..8074043 100644 --- a/sidebar.c +++ b/sidebar.c @@ -51,6 +51,7 @@ static BUFFY *get_incoming() case SB_SRC_NONE: sidebar_source = SB_SRC_INCOMING;
+#ifdef USE_NOTMUCH if (option (OPTVIRTSPOOLFILE) && VirtIncoming) { sidebar_source = SB_SRC_VIRT; return VirtIncoming; @@ -61,6 +62,7 @@ static BUFFY *get_incoming() return VirtIncoming; } break; +#endif case SB_SRC_INCOMING: break; } @@ -418,9 +420,11 @@ void toggle_sidebar(int menu) if (sidebar_source == -1) get_incoming();
+#ifdef USE_NOTMUCH if (sidebar_source == SB_SRC_INCOMING && VirtIncoming) sidebar_source = SB_SRC_VIRT; else +#endif sidebar_source = SB_SRC_INCOMING;
TopBuffy = NULL;
On Mon, Apr 02, 2012 at 11:28:18PM +0200, David Sterba wrote:
browser.c | 2 ++ buffy.c | 8 +++++++- curs_main.c | 4 ++++ sidebar.c | 4 ++++ 4 files changed, 17 insertions(+), 1 deletions(-)
Applied, thanks.
Please, please, use git send-email next time ;-)
Karel
On Thu, Apr 05, 2012 at 04:58:10PM +0200, Karel Zak wrote:
Please, please, use git send-email next time ;-)
Sorry, I'll fix it next time and will place the scissor marker instead of just '---', then 'git am --scissor' will see the patch as produced by git-format-patch :)
david
On Thu, Mar 29, 2012 at 02:23:02PM +0200, David Sterba wrote:
You need to run configure with --enable-notmuch. Somewhere it's used out of the #ifdef NOTMUCH/#endif block.
Are there other options tha tmust be enabled by default?
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT curs_main.o -MD -MP -MF .deps/curs_main.Tpo -c -o curs_main.o curs_main.c curs_main.c: In function ‘mutt_index_menu’: curs_main.c:1168:12: error: ‘OP_MAIN_MODIFY_LABELS’ undeclared (first use in this function) curs_main.c:1168:12: note: each undeclared identifier is reported only once for each function it appears in curs_main.c:1169:12: error: ‘OP_MAIN_MODIFY_LABELS_THEN_HIDE’ undeclared (first use in this function) curs_main.c:1238:12: error: ‘OP_MAIN_VFOLDER_FROM_QUERY’ undeclared (first use in this function) curs_main.c:1251:12: error: ‘OP_MAIN_CHANGE_VFOLDER’ undeclared (first use in this function) make[2]: *** [curs_main.o] Error 1 make[2]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/mcpierce/Programming/mutt-kz' make: *** [all] Error 2
On Thu, Mar 29, 2012 at 08:27:56AM -0400, Darryl L. Pierce wrote:
On Thu, Mar 29, 2012 at 02:23:02PM +0200, David Sterba wrote:
You need to run configure with --enable-notmuch. Somewhere it's used out of the #ifdef NOTMUCH/#endif block.
Are there other options tha tmust be enabled by default?
gcc -DPKGDATADIR="/usr/local/share/mutt" -DSYSCONFDIR="/usr/local/etc" -DBINDIR="/usr/local/bin" -DMUTTLOCALEDIR="/usr/local/share/locale" -DHAVE_CONFIG_H=1 -I. -I. -I. -Iintl -I./intl -Wall -pedantic -Wno-long-long -g -O2 -MT curs_main.o -MD -MP -MF .deps/curs_main.Tpo -c -o curs_main.o curs_main.c curs_main.c: In function ‘mutt_index_menu’: curs_main.c:1168:12: error: ‘OP_MAIN_MODIFY_LABELS’ undeclared (first use in this function) curs_main.c:1168:12: note: each undeclared identifier is reported only once for each function it appears in curs_main.c:1169:12: error: ‘OP_MAIN_MODIFY_LABELS_THEN_HIDE’ undeclared (first use in this function) curs_main.c:1238:12: error: ‘OP_MAIN_VFOLDER_FROM_QUERY’ undeclared (first use in this function) curs_main.c:1251:12: error: ‘OP_MAIN_CHANGE_VFOLDER’ undeclared (first use in this function)
are you sure you have the latest updates? I can see all of the errors under USE_NOTMUCH ifdef and compiles with and without --enable-notmuch .
david
On Mon, Apr 02, 2012 at 11:35:00PM +0200, David Sterba wrote:
are you sure you have the latest updates? I can see all of the errors under USE_NOTMUCH ifdef and compiles with and without --enable-notmuch .
I'm pulling from:
git://github.com/karelzak/mutt-kz.git
and on tip for that. Is that not the right repo to be tracking?
On Tue, Apr 03, 2012 at 07:10:58AM -0400, Darryl L. Pierce wrote:
On Mon, Apr 02, 2012 at 11:35:00PM +0200, David Sterba wrote:
are you sure you have the latest updates? I can see all of the errors under USE_NOTMUCH ifdef and compiles with and without --enable-notmuch .
I'm pulling from:
git://github.com/karelzak/mutt-kz.git
and on tip for that. Is that not the right repo to be tracking?
Sure it is, it was a sanity check. I'm able to trigger the same compilation warnings when I do a configure without --enable-notmuch, build and then configure with --enable-notmuch and build. So it's a relic from the previous build.
So 'make distclean' or maintainerclean should clean it and a fresh configure & make will work (tested here).
david
mutt-kz@lists.fedoraproject.org