It is there: https://github.com/sidmutha/xapian/blob/master/xapian-core/api/Makefile.mk#L53 On an other note, I'm trying to run ./bootstrap from the cloned repo. But it gives an error saying "unknown option -C" followed by "Bootstrap failed". *Siddhant Mutha* Undergraduate Student Department of Computer Science and Engineering IIT Madras Chennai http://www.siddhantmutha.com/ <http:/www.siddhantmutha.com/> On Sun, Apr 13, 2014 at 11:55 PM, James Aylett <james-xapian at tartarus.org>wrote:> On 13 Apr 2014, at 19:21, Pallavi Gudipati <pallavigudipati at gmail.com> > wrote: > > > trie.cc still isnt showing up. > > http://pastebin.com/c89kWz3L > > > > Also we have shifted to git: https://github.com/sidmutha/xapian > > Your commit in github isn't showing any change to api/Makefile.mk; is that > not applied in your tree? > > J > > -- > James Aylett, occasional trouble-maker > xapian.org > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xapian.org/pipermail/xapian-devel/attachments/20140414/44055583/attachment-0002.html>
On 13 Apr 2014, at 19:30, Siddhant Mutha <siddhantmutha at gmail.com> wrote:> It is there: https://github.com/sidmutha/xapian/blob/master/xapian-core/api/Makefile.mk#L53Apologies, it's in the commit but the commit message is in a format which confused me for a minute.> On an other note, I'm trying to run ./bootstrap from the cloned repo. But it gives an error saying "unknown option -C" followed by "Bootstrap failed".Yes, you need to either upgrade git or roll back to b3e3f03d5954ab11e8e2f0952dedd5cef0d72358 (or just git revert 246a4c11d4f34a193f656a6b4aa70946426981ae for the time being); we'll be fixing that properly soon. You need to delete line 23 of include/xapian/trie.h, which includes a file which no longer exists on master. (You could have worked off the tag for 1.2.17, which is what I was doing previously, but working off master is better.) You still have to fix all your (*it)->value problems (you seem to have caught two out of four), and there's another problem where your loop var should be an unsigned int which I forgot to point out beforehand. Hopefully once you get bootstrap working you'll be able to see these failures yourself. The patch I've applied to your master (on top of the git revert above) is: diff --git a/xapian-core/api/trie.cc b/xapian-core/api/trie.cc index eba3a15..b856f39 100644 --- a/xapian-core/api/trie.cc +++ b/xapian-core/api/trie.cc @@ -33,10 +33,10 @@ Trie::Trie() { void Trie::add_term(std::string term) { struct trie_node *curr_node = &root; - for (int i = 0; i < term.size(); ++i) { + for (unsigned int i = 0; i < term.size(); ++i) { for (vector<trie_node *>::iterator it = curr_node->children.begi it != curr_node->children.end(); ++it) { - if (*it->value == term[i]) { + if ((*it)->value == term[i]) { curr_node = *it; break; } else if(it + 1 == curr_node->children.end() || @@ -55,10 +55,10 @@ Trie::add_term(std::string term) { bool Trie::search_term(std::string term) { struct trie_node *curr_node = &root; - for (int i = 0; i < term.size(); ++i) { + for (unsigned int i = 0; i < term.size(); ++i) { for (vector<trie_node *>::iterator it = curr_node->children.begi it != curr_node->children.end(); ++it) { - if (*it->value == term[i]) { + if ((*it)->value == term[i]) { curr_node = *it; break; } else if(it + 1 == curr_node->children.end() || diff --git a/xapian-core/include/xapian/trie.h b/xapian-core/include/xapian/trie index 14de006..79b2371 100644 --- a/xapian-core/include/xapian/trie.h +++ b/xapian-core/include/xapian/trie.h @@ -20,7 +20,6 @@ #ifndef XAPIAN_INCLUDED_TRIE_H #define XAPIAN_INCLUDED_TRIE_H -#include <xapian/base.h> #include <xapian/visibility.h> #include <string> J -- James Aylett, occasional trouble-maker xapian.org
Hi The above problem has been resolved (it is detecting our new modules). We didn't have some dependencies installed and hence it was using an old Makefile. We installed them and now it is fine. Now, while running 'make', we get the following error: http://pastebin.com/VX1HWyJc *Siddhant Mutha* Undergraduate Student Department of Computer Science and Engineering IIT Madras Chennai http://www.siddhantmutha.com/ <http:/www.siddhantmutha.com/> On Mon, Apr 14, 2014 at 12:10 AM, James Aylett <james-xapian at tartarus.org>wrote:> On 13 Apr 2014, at 19:30, Siddhant Mutha <siddhantmutha at gmail.com> wrote: > > > It is there: > https://github.com/sidmutha/xapian/blob/master/xapian-core/api/Makefile.mk#L53 > > Apologies, it's in the commit but the commit message is in a format which > confused me for a minute. > > > On an other note, I'm trying to run ./bootstrap from the cloned repo. > But it gives an error saying "unknown option -C" followed by "Bootstrap > failed". > > Yes, you need to either upgrade git or roll back to > b3e3f03d5954ab11e8e2f0952dedd5cef0d72358 (or just git revert > 246a4c11d4f34a193f656a6b4aa70946426981ae for the time being); we'll be > fixing that properly soon. > > You need to delete line 23 of include/xapian/trie.h, which includes a file > which no longer exists on master. (You could have worked off the tag for > 1.2.17, which is what I was doing previously, but working off master is > better.) > > You still have to fix all your (*it)->value problems (you seem to have > caught two out of four), and there's another problem where your loop var > should be an unsigned int which I forgot to point out beforehand. Hopefully > once you get bootstrap working you'll be able to see these failures > yourself. The patch I've applied to your master (on top of the git revert > above) is: > > diff --git a/xapian-core/api/trie.cc b/xapian-core/api/trie.cc > index eba3a15..b856f39 100644 > --- a/xapian-core/api/trie.cc > +++ b/xapian-core/api/trie.cc > @@ -33,10 +33,10 @@ Trie::Trie() { > void > Trie::add_term(std::string term) { > struct trie_node *curr_node = &root; > - for (int i = 0; i < term.size(); ++i) { > + for (unsigned int i = 0; i < term.size(); ++i) { > for (vector<trie_node *>::iterator it > curr_node->children.begi > it != curr_node->children.end(); ++it) { > - if (*it->value == term[i]) { > + if ((*it)->value == term[i]) { > curr_node = *it; > break; > } else if(it + 1 == curr_node->children.end() || > @@ -55,10 +55,10 @@ Trie::add_term(std::string term) { > bool > Trie::search_term(std::string term) { > struct trie_node *curr_node = &root; > - for (int i = 0; i < term.size(); ++i) { > + for (unsigned int i = 0; i < term.size(); ++i) { > for (vector<trie_node *>::iterator it > curr_node->children.begi > it != curr_node->children.end(); ++it) { > - if (*it->value == term[i]) { > + if ((*it)->value == term[i]) { > curr_node = *it; > break; > } else if(it + 1 == curr_node->children.end() || > diff --git a/xapian-core/include/xapian/trie.h > b/xapian-core/include/xapian/trie > index 14de006..79b2371 100644 > --- a/xapian-core/include/xapian/trie.h > +++ b/xapian-core/include/xapian/trie.h > @@ -20,7 +20,6 @@ > #ifndef XAPIAN_INCLUDED_TRIE_H > #define XAPIAN_INCLUDED_TRIE_H > > -#include <xapian/base.h> > #include <xapian/visibility.h> > > #include <string> > > > J > > -- > James Aylett, occasional trouble-maker > xapian.org > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xapian.org/pipermail/xapian-devel/attachments/20140414/acc133c7/attachment-0002.html>