[Geany-devel] geany-plugins: Autotools usage

Quentin Glidic sardemff7+geany at xxxxx
Sun Apr 22 16:55:16 UTC 2012


On 22/04/2012 18:21, Chow Loong Jin wrote:
> On 22/04/2012 20:09, Quentin Glidic wrote:
>> Hello,
>> 
>> I noticed a global misuse / misunderstanding of the Autotools 
>> (mostly about the dist mechanism) among the geany-plugins.
> 
> Could you elaborate, please?
> 
> I noticed that you have shifted all the ENABLE_$PLUGIN conditionals 
> to the top-level Makefile.am instead, by modifying the SUBDIRS 
> variable there. However, this is not desirable as disabled plugins 
> will be excluded from the release tarball generated by `make dist'.
> 
> --enable/disable-$plugin should only toggle the building of the 
> actual plugin.

That’s exactly the misunderstanding I was speaking about. From the
Automake manual:

“If SUBDIRS is defined conditionally using Automake conditionals,
Automake will define DIST_SUBDIRS automatically from the possible values
of SUBDIRS in all conditions.”

And here’s the diff of the "tar tf" on the make dist tarball:

--- before-patch        2012-04-22 18:33:34.080440186 +0200
+++ after-patch 2012-04-22 18:33:48.168440399 +0200
@@ -127,0 +128 @@
+geany-plugins-1.22/debugger/wscript_configure
@@ -201,0 +203 @@
+geany-plugins-1.22/debugger/wscript_build
@@ -219,0 +222 @@
+geany-plugins-1.22/multiterm/wscript_configure
@@ -243,0 +247 @@
+geany-plugins-1.22/multiterm/wscript_build
@@ -310,0 +315 @@
+geany-plugins-1.22/devhelp/wscript_configure
@@ -367,0 +373 @@
+geany-plugins-1.22/devhelp/wscript_build
@@ -695,0 +702 @@
+geany-plugins-1.22/geanylua/wscript_configure
@@ -749,0 +757 @@
+geany-plugins-1.22/geanylua/wscript_build
@@ -914,0 +923 @@
+geany-plugins-1.22/webhelper/wscript_configure
@@ -934,0 +944 @@
+geany-plugins-1.22/webhelper/wscript_build

Automake handles the conditionnal perfectly by itself, and just need
some "dist_" prefixes here and there, while EXTRA_DIST should be
reserved for files that are used outside of automake scope.

-- 

Quentin "Sardem FF7" Glidic





More information about the Devel mailing list