CMake should offer a switch to disable GGZ support even if GGZ is found

Bug #551716 reported by Jens Beyer
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Unassigned

Bug Description

CMake should offer a switch to disable GGZ support even if GGZ is found

This will reduce dependencies for people who want a standalone application, and don't even think about something called GGZ.

Related branches

Revision history for this message
SirVer (sirver) wrote :

I am very reluctant to offer optional features in widelands. This will result in a lot of hassle with bug reports if different kind of widelands are floating around. I'd vote for keeping ggz mandatory.

Revision history for this message
Jens Beyer (qcumber-some) wrote :

GGZ is not mandatory at the moment. If the user compiles from a source package, and GGZ is not found, it is simply disabled.

Revision history for this message
SirVer (sirver) wrote :

GGZ should be mandatory. We want all players to have the possibility to play on the internetz. GGZ is easy on linux & mac os x and window libraries are made available by jari, so installing GGZ should be not too difficult. I vote for closing this as Won't fix and making GGZ mandatory.

Changed in widelands:
status: New → Incomplete
Revision history for this message
Jens Beyer (qcumber-some) wrote :

As nobody else seems to have a dedicated opinion here, I'll do as specified and make GGZ mandatory for Build16. This will take place in the spice-up-cmake branch first.

Changed in widelands:
status: Incomplete → In Progress
assignee: nobody → Jens Beyer (Qcumber-some) (qcumber-some)
Changed in widelands:
status: In Progress → Fix Committed
assignee: Jens Beyer (Qcumber-some) (qcumber-some) → nobody
Revision history for this message
SirVer (sirver) wrote :

I set this to In Progress as the fix is not yet in trunk.

Changed in widelands:
milestone: none → build16-rc1
status: Fix Committed → In Progress
Revision history for this message
Jens Beyer (qcumber-some) wrote :

Available in spice-up-cmake branch.

Nasenbaer (nasenbaer)
Changed in widelands:
importance: Undecided → Low
Changed in widelands:
status: In Progress → Fix Committed
Revision history for this message
SirVer (sirver) wrote :

Released in build16-rc1

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.