Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Hi Andreas, * Andreas Bauer[2016-03-25 19:20 +0100]: > Hi Elimar, > > thanks again for letting me know about this issue and building the > patch. I too have tested the latest patch for a few weeks and just > made a release of it, as it seems to do the trick: > > https://sourceforge.net/projects/mailfilter/files/Mailfilter/0.8.4/ > > I hope that as such, this package can remain in Debian. Would be much > obliged to get a quick confirmation of this, or further instructions > on how I may be able to help in this regard. mailfilter 0.8.4.1-1 has entered Debian unstable today. Our package checker "lintian" found some spell errors which I patched into the package. Please find the patches attached. Thanks for your contribution Elimar -- Alles was viel bedacht wird ist bedenklich!;-) Friedrich Nietzsche Author: Elimar Riesebieter Subject: Fix spellerror in mailfilterex.5 Forwarded: yes Last-Update: 2016-03-28 diff -Naurd a/man/mailfilterex.5 b/man/mailfilterex.5 --- a/man/mailfilterex.5 2014-05-11 17:24:35.0 +0200 +++ b/man/mailfilterex.5 2016-03-28 13:42:18.030238819 +0200 @@ -214,7 +214,7 @@ errors and the like. Examples in this man page all use extended type. .SH "NOTES" If you are new to Regular Expressions and new to Mailfilter, you might want to -experiment a bit, before you accidently delete messages for real. For such +experiment a bit, before you accidentally delete messages for real. For such cases Mailfilter provides two keywords. TEST can be used to only simulate the deletion of messages and SHOW_HEADERS stores all the e-mail headers that get examined by the program. Author: Elimar Riesebieter Subject: Fix spellerror for occured in several files Forwarded: yes Last-Update: 2016-03-28 diff -Naurd a/src/header.cc b/src/header.cc --- a/src/header.cc 2014-05-11 17:16:44.0 +0200 +++ b/src/header.cc 2016-03-28 13:57:57.307154131 +0200 @@ -130,7 +130,7 @@ } catch (const exception& r_err) { - ERROR_MSG("Runtime exception occured while parsing rcfile: " + ERROR_MSG("Runtime exception occurred while parsing rcfile: " + (string)r_err.what ()); exit (-1); } diff -Naurd a/src/mailfilter.cc b/src/mailfilter.cc --- a/src/mailfilter.cc 2014-05-11 17:16:44.0 +0200 +++ b/src/mailfilter.cc 2016-03-28 13:57:12.452489891 +0200 @@ -111,7 +111,7 @@ } catch (const exception& r_err) { - ERROR_MSG("Runtime exception occured: " + ERROR_MSG("Runtime exception occurred: " + (string)r_err.what ()); return -1; } @@ -183,7 +183,7 @@ } catch (const exception& r_err) { - logger->print_err ("Runtime exception occured: " + logger->print_err ("Runtime exception occurred: " + (string)r_err.what ()); return -1; } diff -Naurd a/src/pop3.cc b/src/pop3.cc --- a/src/pop3.cc 2016-02-29 15:45:10.0 +0100 +++ b/src/pop3.cc 2016-03-28 13:58:44.585691415 +0200 @@ -61,14 +61,14 @@ // Send user name and read server reply. if (conn->c_write (usr_name.c_str ()) == -1 || !REPLY_OK) { - logger->print_err("Error occured while sending username to server."); + logger->print_err("Error occurred while sending username to server."); return false; } // Send password and read server reply. if (conn->c_write (pass_wd.c_str ()) == -1 || !REPLY_OK) { - logger->print_err("Error occured while sending password to server."); + logger->print_err("Error occurred while sending password to server."); return false; } @@ -97,7 +97,7 @@ // The function scans the headers inside a POP3 account for spam. It // will delete all spam messages in the account and return 0 when all // the hard work is done. A negative integer is returned if an error -// occured. +// occurred. int POP3 :: scan (void) const { @@ -111,7 +111,7 @@ // Determine number of messages waiting to be examined. if ((num_messages = status ()) < 0) { - logger->print_err ("Error occured while sending STAT to server."); + logger->print_err ("Error occurred while sending STAT to server."); return GEN_FAILURE_FLAG; } @@ -130,7 +130,7 @@ cmd = "LIST " + msg_no.str () + "\r\n"; if (conn->c_write (cmd.c_str ()) == -1 || !REPLY_OK) { - logger->print_err ("Error occured while sending LIST to server."); + logger->print_err ("Error occurred while sending LIST to server."); return GEN_FAILURE_FLAG; } @@ -143,7 +143,7 @@ cmd = "TOP " + msg_no.str () + " 0\r\n"; if (conn->c_write (cmd.c_str ()) == -1 || !HEADER_OK) { - logger->print_err ("Error occured while sending TOP to server."); + logger->print_err ("Error occurred while sending TOP to server."); return GEN_FAILURE_FLAG; } diff -Naurd a/src/socket.cc b/src/socket.cc --- a/src/socket.cc 2016-03-28 13:53:31.0 +0200 +++ b/src/socket.cc
Processed: Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Processing commands for cont...@bugs.debian.org: > tags 812668 fixed-upstream Bug #812668 [mailfilter] mailfilter: FTBFS - error: no match for 'operator=' Added tag(s) fixed-upstream. > End of message, stopping processing here. Please contact me if you need assistance. -- 812668: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=812668 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Hi gents, * Andreas Bauer[2016-03-25 19:20 +0100]: > Hi Elimar, > > thanks again for letting me know about this issue and building the > patch. I too have tested the latest patch for a few weeks and just > made a release of it, as it seems to do the trick: > > https://sourceforge.net/projects/mailfilter/files/Mailfilter/0.8.4/ > > I hope that as such, this package can remain in Debian. Would be much > obliged to get a quick confirmation of this, or further instructions > on how I may be able to help in this regard. I've prepared a lintian cleaned package. You can find it here: http://mentors.debian.net/package/mailfilter Alternatively, one can download the package with dget using this command: dget -x http://mentors.debian.net/debian/pool/main/m/mailfilter/mailfilter_0.8.4-1.dsc Changelog: [ Elimar Riesebieter ] * New upstream version Fixes a FTBS (Closes: 812668) * Fixed some spellerrors via spellfixpatches. * debian/rules: - Introduced DEB_BUILD_MAINT_OPTIONS = hardening=+all - Build contrib.tar.gz without timestamp * debian/control: - Added Homepage field - Changed Standards-Version to 3.9.7. No changes needed. - Add myself to uploaders * debian/copyright: - Changed to dep5-copyright * Added 'set -e' stanza to maintainer scripts Thanks in advance Elimar -- The path to source is always uphill! -unknown- signature.asc Description: PGP signature
Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
* Andreas Bauer[2016-03-25 19:20 +0100]: > Hi Elimar, > > thanks again for letting me know about this issue and building the > patch. I too have tested the latest patch for a few weeks and just > made a release of it, as it seems to do the trick: > > https://sourceforge.net/projects/mailfilter/files/Mailfilter/0.8.4/ > > I hope that as such, this package can remain in Debian. Would be much > obliged to get a quick confirmation of this, or further instructions > on how I may be able to help in this regard. Would one of the uploaders please update mailfilter to 0.8.4? Thanks Elimar -- Numeric stability is probably not all that important when you're guessing;-) signature.asc Description: PGP signature
Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Hi Elimar, thanks again for letting me know about this issue and building the patch. I too have tested the latest patch for a few weeks and just made a release of it, as it seems to do the trick: https://sourceforge.net/projects/mailfilter/files/Mailfilter/0.8.4/ I hope that as such, this package can remain in Debian. Would be much obliged to get a quick confirmation of this, or further instructions on how I may be able to help in this regard. Many thanks, Andreas On Wed, Mar 02, 2016 at 03:14:37PM +0100, Elimar Riesebieter wrote: > Control: tags -1 patch > > * Andreas Bauer[2016-02-29 16:52 +0100]: > > > Hi Elimar, Michael, > > > > just letting you know that in SVN is a release candidate of 0.8.4 of > > mailfilter, which hopefully fixes the issue. It's a bit ugly, hence I > > want to keep testing it for a while, before I release - say, 1 or 2 > > weeks. > > > > Hope it will still be able to keep the package in Debian this way. If > > either of you is curious, you could grab it from SVN and rebuild, etc. > > > > https://sourceforge.net/p/mailfilter/code/HEAD/tree/ > > I have cherry picked the respective patch and can confirm that > mailfilter builds fine on i386, ppc32 and amd64. I've build my own > package with the patch attached for testing. > > Thanks Andreas for contribution ;-) > > Elimar > -- > Numeric stability is probably not all that > important when you're guessing;-) > Author: Andreas Bauer > Subject: Fix compilation issues stemming from flex >= 2.6.0, which no longer > uses pointers for yyin, but references it seems. > Last-Update: 2016-03-02 > Index: mailfilter-0.8.3/ChangeLog > === > --- mailfilter-0.8.3.orig/ChangeLog > +++ mailfilter-0.8.3/ChangeLog > @@ -1,6 +1,13 @@ > Mailfilter ChangeLog > -=-=-=-=-=-=-=-=-=-= > > +Mon Feb 29 16:44:08 CET 2016 baueran > + > +- src/rcfile.ll: yyin is now a reference. So use different ifstream > + pointer and pass it on. > +- src/pop3.cc: FlexLexer::switch_stream no longer has 2nd argument > + optional, it seems. So pass NULL as second object. > + > Sat May 17 19:21:57 CEST 2014 baueran > > src/preferences.cc: > Index: mailfilter-0.8.3/configure.ac > === > --- mailfilter-0.8.3.orig/configure.ac > +++ mailfilter-0.8.3/configure.ac > @@ -26,7 +26,7 @@ > # Process this file with autoconf to produce a configure script. > > AC_PREREQ(2.54) > -AC_INIT(mailfilter, 0.8.3, [baue...@gmail.com]) > +AC_INIT(mailfilter, 0.8.4, [baue...@gmail.com]) > AC_REVISION($Revision: 1.1.2.4.2.12 $) > AC_SUBST(PACKAGE_COPYRIGHT) > AC_DEFINE(PACKAGE_COPYRIGHT, > Index: mailfilter-0.8.3/NEWS > === > --- mailfilter-0.8.3.orig/NEWS > +++ mailfilter-0.8.3/NEWS > @@ -1,6 +1,13 @@ > Mailfilter NEWS (Summary) > -=-=-=-=-=-=-=-=-=-=-=-=- > > +mailfilter 0.8.4 (under development) > + > + - Fix compilation issues stemming from flex >= 2.6.0, which no > +longer uses pointers for yyin, but references it seems. > + > + - Minor other code fixes that relate to the above. > + > mailfilter 0.8.3 (Sat May 17 19:37:27 CEST 2014) > >- (Hopefully) fixed Debian Bug #716522: mailfilter crashes > Index: mailfilter-0.8.3/src/pop3.cc > === > --- mailfilter-0.8.3.orig/src/pop3.cc > +++ mailfilter-0.8.3/src/pop3.cc > @@ -204,7 +204,7 @@ int POP3 :: invoke_msg_parser (const str > stringstream cur_header; > cur_header << *header; > rfclexer = new rfcFlexLexer; > - rfclexer->switch_streams (_header); > + rfclexer->switch_streams (_header, NULL); > int error = rfcparse ((void*) msg_header); > delete rfclexer; > return error; > Index: mailfilter-0.8.3/src/rcfile.ll > === > --- mailfilter-0.8.3.orig/src/rcfile.ll > +++ mailfilter-0.8.3/src/rcfile.ll > @@ -146,17 +146,21 @@ CTRL_CHAR . > cerr << yytext << "' could not be found." << endl; > exit (-1); > } > + > + // yyin is now a std::istream& it seems. So the following no longer > works: > + // yyin = new ifstream (sub_file.c_str ()); > + std::ifstream* infile = new ifstream (sub_file.c_str ()); > > -yyin = new ifstream (sub_file.c_str ()); > - > -if (!((ifstream*) yyin)->is_open ()) > + // if (!((std::ifstream*) yyin)->is_open ()) > +if (!infile->is_open ()) > { > cerr << PACKAGE_NAME << ": Error: Nested rcfile '"; > cerr << sub_file << "' could not be opened." << endl; > exit (-1); > } > > -yy_switch_to_buffer (yy_create_buffer (yyin, YY_BUF_SIZE)); > +
Processed: Re: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Processing control commands: > tags -1 patch Bug #812668 [mailfilter] mailfilter: FTBFS - error: no match for 'operator=' Added tag(s) patch. -- 812668: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=812668 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Control: tags -1 patch * Andreas Bauer[2016-02-29 16:52 +0100]: > Hi Elimar, Michael, > > just letting you know that in SVN is a release candidate of 0.8.4 of > mailfilter, which hopefully fixes the issue. It's a bit ugly, hence I > want to keep testing it for a while, before I release - say, 1 or 2 > weeks. > > Hope it will still be able to keep the package in Debian this way. If > either of you is curious, you could grab it from SVN and rebuild, etc. > > https://sourceforge.net/p/mailfilter/code/HEAD/tree/ I have cherry picked the respective patch and can confirm that mailfilter builds fine on i386, ppc32 and amd64. I've build my own package with the patch attached for testing. Thanks Andreas for contribution ;-) Elimar -- Numeric stability is probably not all that important when you're guessing;-) Author: Andreas Bauer Subject: Fix compilation issues stemming from flex >= 2.6.0, which no longer uses pointers for yyin, but references it seems. Last-Update: 2016-03-02 Index: mailfilter-0.8.3/ChangeLog === --- mailfilter-0.8.3.orig/ChangeLog +++ mailfilter-0.8.3/ChangeLog @@ -1,6 +1,13 @@ Mailfilter ChangeLog -=-=-=-=-=-=-=-=-=-= +Mon Feb 29 16:44:08 CET 2016 baueran + +- src/rcfile.ll: yyin is now a reference. So use different ifstream + pointer and pass it on. +- src/pop3.cc: FlexLexer::switch_stream no longer has 2nd argument + optional, it seems. So pass NULL as second object. + Sat May 17 19:21:57 CEST 2014 baueran src/preferences.cc: Index: mailfilter-0.8.3/configure.ac === --- mailfilter-0.8.3.orig/configure.ac +++ mailfilter-0.8.3/configure.ac @@ -26,7 +26,7 @@ # Process this file with autoconf to produce a configure script. AC_PREREQ(2.54) -AC_INIT(mailfilter, 0.8.3, [baue...@gmail.com]) +AC_INIT(mailfilter, 0.8.4, [baue...@gmail.com]) AC_REVISION($Revision: 1.1.2.4.2.12 $) AC_SUBST(PACKAGE_COPYRIGHT) AC_DEFINE(PACKAGE_COPYRIGHT, Index: mailfilter-0.8.3/NEWS === --- mailfilter-0.8.3.orig/NEWS +++ mailfilter-0.8.3/NEWS @@ -1,6 +1,13 @@ Mailfilter NEWS (Summary) -=-=-=-=-=-=-=-=-=-=-=-=- +mailfilter 0.8.4 (under development) + + - Fix compilation issues stemming from flex >= 2.6.0, which no +longer uses pointers for yyin, but references it seems. + + - Minor other code fixes that relate to the above. + mailfilter 0.8.3 (Sat May 17 19:37:27 CEST 2014) - (Hopefully) fixed Debian Bug #716522: mailfilter crashes Index: mailfilter-0.8.3/src/pop3.cc === --- mailfilter-0.8.3.orig/src/pop3.cc +++ mailfilter-0.8.3/src/pop3.cc @@ -204,7 +204,7 @@ int POP3 :: invoke_msg_parser (const str stringstream cur_header; cur_header << *header; rfclexer = new rfcFlexLexer; - rfclexer->switch_streams (_header); + rfclexer->switch_streams (_header, NULL); int error = rfcparse ((void*) msg_header); delete rfclexer; return error; Index: mailfilter-0.8.3/src/rcfile.ll === --- mailfilter-0.8.3.orig/src/rcfile.ll +++ mailfilter-0.8.3/src/rcfile.ll @@ -146,17 +146,21 @@ CTRL_CHAR . cerr << yytext << "' could not be found." << endl; exit (-1); } + + // yyin is now a std::istream& it seems. So the following no longer works: + // yyin = new ifstream (sub_file.c_str ()); + std::ifstream* infile = new ifstream (sub_file.c_str ()); -yyin = new ifstream (sub_file.c_str ()); - -if (!((ifstream*) yyin)->is_open ()) + // if (!((std::ifstream*) yyin)->is_open ()) +if (!infile->is_open ()) { cerr << PACKAGE_NAME << ": Error: Nested rcfile '"; cerr << sub_file << "' could not be opened." << endl; exit (-1); } -yy_switch_to_buffer (yy_create_buffer (yyin, YY_BUF_SIZE)); +yy_switch_to_buffer (yy_create_buffer (infile, YY_BUF_SIZE)); + // yy_switch_to_buffer (yy_create_buffer (yyin, YY_BUF_SIZE)); } catch (...) { signature.asc Description: PGP signature
Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Hi Elimar, Michael, just letting you know that in SVN is a release candidate of 0.8.4 of mailfilter, which hopefully fixes the issue. It's a bit ugly, hence I want to keep testing it for a while, before I release - say, 1 or 2 weeks. Hope it will still be able to keep the package in Debian this way. If either of you is curious, you could grab it from SVN and rebuild, etc. https://sourceforge.net/p/mailfilter/code/HEAD/tree/ In the meantime, many thanks again for bringing this to my attention! Cheers, Andreas. -- Andreas Bauer, baue...@gmail.com | "Gott stehe uns bei, die Politik http://baueran.multics.org/ | wird es nicht" - Dirk Müller, 2010
Bug#812668: [m...@debian.org] Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
* Michael Tautschnig[2016-02-08 09:02 +]: > Hi Andreas, Elimar, > > A bit more info on the bug report: this is likely caused by the upgrade to > flex > 2.6. Re-generating the lexer may do the trick, see also > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=812681 find . -name '*.ll' | sed -e 's/[.]ll$/.cc/' | xargs rm -rf doesn't help either. If we want the package in Debian we need to fix this bug. mailfilter will be removed from testing soon. > On Wed, Feb 03, 2016 at 21:18:31 +0100, Andreas Bauer wrote: > > Hi Elimar, > > > > will def. look at it. Thanks for pointing this out! > > > > @Michael, long time no see! :-) > > > > Cheers, > > > > Andreas. > > > > > > On Wed, Feb 03, 2016 at 08:54:34PM +0100, Elimar Riesebieter wrote: > > > Hi Andreas, > > > > > > could you please investigate? > > > > > > Thanks > > > Elimar -- The path to source is always uphill! -unknown-
Bug#812668: mailfilter: FTBFS - error: no match for 'operator='
Package: mailfilter Version: 0.8.3-1 Severity: serious Usertags: goto-cc During a rebuild of all Debian packages in a clean sid chroot (using cowbuilder and pbuilder) the build failed with the following error. [...] Making all in src make[3]: Entering directory '/srv/jenkins-slave/workspace/sid-goto-cc-mailfilter/mailfilter-0.8.3/src' gcc -DHAVE_CONFIG_H -I. -I.. -I/usr/include -I. -I../include -I.. -DLOCALEDIR=\"/usr/share/locale\" -I../intl -I.. -I../include -I. -Wdate-time -g -O0 -fstack-protector-strong -Wformat -Werror=format-security -c -o md5c.o md5c.c bison -y -p rc -d -v -oy.tab.c rcfile.yy; \ mv -f y.tab.c rcparser.cc; \ mv -f y.tab.h rcparser.hh; \ g++ -DHAVE_CONFIG_H -I. -I.. -I/usr/include -I. -I../include -I.. -DLOCALEDIR=\"/usr/share/locale\" -I../intl -I.. -I../include -I. -Wdate-time -Wall -g -O0 -fstack-protector-strong -Wformat -Werror=format-security -c rcparser.cc; \ touch y.tab.c flex -+ -i -Prc -orcfile.cc rcfile.ll g++ -DHAVE_CONFIG_H -I. -I.. -I/usr/include -I. -I../include -I.. -DLOCALEDIR=\"/usr/share/locale\" -I../intl -I.. -I../include -I. -Wdate-time -Wall -g -O0 -fstack-protector-strong -Wformat -Werror=format-security -c -o rcfile.o rcfile.cc rcfile.ll: In member function 'virtual int rcFlexLexer::yylex()': rcfile.ll:150:14: error: no match for 'operator=' (operand types are 'std::istream {aka std::basic_istream}' and 'std::ifstream* {aka std::basic_ifstream*}') yyin = new ifstream (sub_file.c_str ()); ^ In file included from /usr/include/c++/5/iostream:40:0, from rcfile.cc:96: /usr/include/c++/5/istream:58:11: note: candidate: std::basic_istream& std::basic_istream::operator=(const std::basic_istream&) class basic_istream : virtual public basic_ios<_CharT, _Traits> ^ /usr/include/c++/5/istream:58:11: note: no known conversion for argument 1 from 'std::ifstream* {aka std::basic_ifstream*}' to 'const std::basic_istream&' rcfile.ll:152:27: error: invalid cast from type 'std::istream {aka std::basic_istream}' to type 'std::ifstream* {aka std::basic_ifstream*}' if (!((ifstream*) yyin)->is_open ()) ^ Makefile:459: recipe for target 'rcfile.o' failed make[3]: *** [rcfile.o] Error 1 The full build log is attached; please do let me know if the problem is unreproducible, in which case I shall try to investigate further. Best, Michael mailfilter-build-log.txt.gz Description: application/gunzip signature.asc Description: PGP signature