Bugzilla Bug 166500

I use conky with X11 disabled in order to be able to send its output to stdout.
After upgrading to conky-1.4.5 setting the "-X" USE flag seems to have no
effect anymore. It looks like the ebuild assumed --enable-X11 by default.

I have the "-X" flag set in package.use and also tried so set it directly on
the command-line but "conky -v" always gives:

Conky 1.4.5 compiled Mon Feb 12 15:01:49 MET 2007 for Linux 2.6.18-gentoo-r6
(x86_64)

Compiled in features:

 X11:
  * Xdamage extension
  * Xdbe extension (double buffer)
  * xft

 Music detection:

 General features:

Created an attachment (id=110736) [details]
app-admin/conky-1.4.5-r1.ebuild

The attached ebuild revision corrects the problem.  With -X, we were not
explicitly disabling X, but rather falling back to auto-detection.

-X will now strictly disable X and associated items.

Since this ebuild may build a different binary with no change in USE flags,
I am bumping it to -r1.  I believe that is the prescribed policy.

ebuild diff follows.

Created an attachment (id=110737) [details]
conky-1.4.5-r1.ebuild

cvs header cleared.

Created an attachment (id=110739) [details]
conky-1.4.5-r1.ebuild.diff

Daniel,

Note I also removed --enable-proc-uptime which has no effect.

Phil

Created an attachment (id=110740) [details]
app-admin/conky-1.4.5-r1.ebuild

putting --enable-proc-uptime back in, sry.  i pulled this in a later version.

Created an attachment (id=110741) [details]
app-admin/conky-1.4.5-r1.ebuild.diff

Created an attachment (id=111163) [details]
app-admin/conky-1.4.5-r1.ebuild

I wanted to try and sneak this in...its the same as the patch from Philip plus
I removed the runtime dep for mpd, since it is not necessary.

Created an attachment (id=111165) [details]
app-admin/conky-1.4.5-r1.ebuild.diff

I believe you can make the same claim for removing the RDEPEND on hddtemp.

In both cases, the USE flags (mpd or hddtemp) configure conky for use with the
respective package, however it is not *strictly* necessary for either package
to be installed in order to use conky.  If you do configure for either one,
however, the related conky variables are "dead" without the associated runtime
package.

Phil

I've now tested the new ebuild and it does what I need. X11 support is not
included now (when setting the -X USE variable) and conky then works the way I
want it to.

Thanks!

When will the new ebuild appear in the portage tree?

Anthony

*** Bug 169595 has been marked as a duplicate of this bug. ***

Thanks Anthony, Brenden and Philip and Pavel.

Pavel - realized bug 169595 wasn't quite a dup but i did fix it.

Philip - wrt a version bump - tough call.
Though its a new binary I'm guessing most users use it in a X mode. I didn't
want these users doing a recompile for no real benefit.
Weak argument but strictly speaking you're probably right with policy. I'm just
bending it slightly.

Confirmed fixed. Its already in the portage! Great job, guys!