Opened 7 years ago

Closed 7 years ago

Last modified 8 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 Changed 7 years ago by digitall

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 Changed 7 years ago by digitall

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 Changed 7 years ago by digitall

Keywords: build removed

comment:4 Changed 7 years ago by SF/reddwarf69

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

comment:5 Changed 7 years ago by sev-

Owner: set to sev-
Priority: normalblocker

comment:6 Changed 7 years ago by sev-

Raising priority. This is a release-critical bug.

comment:7 Changed 7 years ago by wjp

Owner: changed from sev- to wjp
Status: newclosed

comment:8 Changed 7 years ago by wjp

Confirmed and committed in 478fd0ed2957a766df2860c66fa476aa3665fc8f.

comment:9 Changed 7 years ago by lordhoto

Resolution: fixed

comment:10 Changed 8 months ago by digitall

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