After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 498818 - [PATCH] Add libggz and ggz-client-libs to garnome
[PATCH] Add libggz and ggz-client-libs to garnome
Status: RESOLVED FIXED
Product: GARNOME
Classification: Deprecated
Component: general
2.21.x
Other Linux
: Normal normal
: ---
Assigned To: GARNOME Maintainers
garnome list
Depends on:
Blocks:
 
 
Reported: 2007-11-21 19:53 UTC by Andreas Røsdal
Modified: 2008-03-27 13:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add libggz and ggz-client-libs to garnome. (2.42 KB, patch)
2007-11-21 19:54 UTC, Andreas Røsdal
needs-work Details | Review

Description Andreas Røsdal 2007-11-21 19:53:34 UTC
libggz, ggz-client-libs have been accepted as external dependencies to GNOME
2.22. The attached patch contains adds libggz and ggz-client-libs as dependencies to gnome-games.


See: http://live.gnome.org/TwoPointTwentyone/ExternalDependencies
Comment 1 Andreas Røsdal 2007-11-21 19:54:21 UTC
Created attachment 99450 [details] [review]
Add libggz and ggz-client-libs to garnome.
Comment 2 Andreas Røsdal 2007-11-25 11:57:46 UTC
Can I go ahead and commit the patch?
Comment 3 Karsten Bräckelmann 2007-11-27 22:11:02 UTC
Andreas, thanks for the heads-up. And of course the patch. Most appreciated! :)

However, the patch can not be committed as is. The desktop/ meta garball is for modules that are actually part of the GNOME Desktop suite. External dependencies need to live in bootstrap/ (or freedesktop/ if applicable).
Comment 4 Andreas Røsdal 2007-11-27 22:36:24 UTC
Sorry, I didn't realize that. It should be trivial to modify the patch so ggz is located in the bootstrap directory.
Comment 5 Karsten Bräckelmann 2007-11-27 22:50:15 UTC
Of course that change is trivial. :)

I just didn't do it myself yet, because I am low on time and currently busy preparing a stable release, due tomorrow... Figured I should answer at least, so resorted to explaining and marking the patch appropriately.
Comment 6 Andreas Røsdal 2008-01-03 18:24:23 UTC
How is it doing with this change request, Karsten?
Comment 7 Karsten Bräckelmann 2008-03-27 13:27:14 UTC
Err... Sorry for the delay. This has been fixed a while ago. :)