Hi,
Some comments concerning the new idlc preprocessor module (ucpp):
- The ucpp directory has a makefile and a patch under the LGPL3. These are new files so they should not carry the old license. Furthermore, patches should be under the same license of the upstream code.
- it appears this uses dmake which will be deprecated, for new modules we should use gbuild.
- it should be possible to use a preinstalled ucpp instead of rebuilding it.

1. you are right the license should be changed, i forgot this
2. the patch will be under AL because it creates only a makefile and nothing else. The original source are unpatched.
3. dmake is used because the patch mechanism is adapted to gbuild. This will change when it is available. Maybe i have missed it.
4. ucpp is very small and i simply build it every time instead of extending the configure script etc. Go ahead to change it if you like.

(In reply to comment #2)
> 1. you are right the license should be changed, i forgot this

Thanks.

> 2. the patch will be under AL because it creates only a makefile and nothing
> else. The original source are unpatched.

Yes, I noticed. The uccp makefile is pretty simple though and I noticed for other modules (hunspell) we do use the available makefile. I don't really "get" this build system so just take my suggestions with a grain of salt.

> 3. dmake is used because the patch mechanism is adapted to gbuild. This will
> change when it is available. Maybe i have missed it.

I am missing it too ;-).

> 4. ucpp is very small and i simply build it every time instead of extending the
> configure script etc. Go ahead to change it if you like.

Yes, maybe later. Thanks!

this is obsolete

closed