Moodle

moodle contains GPL and PHP licensed files which appear to link

Details

  • Type: Task Task
  • Status: Reopened Reopened
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.6.1, 1.6.2
  • Fix Version/s: None
  • Component/s: General, Libraries
  • Labels:
    None
  • Environment:
    Linux
  • Database:
    Any
  • Affected Branches:
    MOODLE_16_STABLE

Description

FYI: I'm a packager for the openSUSE distribution and want to create a moodle package. But the legal department stops me with the following comment. Can you give me a "clean" answer here so we can get packages without legal problems?

"There are numerous GPL licensed PHP files as well as quite a few PHP licensed php files such as those in the lib/pear/ directory. Are these files executed in the same address space on the server?

If so, mixing both only works if the "derivative work" is GPL licensed."

As I can see in the sources, the problematic files and the responsible persons:
Martin Jansen mj@php.net
Richard Tango-Lowy <richtl@arscognita.com>
=> moodle/lib/adodb/pear/Auth/Container/ADOdb.php

Sterling Hughes <sterling@php.net>
Stig Bakken <ssb@php.net>
Tomas V.V.Cox <cox@idecnet.com>
Greg Beaver <cellog@php.net>
=> moodle/lib/pear/PEAR.php

Issue Links

Activity

Hide
Martin Dougiamas added a comment -

Hi,

have you seen the README regarding this?

http://moodle.cvs.sourceforge.net/moodle/moodle/lib/pear/README.txt?revision=1.3&view=markup

We've gotten permission from some authors and are waiting on the rest.

We only package these classes to use if PEAR itself is not present. They are completely unmodified copies of PEAR modules.
If PEAR is present we just use that and these are ignored.

So if the "unfinished" nature of our enquiries bothers you and SuSE includes PEAR by default then you can simply exclude the whole lib/pear directory from Moodle.

Show
Martin Dougiamas added a comment - Hi, have you seen the README regarding this? http://moodle.cvs.sourceforge.net/moodle/moodle/lib/pear/README.txt?revision=1.3&view=markup We've gotten permission from some authors and are waiting on the rest. We only package these classes to use if PEAR itself is not present. They are completely unmodified copies of PEAR modules. If PEAR is present we just use that and these are ignored. So if the "unfinished" nature of our enquiries bothers you and SuSE includes PEAR by default then you can simply exclude the whole lib/pear directory from Moodle.
Hide
Lars Rupp added a comment -

Yes, I've seen the README.txt and waiting for the answers, too...

...but as SuSE comes with the main PEAR class, I can live with removing the lib/pear directory from the package and put a "requires: pear" in the specfile. I ask because I don't like patching sources which normally not really need a patch.

So please keep this on your "ToDo"-List so that we can close this as "fixed" as soon as possible, please.

Show
Lars Rupp added a comment - Yes, I've seen the README.txt and waiting for the answers, too... ...but as SuSE comes with the main PEAR class, I can live with removing the lib/pear directory from the package and put a "requires: pear" in the specfile. I ask because I don't like patching sources which normally not really need a patch. So please keep this on your "ToDo"-List so that we can close this as "fixed" as soon as possible, please.
Hide
Martin Dougiamas added a comment -

We'll do our best, but it does depend on those authors getting back to us

Show
Martin Dougiamas added a comment - We'll do our best, but it does depend on those authors getting back to us
Hide
Martín Langhoff added a comment -

There's an additional file from the PEAR repo we'll want for the upcoming WebDAV support - it's System.php, by Tomas V.V.Cox <cox@idecnet.com> (one of the guys above).

What's the status of this?

Show
Martín Langhoff added a comment - There's an additional file from the PEAR repo we'll want for the upcoming WebDAV support - it's System.php, by Tomas V.V.Cox <cox@idecnet.com> (one of the guys above). What's the status of this?
Hide
Martín Langhoff added a comment -

bump :-|

Show
Martín Langhoff added a comment - bump :-|
Hide
Martin Dougiamas added a comment -

Martín, I thought you already got permission for System.php?

Show
Martin Dougiamas added a comment - Martín, I thought you already got permission for System.php?

People

Vote (0)
Watch (4)

Dates

  • Created:
    Updated: