benklop / microblog-purple

Automatically exported from code.google.com/p/microblog-purple
GNU General Public License v3.0
0 stars 0 forks source link

Add an option/a make target to build or not twitgin (to avoid X11/Gtk dependency) #224

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Install/compile pidgin without X11/Gtk support (finch for instance)
2. Try to compile mbpurple (plain make command)

What is the expected output? What do you see instead?
Expected: make does not attempt to compile twitgin

Currently make goes down into twitgin plugin and try to compile it, which fails 
because no graphical library is installed on my system.

What version of the product are you using? On what operating system?
x11-plugins/pidgin-mbpurple-0.3.0 on Gentoo amd64.

Please provide any additional information below.
The current solution used in Gentoo, is to patch the Makefile to remove twitgin 
from the SUBDIRS variable is the user does not want it.
A better solution could be either to have a basic configuration script, or to 
have a no-x11 make target to build only the non-graphical plugins.

Original issue reported on code.google.com by c.nicolas@gmail.com on 13 Jan 2011 at 1:57

GoogleCodeExporter commented 9 years ago
Hmm. this may be "half" a fix: 
- add a "configure.mak" in which the subdir for twitgin is defined.
- add the twitgin-subdir-var to SUBDIRS instead of twitgin.
- for build without twitgin: edit configure.mak and remove the twitgin subdir.

I know it's not much, but it's the shortest way I could think of. In a next 
step the configure.mak could be generated by a (hand-made) configure-script...

Comments?

Original comment by andresen.nils@gmail.com on 4 Feb 2011 at 8:33

Attachments:

GoogleCodeExporter commented 9 years ago
The initial idea is to have every configuration goes into global.mak. But 
anyway having configure.mak is not harmful. Patch merged in SVN.

Original comment by somsaks on 7 Feb 2011 at 9:28

GoogleCodeExporter commented 9 years ago
Issue 232 has been merged into this issue.

Original comment by andresen.nils@gmail.com on 21 Feb 2011 at 6:21