Since the autobuild always performs a clean build, I suppose it means that someone might have broken the build recently.

Maybe it's one of those "works for me on Linux" guys.

bob

On 11/23/2008 6:01 PM, kaver wrote:

the last three days Win builds have been 81 bytes each. This suggests a huge code compaction. Otherwise there may be a problem. Or am I doing something dumber than normal? Erik

---

---

This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK& win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/

---

---

Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel

I don't see this, so I will try to make a clean build.

-----Original Message----- From: Bob Jamison [mailto:rwjj@...127...] Sent: 24 November, 2008 11:18 To: kaver; inkscape Subject: Re: [Inkscape-devel] win builds

On 11/23/2008 9:55 PM, Bob Jamison wrote:

...

Since the autobuild always performs a clean build, I suppose it means that someone might have broken the build recently.

Maybe it's one of those "works for me on Linux" guys. :-)

bob

Apparently this is the problem. If this is another dependency, than that would be a problem.

src/display/nr-filter-gaussian.cpp:22:17: error: omp.h: No such file or directory src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int)': src/display/nr-filter-gaussian.cpp:276: error: there are no arguments to 'omp_get_thread_num' that depend on a template parameter, so a declaration of 'omp_get_thread_num' must be available src/display/nr-filter-gaussian.cpp:276: error: (if you use '-fpermissive', G++ will accept your code, but allowing the use of an undeclared name is deprecated) src/display/nr-filter-gaussian.cpp: In member function 'virtual int NR::FilterGaussian::render(NR::FilterSlot&, const NR::FilterUnits&)': src/display/nr-filter-gaussian.cpp:551: error: 'omp_get_num_procs' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 1u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:302: warning: comparison of unsigned expression < 0 is always false src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:317: warning: comparison of unsigned expression < 0 is always false src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 3u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:646: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 4u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:649: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 4u, bool PREMULTIPLIED_ALPHA = true]': src/display/nr-filter-gaussian.cpp:652: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope

bob

---

This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel

On 11/23/2008 11:30 PM, Preben Soeberg wrote:

I am making a clean build now and have come beyond the point you show below without errors.

I wonder, maybe you didn't notice, that there was a new version of buildtool.cpp to be compiled?

Yes, I took care of that a few days ago.

However, that does not explain the dependency on <omp.h>. Did someone assume that it is available everywhere?

bob

...

-----Original Message----- From: Bob Jamison [mailto:rwjj@...127...] Sent: 24 November, 2008 11:18 To: kaver; inkscape Subject: Re: [Inkscape-devel] win builds

On 11/23/2008 9:55 PM, Bob Jamison wrote:

...

Since the autobuild always performs a clean build, I suppose it means that someone might have broken the build recently.

Maybe it's one of those "works for me on Linux" guys. :-)

bob

Apparently this is the problem. If this is another dependency, than that would be a problem.

src/display/nr-filter-gaussian.cpp:22:17: error: omp.h: No such file or directory src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int)': src/display/nr-filter-gaussian.cpp:276: error: there are no arguments to 'omp_get_thread_num' that depend on a template parameter, so a declaration of 'omp_get_thread_num' must be available src/display/nr-filter-gaussian.cpp:276: error: (if you use '-fpermissive', G++ will accept your code, but allowing the use of an undeclared name is deprecated) src/display/nr-filter-gaussian.cpp: In member function 'virtual int NR::FilterGaussian::render(NR::FilterSlot&, const NR::FilterUnits&)': src/display/nr-filter-gaussian.cpp:551: error: 'omp_get_num_procs' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 1u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:302: warning: comparison of unsigned expression< 0 is always false src/display/nr-filter-gaussian.cpp:643: instantiated from here src/display/nr-filter-gaussian.cpp:317: warning: comparison of unsigned expression< 0 is always false src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 3u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:646: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 4u, bool PREMULTIPLIED_ALPHA = false]': src/display/nr-filter-gaussian.cpp:649: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope src/display/nr-filter-gaussian.cpp: In function 'void NR::filter2D_IIR(PT*, int, int, const PT*, int, int, int, int, const IIRValue*, const double*, IIRValue* const*, int) [with PT = unsigned char, unsigned int PC = 4u, bool PREMULTIPLIED_ALPHA = true]': src/display/nr-filter-gaussian.cpp:652: instantiated from here src/display/nr-filter-gaussian.cpp:276: error: 'omp_get_thread_num' was not declared in this scope

bob

Have you any idea, what went wrong with my compilation of simple-node.cpp?

And why I can compile it now without any problem?

When I look in the source now, I can't see any changes done that should trigger the error, I mentioned in the other post. Could there have been some kind of glitch in SVN?

Preben

-----Original Message----- From: Ted Gould [mailto:ted@...11...] Sent: 24 November, 2008 13:01 To: Bob Jamison Cc: Preben Soeberg; inkscape Subject: Re: [Inkscape-devel] win builds

On Sun, 2008-11-23 at 23:45 -0600, Bob Jamison wrote:

...

On 11/23/2008 11:30 PM, Preben Soeberg wrote:

...

I am making a clean build now and have come beyond the point you show

below

...

...

without errors.

I wonder, maybe you didn't notice, that there was a new version of buildtool.cpp to be compiled?

Yes, I took care of that a few days ago.

However, that does not explain the dependency on <omp.h>. Did someone assume that it is available everywhere?

Yes. I committed Jasper's OpenMP patch with a modified version of Felipe's build files. I believe you find instructions on getting it to compile in Windows in Jasper's original e-mail on 18 Nov 2008. Of course, of you build for Windows using autotools it will check for the files and error appropriately. It is my understanding that this file should come with the compiler.

--Ted

Thanks, I am on XP, but

When I saw the problems compiling and the weirdness of the SVN log after updating to rev. 20247, I did take a backup of my working copy. It says:

xml/simple-node.cpp 2008/11/22 10:07 (my timestamps are GMT +7) This file does NOT contain '#include "xml/node.h"' I am also having a SVN log file in the back up telling me that xml/simple-node.cpp was updated in SVN in rev. 20245.

My current version of xml/simple-node.cpp has a timestamp xml/simple-node.cpp 2008/11/22 15:17, and this file DOES contain '#include "xml/node.h"' (and I did not modify this file).

In the meantime, I reverted to an older rev., and then back to the newest again. The SVN log shows only one update of xml/simple-node.cpp since rev. 19917.

In my eyes, it appears as an SVN glitch.

Preben

-----Original Message----- From: Ted Gould [mailto:ted@...11...] Sent: 24 November, 2008 21:56 To: Preben Soeberg Cc: inkscape-devel@lists.sourceforge.net Subject: RE: [Inkscape-devel] win builds

On Mon, 2008-11-24 at 13:26 +0700, Preben Soeberg wrote:

...

Have you any idea, what went wrong with my compilation of simple-

node.cpp?

...

And why I can compile it now without any problem?

When I look in the source now, I can't see any changes done that should trigger the error, I mentioned in the other post. Could there have been

some

...

kind of glitch in SVN?

I imagine that it was more a glitch in your build. That's one of the reason that I always build in another directory. Here's my basic workflow.

$ bzr branch trunk mywork
$ cd mywork
$ ./autogen.sh
$ mkdir build
$ cd build
$ ../configure --prefix=`pwd`/../install --enable-inkboard
$ nice make -j 2 install

That puts all of the built files in the "build" directory but also sets up a clean install that isn't connected to the other instances of Inkscape on my system.

--Ted

running Win32 nightly build rev 20288 the problem is solved, Thanks!

On 11/24/2008 2:11 AM, Jasper van de Gronde wrote:

Ted Gould wrote:

...

On Sun, 2008-11-23 at 23:45 -0600, Bob Jamison wrote:

...

Yes, I took care of that a few days ago.

However, that does not explain the dependency on<omp.h>. Did someone assume that it is available everywhere?

Yes. I committed Jasper's OpenMP patch with a modified version of Felipe's build files. I believe you find instructions on getting it to compile in Windows in Jasper's original e-mail on 18 Nov 2008. Of course, of you build for Windows using autotools it will check for the files and error appropriately. It is my understanding that this file should come with the compiler.

Yes, it does, but only with (very) recent versions of MingW, so you need the mingw package that is linked to from here: http://wiki.inkscape.org/wiki/index.php/Win32Port

(At least until the official MingW distributions finally get up to speed with the latest GCC versions.)

Yes, I looked at the compilers, and anyone with the inclination would need the latest gcc source for which MinGW has a patch on their site.

I'll get to it eventually.

bob