x11-wm/xfce4-panel: Add DEBUG option

[koobs]

Summary

Proposed commit log:

x11-wm/xfce4-panel: Add DEBUG option

* Add DEBUG option to facilitate easier diagnosis of crashs by users [1]
* Group, sort and align OPTIONS_*, USE{S}, LICENSE* sections/directives
* Re-patch configure patch (makepatch compatible)

[1] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=206196

[olivierd]

For me the patch is rejected, because:

• DEBUG option should be also add to others compoments, at least its dependencies (see graph on wiki page [1]).
• In configure script debug options comes with several arguments, especially 'full', this one is more insteresting.

For cosmetic changes it's ok.

[1] https://wiki.freebsd.org/Xfce?action=AttachFile&do=view&target=xfce4.png

[koobs]

In D4922#104387, @olivierd wrote:

For me the patch is rejected, because:

• DEBUG option should be also add to others compoments, at least its dependencies (see graph on wiki page [1]).

It's perfectly fine to add a DEBUG option for an individual port, without requiring dependents to be modified (particularly and especially in the same commit)

• In configure script debug options comes with several arguments, especially 'full', this one is more insteresting.

I looked at this, it only adds -Werror, which we dont want and enabled no other tests. min is the default (now), 'yes' is the correct (and default) value when --enable-debug is set

[olivierd]

In D4922#104389, @koobs wrote:

In D4922#104387, @olivierd wrote:

For me the patch is rejected, because:

• DEBUG option should be also add to others compoments, at least its dependencies (see graph on wiki page [1]).

It's perfectly fine to add a DEBUG option for an individual port, without requiring dependents to be modified (particularly and especially in the same commit)

And others Xfce's ports? It's not coherent. Xfce is whole, why to add new option for this one and not others?

I would agree to have this option in Xfce USES framework.

Xfce 4.12 was published almost one year ago, I doubt suddenly xfce4-panel crashes. It was tested (use) by lots of users I think (and I regularly test with some libraries from GNOME devel repository especially for GTK+3, but also GLib 2.46.x, DBus 1.10.x and 1.11.x and nothing happens).

This bug already exists on the Xfce's Bugzilla, but without resolution.

https://bugzilla.xfce.org/show_bug.cgi?id=10344
https://bugzilla.xfce.org/show_bug.cgi?id=10616
https://bugzilla.xfce.org/show_bug.cgi?id=11300

• In configure script debug options comes with several arguments, especially 'full', this one is more insteresting.

I looked at this, it only adds -Werror, which we dont want and enabled no other tests. min is the default (now), 'yes' is the correct (and default) value when --enable-debug is set

[sbruno]

Is there further work to do here? It looks like this is maintainer rejected and the request is to move this feature to Mk/Uses/xfce.mk to enable DEBUG across the entire XFCE suite. Is this not correct?