Hi,

Am 16.08.2010 10:44, schrieb IOhannes m zmoelnig:
> hi
>
> in the course of packaging gmerlin-avdecoder for debian, we noticed that
> gmerlin-avdecoder has some files that have a 4-clause BSD code pasted
> in, which - unfortunately - is incompatible with the GPL.
>
> these are:
>   lib/base64.c
>   lib/os.c
>   include/bgav_sem.h
>

The os.c was added by august while porting gmerlin-avdecoder to windows.

I just found another problem: For the semaphores, the implementation is missing
as well. For linux, we can use the system supplied semaphores. But for OSX,
the sem_* functions are present but non-working. This basically means
that using RTSP on OSX doesn't work right now.

So if someone finds a GPLed semaphore implementation, that would be great as
well (sometimes I feel like I'm the only one who uses semaphores at all).

> both os.c and bgav_sem.h use BSD code only as a replacement if certain
> system features cannot be detected by autoconf (semaphore.h resp.
> inet_aton() not present)
> this would be relatively easy to fix, e.g. by factoring the code out
> into separate files that only include the BSD code, so debian's
> mandatory .orig.tar.gz ("upstream tarball") can simply exclude these files.
>
> the base64.c is a little bit more complicated, as it is the sole
> implementation of the base64 encoders/decoders, and it's only available
> under BSD (in the gmerlin project of course)
>
> it would be necessary to replace the entire file by GPL'ed code.
>
> would it be ok if i create a patch that would implement these changes?

Yes, if you find a GPLed drop-in replacement for base64 en-/decoding, that
would be great.

> something i should take special care of?

base64 is used for http authentication and some RTSP variants, so these should
work afterwards.

Burkhard

------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev 
_______________________________________________
Gmerlin-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gmerlin-general

Reply via email to