Opened 8 years ago

Closed 7 years ago

Last modified 13 months ago

#6038 closed defect (fixed)

xcode.cpp needs a license clarification

Reported by: SF/reddwarf69 Owned by: wjp
Priority: blocker Component: Tools
Keywords: Cc:
Game:

Description

devtools/create_project/xcode.cpp includes the text: "Heavily modified (not in a good way) function, imported from QMake XCode project generator (licensed under the QT license)".

The problem is there is no such a thing as a "QT license". Qt has been distributed in various licenses over the years, and not all of them where free or compatible with the GPL.

Ticket imported from: #3527268. Ticket imported from: bugs/6038.

Change History (10)

comment:1 by digitall, 8 years ago

To be clear, the devtools folders contain supplementary tools and are not generally necessary for building ScummVM. This file is part of the create_project tool, which builds IDE project files from our base POSIX style configure scripts to aid developers using IDEs and those platforms where the standard toolchain uses non-POSIX systems i.e. MSVC etc.

comment:2 by digitall, 8 years ago

This code was added to our tree in commit 9854f1b48667b082f9ee1c377dcba5b44e92827b by littleboy as part of developing adding XCode output functionality to create_project, to aid in developers on Mac OSX. This occurred on 2011-06-02 19:09:46. By comparing the relevant code module in ScummVM: https://github.com/scummvm/scummvm/blob/master/devtools/create_project/xcode.cpp with the QT Qmake files, the code is based on/derived from: https://qt.gitorious.org/qt/qt/blobs/4.8/qmake/generators/mac/pbuilder_pbx.cpp

According to the header, this Qmake file can be used under various licenses including LGPL-2.1 and GPL-3.0. IANAL, but if I am reading this correctly, the LGPL-2.1 code is compatible with GPL-2 i.e. our nominal license and thus no licensing issue should exist here.

comment:3 by digitall, 8 years ago

Keywords: build removed

comment:4 by SF/reddwarf69, 8 years ago

I agree. If the file could be updated with this information I would consider the bug fixed.

comment:5 by sev-, 7 years ago

Owner: set to sev-
Priority: normalblocker

comment:6 by sev-, 7 years ago

Raising priority. This is a release-critical bug.

comment:7 by wjp, 7 years ago

Owner: changed from sev- to wjp
Status: newclosed

comment:8 by wjp, 7 years ago

Confirmed and committed in 478fd0ed2957a766df2860c66fa476aa3665fc8f.

comment:9 by lordhoto, 7 years ago

Resolution: fixed

comment:10 by digitall, 13 months ago

Component: Tools
Note: See TracTickets for help on using tickets.