Dear all, simply removing the "else" is IMHO not the correct fix. After prpopagating all changes from the development trunk to the PF_ROX branch, the problem persisted. I just checked in a fix, that basically re-applies some (local) changes in milprint_summer.c from the development trunk to the PF_ROX branch --- hopefully not messing up any changes intentionally made on the PF_ROX branch. Some simple initial analysis --- simply undoing all changes from the development trunk since the creation of the PF_ROX branch via `cvs up -jCurrent_PF_ROX_sync -jPF_ROX_base` triggers conflicts in 13 files: ======== compiler/algebra/opt/opt_algebra_cse.c compiler/compile.c compiler/debug/abssynprint.c compiler/debug/coreprint.c compiler/include/pathfinder.h compiler/mil/milgen.brg compiler/mil/milprint_summer.c compiler/semantics/heuristic.c compiler/sql/lalg2sql.brg compiler/utils/string_utils.c runtime/pathfinder.mx runtime/pf_support.mx src/sqlhelpers/serializer/pfserialize.pl ======= While some of these might be due to conflicting but intended changes on the PF_ROX branch, other might also be due to "lost" and/or failed propagations from the development trunk to the PF_ROX branch. Conclusions: 1) We need to crefully check, whether all changes on the development trunk since the creation of the PF_ROX branch have been properly propagated to the latter. 2) We should consider adding the PF_ROX branh tot he nightly yetsing --- at least testing the compilation. Stefan On Wed, May 21, 2008 at 01:22:02PM +0300, Lefteris wrote:

Yeap, just delete that else statement:) or you can merge the diff from the development branch.

On Wed, May 21, 2008 at 1:18 PM, Riham Abdel Kader wrote:

...

Hi all,

I'm getting the following error while compiling PF-ROX:

make[5]: Entering directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' /bin/sh ../../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I../.. -I. -I../include -I./../include -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/common -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/gdk -DLIBMIL -g -O2 -Wall -Wextra -std=c99 -Werror-implicit-function-declaration -Werror -Wpointer-arith -Wdeclaration-after-statement -Wundef -Wp,-D_FORTIFY_SOURCE=2 -Wno-declaration-after-statement -c -o libmil_la-milprint_summer.lo `test -f 'milprint_summer.c' || echo './'`milprint_summer.c gcc -DHAVE_CONFIG_H -I. -I. -I../.. -I. -I../include -I./../include -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/common -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/gdk -DLIBMIL -g -O2 -Wall -Wextra -std=c99 -Werror-implicit-function-declaration -Werror -Wpointer-arith -Wdeclaration-after-statement -Wundef -Wp,-D_FORTIFY_SOURCE=2 -Wno-declaration-after-statement -c milprint_summer.c -fPIC -DPIC -o .libs/libmil_la-milprint_summer.o milprint_summer.c: In function 'translateFunction': milprint_summer.c:8518: error: expected expression before 'else' make[5]: *** [libmil_la-milprint_summer.lo] Error 1 make[5]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' make[4]: *** [install] Error 2 make[4]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' make[3]: *** [install-recursive] Error 1 make[3]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler' make[2]: *** [install] Error 2 make[2]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler' make[1]: *** [install-recursive] Error 1 make[1]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder' make: *** [install] Error 2

I checked out all needed modules yesterday night.

Best regards, Riham

-- Address: Database Group, Dept of EEMCS, University of Twente, POBox 217, 7500 AE Enschede, The Netherlands Phone: +31 (0)53 489 4628 Fax: +31 (0)53 489 2927 Email: r.abdelkader@ewi.utwente.nl

------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers

------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers

-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4312 |

Dear all, I checked all (modified) files in the PF_ROX branch by hand, and all but runtime/pathfinder.mx & runtime/pf_support.mx (this means in particular the compiler/mil/milprint_summer.c I fixed the other day) are OK, i.e., they contain all (intended) changes applied in the PF_ROX branch as well as all chnages that were made on the development trunk since the creation of the PF_ROX branch (and hence properly propagated). For runtime/pathfinder.mx & runtime/pf_support.mx, I am not yet sure, whether everything is OK. There have been numerous "interleaved" (and partly "redundant" or "conflicting") changes both genuine on the PF_ROX branch and propagated from the development trunk, and the (very tedious) analysis requires more time than I could spent today. More to come "soon", I hope ... Unfortunately, while only 50 (100) tests fail in the new stable branch (and in the development trunk), more than 500 tests seem to fail in the PF_ROX branch --- also this requires "some" more ananlysis ... Stefan On Sun, May 25, 2008 at 01:06:24PM +0300, Lefteris wrote:

Simply deleting the else statement is *not* the correct solution, that is why I never checked in that. But, since this is in the pftijah code and since Riham does not use pftijah for PFROX it was an easy quick solution for her so whe could compile and continue with her work.

The correct solution is to fix the brackets of the if-else statements.

On Sun, May 25, 2008 at 12:58 PM, Stefan Manegold wrote:

...

Dear all,

simply removing the "else" is IMHO not the correct fix. After prpopagating all changes from the development trunk to the PF_ROX branch, the problem persisted. I just checked in a fix, that basically re-applies some (local) changes in milprint_summer.c from the development trunk to the PF_ROX branch --- hopefully not messing up any changes intentionally made on the PF_ROX branch.

Some simple initial analysis --- simply undoing all changes from the development trunk since the creation of the PF_ROX branch via `cvs up -jCurrent_PF_ROX_sync -jPF_ROX_base` triggers conflicts in 13 files: ======== compiler/algebra/opt/opt_algebra_cse.c compiler/compile.c compiler/debug/abssynprint.c compiler/debug/coreprint.c compiler/include/pathfinder.h compiler/mil/milgen.brg compiler/mil/milprint_summer.c compiler/semantics/heuristic.c compiler/sql/lalg2sql.brg compiler/utils/string_utils.c runtime/pathfinder.mx runtime/pf_support.mx src/sqlhelpers/serializer/pfserialize.pl =======

While some of these might be due to conflicting but intended changes on the PF_ROX branch, other might also be due to "lost" and/or failed propagations from the development trunk to the PF_ROX branch.

Conclusions:

1) We need to crefully check, whether all changes on the development trunk since the creation of the PF_ROX branch have been properly propagated to the latter.

2) We should consider adding the PF_ROX branh tot he nightly yetsing --- at least testing the compilation.

Stefan

On Wed, May 21, 2008 at 01:22:02PM +0300, Lefteris wrote:

...

Yeap, just delete that else statement:) or you can merge the diff from the development branch.

On Wed, May 21, 2008 at 1:18 PM, Riham Abdel Kader wrote:

...

Hi all,

I'm getting the following error while compiling PF-ROX:

make[5]: Entering directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' /bin/sh ../../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I../.. -I. -I../include -I./../include -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/common -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/gdk -DLIBMIL -g -O2 -Wall -Wextra -std=c99 -Werror-implicit-function-declaration -Werror -Wpointer-arith -Wdeclaration-after-statement -Wundef -Wp,-D_FORTIFY_SOURCE=2 -Wno-declaration-after-statement -c -o libmil_la-milprint_summer.lo `test -f 'milprint_summer.c' || echo './'`milprint_summer.c gcc -DHAVE_CONFIG_H -I. -I. -I../.. -I. -I../include -I./../include -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/common -I/local/kader/home/workspace_pathfinder/MonetDB_XQuery/include/MonetDB/gdk -DLIBMIL -g -O2 -Wall -Wextra -std=c99 -Werror-implicit-function-declaration -Werror -Wpointer-arith -Wdeclaration-after-statement -Wundef -Wp,-D_FORTIFY_SOURCE=2 -Wno-declaration-after-statement -c milprint_summer.c -fPIC -DPIC -o .libs/libmil_la-milprint_summer.o milprint_summer.c: In function 'translateFunction': milprint_summer.c:8518: error: expected expression before 'else' make[5]: *** [libmil_la-milprint_summer.lo] Error 1 make[5]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' make[4]: *** [install] Error 2 make[4]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler/mil' make[3]: *** [install-recursive] Error 1 make[3]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler' make[2]: *** [install] Error 2 make[2]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder/compiler' make[1]: *** [install-recursive] Error 1 make[1]: Leaving directory `/local/kader/home/workspace_pathfinder/MonetDB_XQuery/pathfinder' make: *** [install] Error 2

I checked out all needed modules yesterday night.

Best regards, Riham

-- Address: Database Group, Dept of EEMCS, University of Twente, POBox 217, 7500 AE Enschede, The Netherlands Phone: +31 (0)53 489 4628 Fax: +31 (0)53 489 2927 Email: r.abdelkader@ewi.utwente.nl

------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers

------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers

-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4312 |

------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers

-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4312 |