[Xfce4-commits] <xfce4-dev-tools:master> Fix XDT_FEATURE_LINKER_OPTS and XDT_FEATURE_VISIBILITY.

[Brian J. Tarricone]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Ok, now that I'm not half asleep, looking at this again.

On 09/19/2009 08:36 AM, Jannis Pohlmann wrote:
>     
>     AC_ARG_ENABLE executes the code its last parameter if the --enable flag
>     is not provided, not if the --disable flag is not provided.

No, that's incorrect.  It's:

AC_ARG_ENABLE(feature, help-string, action-if-given, action-if-not-given)

It has nothing to do with whether you pass the enable or disable
variant.  "action-if-given" is run if you pass *either* --disable-foo or
- --enable-foo.  "action-if-not-given" is run if you don't pass anything.

> That's why
>     we need to set the value to "no", not to "yes".

Well, it depends.  You set the value in action-if-not-given to whatever
you want the default to be.  So your patch isn't right in that respect
since I intended 'yes' to be the default for the visibility and linker
options.

Now, there's the action-if-given block, which is blank.  That's ok in
this case because the shell variable matches the "shell safe" version of
the 'feature' argument ("linker-opts" -> "enable_linker_opts"), which
autoconf sets automatically.  It's a bit of behind-the-scenes autoconf
voodoo that makes things confusing, so I'll probably (redundantly) put
code there to set it explicitly.

>     Also don't show the --disable-debug option (with weird indentation)
>     because the help message already explains that there are --disable
>     variants for all --enable options.

Eh, I kinda liked that there.  Gonna put it back if I can figure out how
to fix the indent.

	-b
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)

iEYEARECAAYFAkq1Q+wACgkQ6XyW6VEeAnulswCfV51pfpbnQ3jksCIjPBXldLkK
pWUAoNM67Y8jVik7rqdlmd5ND8hBYf6F
=GdX9
-----END PGP SIGNATURE-----